Re: Re[2]: [PATCH] perform keywords substitution in mod_dav_svn

2013-04-10 Thread Julian Foad
jinfroster wrote: > CMP> Finished. > > CMP>    Sending        subversion/mod_dav_svn/dav_svn.h > CMP>    Sending        subversion/mod_dav_svn/repos.c > CMP>    Transmitting file data .. > CMP>    Committed revision 1466055. > > Great! Thanks a lot, Michael! I'm just curious... Do you have any

Re[2]: [PATCH] perform keywords substitution in mod_dav_svn

2013-04-09 Thread jinfroster
CMP> Finished. CMP>Sendingsubversion/mod_dav_svn/dav_svn.h CMP>Sendingsubversion/mod_dav_svn/repos.c CMP>Transmitting file data .. CMP>Committed revision 1466055. Great! Thanks a lot, Michael! -- Best regards, jinfrostermailto:jinfr

Re: [PATCH] perform keywords substitution in mod_dav_svn

2013-04-09 Thread C. Michael Pilato
On 04/09/2013 09:46 AM, C. Michael Pilato wrote: > On 04/07/2013 01:38 PM, jinfroster wrote: >> + keyword_subst = apr_table_get(pairs, "kw"); >> + if (keyword_subst && *keyword_subst != '0') >> +comb->priv.keyword_subst = TRUE; >> + >>return NULL; >> } > > I'm with Daniel -- this should

Re: [PATCH] perform keywords substitution in mod_dav_svn

2013-04-09 Thread C. Michael Pilato
On 04/07/2013 01:38 PM, jinfroster wrote: > Index: subversion/mod_dav_svn/dav_svn.h > === > --- subversion/mod_dav_svn/dav_svn.h (revision 1465320) > +++ subversion/mod_dav_svn/dav_svn.h (working copy) > @@ -289,6 +289,9 @@ > >

Re: [PATCH] perform keywords substitution in mod_dav_svn

2013-04-07 Thread Daniel Shahaf
jinfroster wrote on Sun, Apr 07, 2013 at 21:38:07 +0400: > Hello, dev! > Here is my second attempt: > > [[[ > Accept "kw" parameter in query string. Treat any value except "0" as a > request to perform server-side keywords substitution in file contents. I would suggest requiring "1" as the value,

[PATCH] perform keywords substitution in mod_dav_svn

2013-04-07 Thread jinfroster
Hello, dev! Here is my second attempt: [[[ Accept "kw" parameter in query string. Treat any value except "0" as a request to perform server-side keywords substitution in file contents. * subversion/mod_dav_svn/dav_svn.h (struct dav_resource_private): add keyword_subst flag. * subversion/mod_da