On Sun, 2012-02-26 at 11:28 +0200, jaalto wrote: > On 2012-02-26 09:20, Adam D. Barratt wrote: > | fwiw, it breaks even if you remember to use the right variable in the > | second example... ;-) > | > | $ foo=/etc/ld.so.conf; echo "$(head -n 1 \"$foo\")" > | head: cannot open `"/etc/ld.so.conf"' for reading: No such file or directory > > One of the benfits of POSIX command substitution is also, that there > is no need for extra quoting:
You're missing James's point, though. You provided a patch which broke the script and therefore obviously hadn't been tested. That also demonstrated that the change isn't as trivial and simply aesthetic as claimed. Regards, Adam -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org