[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

RE: [PATCH] Perl bindings corrupt "{DATE}" revision variable

From: Bert Huijben <bert_at_qqmail.nl>
Date: Thu, 3 Nov 2016 10:27:30 +0100

I moved your nominations to approved with another +0.

With our Subversion binding rules one +1 of a committer for that region (yours) and one +0 from another is enough to approve changes for the binding code.

 

I’m not sure about the perl rules applied around this function, but perhaps we should just copy the string instead of changing memory that we receive as constant. We might also have a different api function that behaves in a cleaner way.

(The current code is better than it used to be, so my +0 stands anyway)

 

                Bert

 

From: Roderich Schupp [mailto:roderich.schupp_at_gmail.com]
Sent: woensdag 2 november 2016 21:07
To: Daniel Shahaf <danielsh_at_apache.org>
Cc: raimue_at_codingfarm.de; Subversion devel mailing list <dev_at_subversion.apache.org>
Subject: Re: [PATCH] Perl bindings corrupt "{DATE}" revision variable

 

On Wed, Nov 2, 2016 at 8:41 PM, Daniel Shahaf <danielsh_at_apache.org <mailto:danielsh_at_apache.org> > wrote:

Thanks, Roderich. Could you update the log message with an appropriate
"Verbed by"[1] clause, for contribulyzer?

 

Done.
 

I assume this should be backported, in which case, here is my +0 for
backport of the patch as committed.

Nominated for 1.8 and 1.9 incl. your vote.

Cheers, Roderich
Received on 2016-11-03 10:27:43 CET

This is an archived mail posted to the Subversion Dev mailing list.