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.
