Jeff King <peff@xxxxxxxx> writes: > On Mon, Sep 19, 2011 at 10:57:37AM -0700, Junio C Hamano wrote: > >> Jeff King <peff@xxxxxxxx> writes: >> >> > Hmm. Actually, the one for custom browser commands might need it, >> > because that one is expected to be a shell snippet. I suspect the >> > simplest thing is to do something like: >> > >> > eval "$browser_cmd \"\$@\"" >> >> Yeah, I agree, and the dq around $browser_cmd is kind of important, too, >> for that to work and be readable. > > Oops, good catch. Probably the most readable version would be: > > eval "\"$browser_cmd\"" '"$@"' This make the use of eval even more questionable. If $browser_cmd is supposed to be whitespace splitted then this won't do it (unless it contains embedded double quotes). Andreas. -- Andreas Schwab, schwab@xxxxxxxxxxxxxx GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 "And now for something completely different." -- 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