You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by rp...@apache.org on 2007/08/27 13:28:23 UTC
svn commit: r570074 - /httpd/httpd/branches/2.0.x/STATUS
Author: rpluem
Date: Mon Aug 27 04:28:22 2007
New Revision: 570074
URL: http://svn.apache.org/viewvc?rev=570074&view=rev
Log:
* Cast vote and add some comments
Modified:
httpd/httpd/branches/2.0.x/STATUS
Modified: httpd/httpd/branches/2.0.x/STATUS
URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.0.x/STATUS?rev=570074&r1=570073&r2=570074&view=diff
==============================================================================
--- httpd/httpd/branches/2.0.x/STATUS (original)
+++ httpd/httpd/branches/2.0.x/STATUS Mon Aug 27 04:28:22 2007
@@ -165,6 +165,10 @@
adjusted for 2.0 including both patches;
http://people.apache.org/~wrowe/r568779-backport-2.0-r2.patch
+1: wrowe
+ rpluem says: What is the reason for the patch to Makefile.in in your
+ proposal? Besides I think
+ sed 's#i^LIBTOOL = \(.*\)#LIBTOOL = $(SHELL) $(installbuilddir)/libtool $(LTFLAGS)#'
+ is wrong (Note the i before ^).
and prior to r569934;
+1: rpluem
@@ -175,6 +179,9 @@
Backported to 2.0 (on top of log core patch, above);
http://people.apache.org/~wrowe/r569535-backport-2.0.patch
+1: wrowe
+ -1: rpluem: It does not apply to 2.0. Even if
+ http://people.apache.org/~wrowe/r568326-backport-2.0-r2.patch
+ is applied first.
* mod_ssl: Version reporting update; displays 'compiled against'
Apache and build-time SSL Library versions at loglevel [info],
@@ -185,7 +192,7 @@
http://svn.apache.org/viewvc?view=rev&revision=520701
Backported to 2.0 (applies with a bit of fuzz);
http://people.apache.org/~wrowe/r520701-backport-2.2.patch
- +1: wrowe
+ +1: wrowe, rpluem
PATCHES TO BACKPORT THAT ARE ON HOLD OR NOT GOING ANYWHERE SOON:
Re: svn commit: r570074 - /httpd/httpd/branches/2.0.x/STATUS
Posted by "William A. Rowe, Jr." <wr...@rowe-clan.net>.
Ruediger Pluem wrote:
>
> On 08/27/2007 09:17 PM, William A. Rowe, Jr. wrote:
>> Yes, I'm a vim user who always forgets modern linux is happy to navigate and
>> stay in insert mode ;-) That file will not be applied, please ignore (it's
>
> I thought just that ;-). Happened to me hundred times before. Especially if you
> work alternately with vi on Solaris and vi (aka vim) on Linux.
Worse, vi on hpux where you have to cancel insert mode to avoid lots of extra
spurious commands on arrow keys.
Re: svn commit: r570074 - /httpd/httpd/branches/2.0.x/STATUS
Posted by Ruediger Pluem <rp...@apache.org>.
On 08/27/2007 09:17 PM, William A. Rowe, Jr. wrote:
> rpluem@apache.org wrote:
>> Modified: httpd/httpd/branches/2.0.x/STATUS
>> URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.0.x/STATUS?rev=570074&r1=570073&r2=570074&view=diff
>> ==============================================================================
>> --- httpd/httpd/branches/2.0.x/STATUS (original)
>> +++ httpd/httpd/branches/2.0.x/STATUS Mon Aug 27 04:28:22 2007
>> @@ -165,6 +165,10 @@
>> adjusted for 2.0 including both patches;
>> http://people.apache.org/~wrowe/r568779-backport-2.0-r2.patch
>> +1: wrowe
>> + rpluem says: What is the reason for the patch to Makefile.in in your
>> + proposal? Besides I think
>> + sed 's#i^LIBTOOL = \(.*\)#LIBTOOL = $(SHELL) $(installbuilddir)/libtool $(LTFLAGS)#'
>> + is wrong (Note the i before ^).
>> and prior to r569934;
>> +1: rpluem
>
> Yes, I'm a vim user who always forgets modern linux is happy to navigate and
> stay in insert mode ;-) That file will not be applied, please ignore (it's
I thought just that ;-). Happened to me hundred times before. Especially if you
work alternately with vi on Solaris and vi (aka vim) on Linux.
Regards
Rüdiger
Re: svn commit: r570074 - /httpd/httpd/branches/2.0.x/STATUS
Posted by "William A. Rowe, Jr." <wr...@rowe-clan.net>.
rpluem@apache.org wrote:
> Modified: httpd/httpd/branches/2.0.x/STATUS
> URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.0.x/STATUS?rev=570074&r1=570073&r2=570074&view=diff
> ==============================================================================
> --- httpd/httpd/branches/2.0.x/STATUS (original)
> +++ httpd/httpd/branches/2.0.x/STATUS Mon Aug 27 04:28:22 2007
> @@ -165,6 +165,10 @@
> adjusted for 2.0 including both patches;
> http://people.apache.org/~wrowe/r568779-backport-2.0-r2.patch
> +1: wrowe
> + rpluem says: What is the reason for the patch to Makefile.in in your
> + proposal? Besides I think
> + sed 's#i^LIBTOOL = \(.*\)#LIBTOOL = $(SHELL) $(installbuilddir)/libtool $(LTFLAGS)#'
> + is wrong (Note the i before ^).
> and prior to r569934;
> +1: rpluem
Yes, I'm a vim user who always forgets modern linux is happy to navigate and
stay in insert mode ;-) That file will not be applied, please ignore (it's
the proposed fix to solve vpath with external apr junk.)
> @@ -175,6 +179,9 @@
> Backported to 2.0 (on top of log core patch, above);
> http://people.apache.org/~wrowe/r569535-backport-2.0.patch
> +1: wrowe
> + -1: rpluem: It does not apply to 2.0. Even if
> + http://people.apache.org/~wrowe/r568326-backport-2.0-r2.patch
> + is applied first.
Reviewing, and will update, thank you.