Eric Sunshine <sunshine@xxxxxxxxxxxxxx> writes: > On Thu, Feb 13, 2020 at 5:58 PM Emily Shaffer <emilyshaffer@xxxxxxxxxx> wrote: >> On Thu, Feb 06, 2020 at 08:35:33AM +0700, Danh Doan wrote: >> > - while read line >> > - do >> > - echo " \"$line\"," >> > - done >> > + sed 's/^/ "/; s/$/",/' >> >> 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.
