Mailing List Archive: Apache: Dev

Re: svn commit: r791178 - /httpd/httpd/branches/2.2.x/STATUS

Index | Next | Previous | View Threaded

nick at webthing

Jul 4, 2009, 3:16 PM

Post #1 of 2 (204 views)
Permalink

Re: svn commit: r791178 - /httpd/httpd/branches/2.2.x/STATUS

rpluem[at]apache.org wrote:

> @@ -157,17 +157,21 @@
> PR 44729
> patch: http://svn.apache.org/viewvc?view=rev&revision=778942
> +1: niq
> + -0.5: rpluem: There are further revisons in trunk that make this much more
> + complete and sane: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/mappers/mod_alias.c?r1=778942&r2=780699

Heh. OK, I'll take a look and either revise the proposal or
try to persuade you.

> * mod_rewrite: Remove locking for writing to the rewritelog.
> PR 46942
> trunk patch: http://svn.apache.org/viewvc?view=rev&revision=783734
> 2.2.x patch: http://people.apache.org/~niq/patches/46942.patch
> +1: niq
> + rpluem says:
> + http://people.apache.org/~niq/patches/46942.patch returns a 404

Oops, sorry, typo in uploading. Fixed.

(it's essentially identical to the trunk patch, but fails because the
immediate context is different).

--
Nick Kew

nick at webthing

Jul 4, 2009, 3:20 PM

Post #2 of 2 (185 views)
Permalink

Re: svn commit: r791178 - /httpd/httpd/branches/2.2.x/STATUS [In reply to]

Nick Kew wrote:
> rpluem[at]apache.org wrote:
>
>> @@ -157,17 +157,21 @@
>> PR 44729
>> patch: http://svn.apache.org/viewvc?view=rev&revision=778942
>> +1: niq
>> + -0.5: rpluem: There are further revisons in trunk that make this
>> much more
>> + complete and sane:
>> http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/mappers/mod_alias.c?r1=778942&r2=780699
>>
>
> Heh. OK, I'll take a look and either revise the proposal or
> try to persuade you.

OK, missed your URL when I wrote that. Reading the change,
I'm neutral on it. If you want to add it to the proposal,
my +1 remains valid.

--
Nick Kew

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact lists@gossamer-threads.com