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\"" '"$@"' -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