Thread (222 messages) 222 messages, 11 authors, 2020-04-06

Re: [PATCH v6 01/15] help: move list_config_help to builtin/help

From: Eric Sunshine <hidden>
Date: 2020-02-13 23:08:13

On Thu, Feb 13, 2020 at 5:58 PM Emily Shaffer [off-list ref] wrote:
On Thu, Feb 06, 2020 at 08:35:33AM +0700, Danh Doan wrote:
quoted
-     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",/'
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help