Bug#659084: devscripts: [PATCH] debsign - use POSIX command substitution $()

2012-02-26 Thread jaalto
On 2012-02-26 09:32, Adam D. Barratt wrote: | 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

Bug#659084: devscripts: [PATCH] debsign - use POSIX command substitution $()

2012-02-26 Thread Adam D. Barratt
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"' f

Bug#659084: devscripts: [PATCH] debsign - use POSIX command substitution $()

2012-02-26 Thread jaalto
On 2012-02-26 09:20, Adam D. Barratt wrote: | On Sun, 2012-02-26 at 03:30 +0100, Jakub Wilk wrote: | > * Jari Aalto , 2012-02-08, 02:58: | > >-[ "`head -n 1 \"$1\"`" = "-BEGIN PGP SIGNED MESSAGE-" ] || \ | > >+[ "$(head -n 1 \"$1\")" = "-BEGIN PGP SIGNED MESSAGE-" ] || \ | >

Bug#659084: devscripts: [PATCH] debsign - use POSIX command substitution $()

2012-02-26 Thread Adam D. Barratt
On Sun, 2012-02-26 at 03:30 +0100, Jakub Wilk wrote: > * Jari Aalto , 2012-02-08, 02:58: > >-[ "`head -n 1 \"$1\"`" = "-BEGIN PGP SIGNED MESSAGE-" ] || \ > >+[ "$(head -n 1 \"$1\")" = "-BEGIN PGP SIGNED MESSAGE-" ] || \ > > Bzzt, wrong. > > $ foo=/etc/ld.so.conf; echo "`he

Bug#659084: devscripts: [PATCH] debsign - use POSIX command substitution $()

2012-02-25 Thread Jakub Wilk
* Jari Aalto , 2012-02-08, 02:58: -[ "`head -n 1 \"$1\"`" = "-BEGIN PGP SIGNED MESSAGE-" ] || \ +[ "$(head -n 1 \"$1\")" = "-BEGIN PGP SIGNED MESSAGE-" ] || \ Bzzt, wrong. $ foo=/etc/ld.so.conf; echo "`head -n 1 \"$foo\"`" include /etc/ld.so.conf.d/*.conf $ foo=/etc/ld

Bug#659084: devscripts: [PATCH] debsign - use POSIX command substitution $()

2012-02-09 Thread Benjamin Drung
Am Donnerstag, den 09.02.2012, 07:52 +0200 schrieb jaalto: > On 2012-02-08 18:28, James McCoy wrote: > | > | > Motivation: > | > | > > | > | > - Recommended practise. The $() is recommend by many[2]. It nests > easily. > | > | > - Readability. In high resolution display a "tick" is hard to see. >

Bug#659084: devscripts: [PATCH] debsign - use POSIX command substitution $()

2012-02-08 Thread jaalto
On 2012-02-08 18:28, James McCoy wrote: | > | > Motivation: | > | > | > | > - Recommended practise. The $() is recommend by many[2]. It nests easily. | > | > - Readability. In high resolution display a "tick" is hard to see. | > | > Different font may also make reading the "tick" difficult to

Bug#659084: devscripts: [PATCH] debsign - use POSIX command substitution $()

2012-02-08 Thread James McCoy
On Wed, Feb 08, 2012 at 11:56:01PM +0200, jaalto wrote: > On 2012-02-08 21:45, Adam D. Barratt wrote: > | On Wed, 2012-02-08 at 02:58 -0500, Jari Aalto wrote: > | > The following patch updates to use POSIX[1] command substitution $() in > | > place of backtics (``). > | > > | > Motivation: > | >

Bug#659084: devscripts: [PATCH] debsign - use POSIX command substitution $()

2012-02-08 Thread jaalto
On 2012-02-08 21:45, Adam D. Barratt wrote: | On Wed, 2012-02-08 at 02:58 -0500, Jari Aalto wrote: | > The following patch updates to use POSIX[1] command substitution $() in | > place of backtics (``). | > | > Motivation: | > | > - Recommended practise. The $() is recommend by many[2]. It nests

Bug#659084: devscripts: [PATCH] debsign - use POSIX command substitution $()

2012-02-08 Thread Adam D. Barratt
On Wed, 2012-02-08 at 02:58 -0500, Jari Aalto wrote: > The following patch updates to use POSIX[1] command substitution $() in > place of backtics (``). > > Motivation: > > - Recommended practise. The $() is recommend by many[2]. It nests easily. > - Readability. In high resolution display a "tic

Bug#659084: devscripts: [PATCH] debsign - use POSIX command substitution $()

2012-02-08 Thread Jari Aalto
Package: devscripts Version: 2.11.3 Severity: wishlist The following patch updates to use POSIX[1] command substitution $() in place of backtics (``). Motivation: - Recommended practise. The $() is recommend by many[2]. It nests easily. - Readability. In high resolution display a "tick" is hard