On Fri, Mar 04, 2016 at 11:30:36AM +0100, Christian Couder wrote: > > Those are just guesses, but if we are tickling a bug in perl's parser, > > this might avoid them. I also wondered when "/r" appeared. It was in > > 5.14, so you're presumably good there. > > If I just remove the "r" at the end of "s/\\/\//gr", I get with both > Perl versions: > > Can't modify non-lvalue subroutine call at t/t9700/test.pl line 36.
Right, because the string being operated on is the return value of a function, so we can't do substitution on it (unless with "r", whose purpose is to allow exactly such a thing). > > The "use" statement at the top of > > the script says "5.008", so perhaps we should be writing it out longhand > > anyway (that version is "only" 5 years old, so I suspect there are still > > systems around with 5.12 or older). > > Yeah, it would work. Thanks for confirming the longhand does work; I think the patch I just posted elsewhere in the thread should be good for you, then. -Peff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
