On Wed, Jul 27, 2022 at 11:06:39AM +0200, Alexander Hall wrote: > I think replacing '*' with '*([![:cntrl:]])' can be the droid your looking > for.
As I was falling asleep last night I was trying to figure out how to get '*([!$_nl])' into the match and couldn't think of a good one. Your solution is much better. I think that would end up looking like: ^cpu0:*([![:cntrl:]])Intel(R) although I recall some weirdness with metachars in a variable so I'll have to test it out. On Wed, Jul 27, 2022 at 12:01:51PM +0000, Klemens Nanni wrote: > On Wed, Jul 27, 2022 at 01:55:58PM +0200, Alexander Hall wrote: > > On a sidenote, the > > > > _nl=$(echo) > > > > in fw_update.sh is AFAICT deceitful and moot, since ksh strips away > > trailing linefeeds. Use the uglier > > > > _nl=' > > ' > > > > instead. > > Correct: > > $ _nl=$(echo) > $ printf '%s' "$_nl" > $ _nl=' > > ' > $ printf '%s' "$_nl" > > $ printf '\n' > > $ This is correct, I swear I had tested it before and that it worked, but I can't reproduce it now. The switch to per-line was using the ugly version. I'll work up a patch to get that fixed anyway.
