Debian Bug report logs - #530981
ldm: bashism in /bin/sh script

version graph

Package: ldm; Maintainer for ldm is Debian LTSP Maintainers <team+ltsp@tracker.debian.org>; Source for ldm is src:ldm (PTS, buildd, popcon).

Reported by: Raphael Geissert <atomo64@gmail.com>

Date: Fri, 29 May 2009 04:33:17 UTC

Severity: minor

Found in version ldm/2:2.0.40-1

Fixed in version ldm/2:2.0.40-2

Done: Vagrant Cascadian <vagrant@freegeek.org>

Bug is archived. No further changes may be made.

Toggle useless messages

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to LTSP Debian/Ubuntu Maintainers <pkg-ltsp-devel@lists.alioth.debian.org>:
Bug#530981; Package ldm. (Fri, 29 May 2009 04:33:19 GMT) (full text, mbox, link).


Message #3 received at maintonly@bugs.debian.org (full text, mbox, reply):

From: Raphael Geissert <atomo64@gmail.com>
To: maintonly@bugs.debian.org
Subject: ldm: bashism in /bin/sh script
Date: Thu, 28 May 2009 23:28:13 -0500
Package: ldm
Severity: minor
Version: 2:2.0.40-1
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 ./usr/share/ldm/ldm-script line 37 ($"foo" should be
> eval_gettext "foo"):
>	  if [ $"test" = "\$test" ]; then
> possible bashism in ./usr/share/ldm/ldm-script line 40 ($"foo" should be
> eval_gettext "foo"):
>	      echo $"$*"


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 although bash is currently the default /bin/sh there was
a release goal for Lenny to make dash the default /bin/sh[1], and has been
proposed for squeeze as well[2].

If you want more information about dash as /bin/sh, you can read:
http://lists.debian.org/debian-release/2008/01/msg00189.html

For more information supporting this goal please refer to Debian Policy,
section 10.4, at:
http://www.debian.org/doc/debian-policy/ch-files.html#s-scripts

Hints about how to fix bashisms:
Sometimes these bugs are already fixed in Ubuntu, look at the PTS.
If not already fixed you can read:
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'.

[1]http://release.debian.org/lenny/goals.txt
[2]http://lists.debian.org/debian-release/2009/04/msg00133.html

Thank you,
Raphael Geissert




Reply sent to Vagrant Cascadian <vagrant@freegeek.org>:
You have taken responsibility. (Fri, 29 May 2009 07:57:17 GMT) (full text, mbox, link).


Notification sent to Raphael Geissert <atomo64@gmail.com>:
Bug acknowledged by developer. (Fri, 29 May 2009 07:57:17 GMT) (full text, mbox, link).


Message #8 received at 530981-close@bugs.debian.org (full text, mbox, reply):

From: Vagrant Cascadian <vagrant@freegeek.org>
To: 530981-close@bugs.debian.org
Subject: Bug#530981: fixed in ldm 2:2.0.40-2
Date: Fri, 29 May 2009 07:17:05 +0000
Source: ldm
Source-Version: 2:2.0.40-2

We believe that the bug you reported is fixed in the latest version of
ldm, which is due to be installed in the Debian FTP archive:

ldm-server_2.0.40-2_all.deb
  to pool/main/l/ldm/ldm-server_2.0.40-2_all.deb
ldm_2.0.40-2.diff.gz
  to pool/main/l/ldm/ldm_2.0.40-2.diff.gz
ldm_2.0.40-2.dsc
  to pool/main/l/ldm/ldm_2.0.40-2.dsc
ldm_2.0.40-2_i386.deb
  to pool/main/l/ldm/ldm_2.0.40-2_i386.deb



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 530981@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Vagrant Cascadian <vagrant@freegeek.org> (supplier of updated ldm package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Thu, 28 May 2009 23:52:30 -0700
Source: ldm
Binary: ldm ldm-server
Architecture: source all i386
Version: 2:2.0.40-2
Distribution: unstable
Urgency: low
Maintainer: LTSP Debian/Ubuntu Maintainers <pkg-ltsp-devel@lists.alioth.debian.org>
Changed-By: Vagrant Cascadian <vagrant@freegeek.org>
Description: 
 ldm        - LTSP display manager
 ldm-server - LTSP display manager
Closes: 530981
Changes: 
 ldm (2:2.0.40-2) unstable; urgency=low
 .
   * apply patch to remove buggy eval_gettext function. instead, always use
     eval_gettext function from gettext.sh. depend on gettext-base to ensure
     gettext.sh is present.  (Closes: #530981)
 .
   * remove largely un-used code to generate debian/control.
Checksums-Sha1: 
 4e5f74a0afb0250edb3d820531c353628109495a 1497 ldm_2.0.40-2.dsc
 9e0cad10f305c1a6ec595a29cedc04d00dfad9a7 10214 ldm_2.0.40-2.diff.gz
 94e066473f542cfd1cf6b67533a744a1bd24bc4f 11382 ldm-server_2.0.40-2_all.deb
 f6c47c70cb81ddb6528483caf66670087d84a4dc 73076 ldm_2.0.40-2_i386.deb
Checksums-Sha256: 
 df5df363f68e3ef210a2e587131f7a3bfb6c57a79f582871dccbad5b7742d8af 1497 ldm_2.0.40-2.dsc
 50aaa38dbce718ad9c895cf763c6b3c9a9ba3d56656577bb31f6f95e90b279b8 10214 ldm_2.0.40-2.diff.gz
 adf796f0557e2e405962b3c822d3e798fb9f24297cd9c90bbdf710621401cd5b 11382 ldm-server_2.0.40-2_all.deb
 17822f37e19c6cd56d1ec6e410750250d3f7fb9451b318a26d6819773c9fb54c 73076 ldm_2.0.40-2_i386.deb
Files: 
 7fe2752affdfe7bb1943f00e59738a00 1497 misc extra ldm_2.0.40-2.dsc
 1208a37c4407f52fa1e39b67c942ffa5 10214 misc extra ldm_2.0.40-2.diff.gz
 635c106c6890607a5263a00e9ada3f07 11382 misc extra ldm-server_2.0.40-2_all.deb
 84562d9d18d68d9c0b0a17229ba992bc 73076 misc extra ldm_2.0.40-2_i386.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)

iEYEARECAAYFAkofiSAACgkQlPc63BPWGpn8PwCZAVrU+Ih2w7NE9CXB0yEYcPwp
tu8AnRkWaaVzZPxelzuZXrffJIbyJRsD
=rawu
-----END PGP SIGNATURE-----





Bug archived. Request was from Debbugs Internal Request <owner@bugs.debian.org> to internal_control@bugs.debian.org. (Mon, 06 Jul 2009 07:31:49 GMT) (full text, mbox, link).


Send a report that this bug log contains spam.


Debian bug tracking system administrator <owner@bugs.debian.org>. Last modified: Sat Jul 1 13:23:44 2023; Machine Name: buxtehude

Debian Bug tracking system

Debbugs is free software and licensed under the terms of the GNU Public License version 2. The current version can be obtained from https://bugs.debian.org/debbugs-source/.

Copyright © 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson, 2005-2017 Don Armstrong, and many other contributors.