[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#476760: marked as done (mkrboot: bashism in /bin/sh script)



Your message dated Tue, 01 Jul 2008 19:02:19 +0000
with message-id <E1KDl7L-0000TT-M3@ries.debian.org>
and subject line Bug#476760: fixed in mkrboot 0.96
has caused the Debian Bug report #476760,
regarding mkrboot: bashism in /bin/sh script
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
476760: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=476760
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: mkrboot
Severity: important
Version: 0.95
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/sbin/mkrboot line 96 (brace expansion):
>         zip -j $FILE /tmp/$$/{linux,root.bin,install.bat,loadlin.exe}
> >/dev/null

Not using bash (or a Debian Policy conformant shell interpreter which does 
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's a 
release goal for Lenny to make dash the default /bin/sh[1].

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 please refer to the Debian Policy section 10.4 supporting 
this argument 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

[1]http://release.debian.org/lenny/goals.txt

Thank you,
-- 
Atomo64 - Raphael

Please avoid sending me Word, PowerPoint or Excel attachments.
See http://www.gnu.org/philosophy/no-word-attachments.html



--- End Message ---
--- Begin Message ---
Source: mkrboot
Source-Version: 0.96

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

mkrboot_0.96.dsc
  to pool/main/m/mkrboot/mkrboot_0.96.dsc
mkrboot_0.96.tar.gz
  to pool/main/m/mkrboot/mkrboot_0.96.tar.gz
mkrboot_0.96_amd64.deb
  to pool/main/m/mkrboot/mkrboot_0.96_amd64.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 476760@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Raphael Geissert <atomo64@gmail.com> (supplier of updated mkrboot 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: Sat, 28 Jun 2008 17:23:50 -0500
Source: mkrboot
Binary: mkrboot
Architecture: source amd64
Version: 0.96
Distribution: unstable
Urgency: low
Maintainer: Debian QA Group <packages@qa.debian.org>
Changed-By: Raphael Geissert <atomo64@gmail.com>
Description: 
 mkrboot    - Make a kernel + root image bootable from one disk or from DOS
Closes: 476760
Changes: 
 mkrboot (0.96) unstable; urgency=low
 .
   * QA upload.
   * Fixed bashism in mkrboot (Closes: #476760)
     - Also made temp dir not easy to guess.
     + Thanks to Chris Lamb <chris@chris-lamb.co.uk> for the patch.
   * Moved README.Debian to debian/ for consistency
   * debian/rules completely rewritten using debhelper
   * mkrboot.8: small tweaks so dh_installman detects the section
Checksums-Sha1: 
 b463ef75ca2080f3f69effb2a9622ed98584882f 702 mkrboot_0.96.dsc
 61c584630f3bb5635b058c2ccc099a4d3b6c0f72 12720 mkrboot_0.96.tar.gz
 62a1c2801800cd5974793718f790a0b06a7d813f 14900 mkrboot_0.96_amd64.deb
Checksums-Sha256: 
 74133d20b21cf9a9b2fa11af7deef818f3c4f1f53a44730d6e7d919d93bb421c 702 mkrboot_0.96.dsc
 e2638c5a62200a29d002fb157e59fd5e1dd1860a03fe50646d50d680905bfb32 12720 mkrboot_0.96.tar.gz
 e74476c9ee83d77dab807ef621074c4449f9b33f2b5336ea03a64b6d6949a838 14900 mkrboot_0.96_amd64.deb
Files: 
 9c1d585f9dc4c035af4cc5aae3cbd5a6 702 admin optional mkrboot_0.96.dsc
 2c0d88302372a40caa84e58929992d4a 12720 admin optional mkrboot_0.96.tar.gz
 6428f747050be953858c329e282c0386 14900 admin optional mkrboot_0.96_amd64.deb

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

iEYEARECAAYFAkhqfM8ACgkQKFvXofIqeU5kbQCglY+iTAxeEfKBWA0JAbhKjBrE
rnkAn2ZXN/x3dqrXpPc4MD1KjhvKoJ+T
=Evi1
-----END PGP SIGNATURE-----



--- End Message ---

Reply to: