On Sun, Jan 18, 2009 at 10:41:13AM +0100, Stephan Beyer wrote: > > Looks sane except that I do not think you need printf nor the leading > > blank line, i.e. > > > > echo "Patch failed at $msgnum ($FIRSTLINE)" > > Hmm, IIRC if $FIRSTLINE contains \n or something like that, it will > interpret this as newline in some shell/echo implementations. > > So printf "...%s..." "$FOO" is always sane for user input. Yes, I'm surprised Junio doesn't remember the mass conversions we already had to do (4b7cc26a and 293623ed). But looking at the date, I guess it _has_ been a year and a half. :) -Peff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
