Package: lvm2
Severity: important
Version: 2.02.51-4
User: debian-release@lists.debian.org
Usertags: goal-dash

Hello maintainer,

While performing an archive wide checkbashisms (from the 'devscripts' package)
check I've found your package containing a /bin/sh script making use
of a bashism.

checkbashisms' output:
> possible bashism in ./sbin/fsadm line 62 ($RANDOM):
> TEMPDIR="${TMPDIR:-/tmp}/${TOOL}_${RANDOM}$$/m"
> possible bashism in ./sbin/vgimportclone line 40 (should be >word 2>&1):
> "$LVM" version >& /dev/null || die 2 "Could not run lvm binary '$LVM'"
> possible bashism in ./sbin/vgimportclone line 43 ('function' is useless):
> function getvgname {
> possible bashism in ./sbin/vgimportclone line 55 (alternative test command
> ([[ foo ]] should be [ foo ])):
>     while [[ "${VGLIST}" =~ "${NAME}" ]]
> possible bashism in ./sbin/vgimportclone line 64 ('function' is useless):
> function checkvalue {
> possible bashism in ./sbin/vgimportclone line 73 ('function' is useless):
> function usage {
> possible bashism in ./sbin/vgimportclone line 89 ('function' is useless):
> function cleanup {
> possible bashism in ./sbin/vgimportclone line 103 ($UID should be "$(id
> -ru)"):
> if [ "$UID" != "0" -a "$EUID" != "0" ]
> possible bashism in ./sbin/vgimportclone line 103 ($EUID should be "$(id
> -u)"):
> if [ "$UID" != "0" -a "$EUID" != "0" ]
> possible bashism in ./sbin/vgimportclone line 154 (let ...):
>	      let VERBOSE_COUNT=VERBOSE_COUNT+1
> possible bashism in ./sbin/vgimportclone line 320 (alternative test command
> ([[ foo ]] should be [ foo ])):
>     if [[ "${PVLIST}" =~ "unknown" ]]


Not using bash (or a Debian Policy conformant shell interpreter which doesn't
provide such an extra feature) as /bin/sh is likely to lead to errors or
unexpected behaviours. Please be aware that dash is now the default /bin/sh.

Hints about how to fix bashisms can be found at:
https://wiki.ubuntu.com/DashAsBinSh

If you still don't know how to fix the bashisms don't hesitate to reply to
this email, or tag the bug as 'help'.

Thank you,
Raphael Geissert