Mailing List Archive: Apache: Dev

Re: svn commit: r831031 - in /httpd/httpd/branches/2.2.x: ./ CHANGES STATUS docs/manual/caching.xml modules/cache/mod_cache.c

Index | Next | Previous | View Threaded

trawick at gmail

Oct 30, 2009, 4:58 AM

Post #1 of 3 (98 views)
Permalink

Re: svn commit: r831031 - in /httpd/httpd/branches/2.2.x: ./ CHANGES STATUS docs/manual/caching.xml modules/cache/mod_cache.c

On Thu, Oct 29, 2009 at 1:06 PM, <poirier[at]apache.org> wrote:
> Author: poirier
> Date: Thu Oct 29 17:06:15 2009
> New Revision: 831031
>
> URL: http://svn.apache.org/viewvc?rev=831031&view=rev
> Log:
> Merge r823536, r823563 from trunk:
>
> mod_cache: add Cache-control: s-maxage to cacheability decisions per RFC 2616.
>
> Update doc to match change to mod_cache to include s-maxage in
> some cacheability decisions.
>
> Reviewed by: poirier

The value for "Reviewed by" should be the two or more people in this
list other than the author:

+1: poirier, sf, covener

poirier at pobox

Oct 30, 2009, 5:18 AM

Post #2 of 3 (96 views)
Permalink

Re: svn commit: r831031 - in /httpd/httpd/branches/2.2.x: ./ CHANGES STATUS docs/manual/caching.xml modules/cache/mod_cache.c [In reply to]

Jeff Trawick <trawick[at]gmail.com> writes:

> On Thu, Oct 29, 2009 at 1:06 PM, <poirier[at]apache.org> wrote:
>> Author: poirier
>> Date: Thu Oct 29 17:06:15 2009
>> New Revision: 831031
>>
>> URL: http://svn.apache.org/viewvc?rev=831031&view=rev
>> Log:
>> Merge r823536, r823563 from trunk:
...
>> Reviewed by: poirier
>
> The value for "Reviewed by" should be the two or more people in this
> list other than the author:
>
> +1: poirier, sf, covener

Makes sense; fixed.

I used http://people.apache.org/~jorton/svn.merge to do this backport
merge, which is where the "Reviewed by: poirier" came from. Is there an
updated version of that or another script for doing the routine work
of a backport?

Dan

rpluem at apache

Oct 30, 2009, 2:01 PM

Post #3 of 3 (91 views)
Permalink

Re: svn commit: r831031 - in /httpd/httpd/branches/2.2.x: ./ CHANGES STATUS docs/manual/caching.xml modules/cache/mod_cache.c [In reply to]

On 10/30/2009 01:18 PM, Dan Poirier wrote:
> Jeff Trawick <trawick[at]gmail.com> writes:
>
>> On Thu, Oct 29, 2009 at 1:06 PM, <poirier[at]apache.org> wrote:
>>> Author: poirier
>>> Date: Thu Oct 29 17:06:15 2009
>>> New Revision: 831031
>>>
>>> URL: http://svn.apache.org/viewvc?rev=831031&view=rev
>>> Log:
>>> Merge r823536, r823563 from trunk:
> ...
>>> Reviewed by: poirier
>> The value for "Reviewed by" should be the two or more people in this
>> list other than the author:
>>
>> +1: poirier, sf, covener
>
> Makes sense; fixed.
>
> I used http://people.apache.org/~jorton/svn.merge to do this backport
> merge, which is where the "Reviewed by: poirier" came from. Is there an
> updated version of that or another script for doing the routine work
> of a backport?

Not that I am aware of, but Joe's script is really good. Usually there
are two things of manual work left:

1. Fix the CHANGES entry.
2. Fix the 'Reviewed by' in the clog file.

The script takes care of the remaining parts.

Regards

Rüdiger

Index | Next | Previous | View Threaded

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