On Thu, Feb 13, 2020 at 06:29:39PM -0500, Eric Sunshine wrote: > On Thu, Feb 13, 2020 at 6:24 PM Junio C Hamano <gitster@xxxxxxxxx> wrote: > > Eric Sunshine <sunshine@xxxxxxxxxxxxxx> writes: > > > On Thu, Feb 13, 2020 at 5:58 PM Emily Shaffer <emilyshaffer@xxxxxxxxxx> wrote: > > >> Thanks for the suggestion. You're right that I didn't look into the > > >> contents of this script much, because I was just moving it; I modified > > >> your sed expression slightly so it performs just one operation: > > >> > > >> sed 's/^.*$/ "\0",/' > > > > > > This isn't portable and won't work with 'sed' from BSD lineage, > > > including Mac OS. More portable would be: > > > > > > sed 's/^\(.*\)$/ "\1",/' > > > > Or just replace Emily's \0 with &; that should be in POSIX. > > Yes, that's nicer. Sure, thanks.
