Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/etc



On Tue, 05 Jun 2012, Aleksej Saushev wrote:
                if [ -n "$count_nodes" ]; then
                        count_nodes=$((count_nodes + \
-                           $(linecount "$(sh "$0.local" $opts -s all)") ))
+                           $(linecount "$("$HOST_SH" "$0.local" $opts -s 
all)") ))

Are \\s really not needed here?

$() and $(()) constructs nest correctly without needing any backslashes.

$opts will be a problem if it contains "-m PATH_TO_MKNOD" and
PATH_TO_MKNOD contains spaces or other special characters,
but backslashes here won't fix that.

--apb (Alan Barrett)


Home | Main Index | Thread Index | Old Index