Re: [PATCH 13/13] move LESS/LV pager environment to Makefile
- From
Jeff King <peff@peff.net>
- Date
- Feb 5, 2014, 19:52 UTC
- Message-ID
- <20140205195206.GA16753@sigill.intra.peff.net>
- In-Reply-To
- <20140205192350.GA16009@sigill.intra.peff.net>
On Wed, Feb 05, 2014 at 02:23:50PM -0500, Jeff King wrote:
Show 10 quoted lines
> On Wed, Feb 05, 2014 at 01:08:57PM -0500, Jeff King wrote:
>
> > +quote() {
> > + echo "$1" | sed 's/\\/\\\\/g; s/"/\\"/'
> > +}
>
> This of course has the same "/g" bug as the earlier patch in the series.
>
> I was tempted to pull the function out into script/lib-c.sh, so that
> both can source it, but perhaps that is overkill.Also, this (and the other spot) should probably be using "printf" instead of echo for portability (i.e., the config variables might have backslashes in them).
-Peff