develooper Front page | perl.perl5.porters | Postings from August 2009

Re: [PATCH] backport of: fix setuid execution that was broken byf20b29985d

Thread Previous | Thread Next
Rainer Tammer
August 26, 2009 12:07
Re: [PATCH] backport of: fix setuid execution that was broken byf20b29985d
Message ID:

demerphq wrote:
> 2009/8/26 Nicholas Clark <>:
>> On Wed, Aug 26, 2009 at 03:29:58PM +0200, Rainer Tammer wrote:
>>> Hello,
>>> please could someone with commit right apply the attached patch to
>>> maint-5.8 ?
>>> If there will ever be a 5.8.10 then the suidperl problem would be fixed...
>> Whilst agreeing with the intent, I disagree with the mechanism.
>> Someone should cherrypick the correct patch from maint-5.10.
>> They shouldn't cold apply patches to a branch, as git then fails to track
>> what is not yet merged.
> Er, I dont get it. Cherry picking /is/ applying patches. If the exact
> same patch is applied by hand to two branches git will not know
> whether it has been done by rebasing, cherry-picking or by hand. It is
> all the same to git (message details aside, which git does not look
> at.)
> Also, IMO, and I say this only because 5.10.1 is out now, I think
> patches for 5.8 should go to 5.8 first and THEN be applied to blead if
> necessary. In fact, its arguable we should just start collecting topic
> branches. And then rebase them into the other branches as needed. Then
> there is no need for the "which patches have been back ported, etc"
> type tracking. Either the topic branch has been merged/rebased into
> the branch, or it has not.
> Yves
This patch (same content) was applied to maint-5.10.
This can not be applied to bleed as the setuid perl functionallity was 
removed from bleed (5.11 and up).
So this is already fixed for maint-5.10 but not jet on maint-5.8.

This is the patch for in 5.10:

So it should be no problem to pick this one.


Thread Previous | Thread Next Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at | Group listing | About