>>>>> Daniel Baumann <daniel.baum...@progress-technologies.net> writes:
> i've fixed both in git, in a somewhat less complicated way though,
> thanks.
Even if wc(1) is for some reason deemed necessary, ls(1) is
useless in this case. Roughly speaking, the difference between
$ ls foo* and $ printf %s\\n foo* is the same as between
$ cat foo | grep bar and $ grep bar foo. (To note is that
printf is built into both Bash and Dash.)
As for the second change, the use of the ${VAR#PATTERN}
construct: is POSIX [1] (and implemented by Bash, Dash, and
Busybox' Ash), saves two forks (basename(1), sed(1)), /and/
makes the code more concise. So, I'd rather recommend doing it
that way.
However, contrary to my previous patch, it's the ${VAR##PATTERN}
(“longest match”) form that should be used.
TIA.
[1]
http://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#tag_18_06_02
--
FSF associate member #7257
--
To UNSUBSCRIBE, email to debian-live-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/86wqxn9q72....@gray.siamics.net
