Bug#530126: live-helper: bashism in /bin/sh script

[Daniel Baumann <daniel@debian.org>]

Raphael Geissert wrote:
> Processing Deb/mirror/pool/main/l/live-helper/live-helper_1.0.5-1_all.deb...
> possible bashism in ./etc/cron.daily/live-helper line 38 ($HOST(TYPE|NAME)):
> echo "$(date +%b\ %d\ %H:%M:%S) ${HOSTNAME} live-helper: begin daily build." 
>>> /var/log/live
> possible bashism in ./etc/cron.daily/live-helper line 154 ($HOST(TYPE|NAME)):
> echo "$(date +%b\ %d\ %H:%M:%S) ${HOSTNAME} live-helper: end daily build." 
>>> /var/log/live
> done

that one is fixed in git now.

>> (only a false positive about $HOSTNAME in debian/live-initramfs.init).
> 
> Are you sure? for -initramfs (1.157.2-1) what I get is:
> possible bashism in ./etc/init.d/live-initramfs line 164 (echo -e):
>                 echo -n -e "\n\n${MESSAGE} and press ENTER to continue:" 
>> /dev/console

this has been fixed in git weeks ago.

> You can use the following script to check a package for bashisms (it is very 
> similar to the one I use on my archive-wide checks), it takes one or more 
> file names as parameters: 
> http://git.debian.org/?p=users/atomo64-guest/misc-devscripts.git;a=blob;f=checkbashisms-in-package.sh;hb=HEAD

we use checkbashism for more than a year in the test target of the
makefiles, but thanks.

-- 
Address:        Daniel Baumann, Burgunderstrasse 3, CH-4562 Biberist
Email:          daniel.baumann@panthera-systems.net
Internet:       http://people.panthera-systems.net/~daniel-baumann/