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

Bug#489562: marked as done (debian-cd: bashism in /bin/sh script)



Your message dated Sun, 26 Oct 2008 23:47:03 +0000
with message-id <E1KuFK3-0006ge-DV@ries.debian.org>
and subject line Bug#489562: fixed in debian-cd 3.0.5
has caused the Debian Bug report #489562,
regarding debian-cd: 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.)


-- 
489562: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=489562
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: debian-cd
Severity: important
Version: 3.0.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 one or more /bin/sh scripts making 
use of bashisms.

checkbashisms' output:
> possible bashism in ./usr/share/debian-cd/tools/grab_md5 line 25 (echo -e):
>                         echo -e "Using MD5 sums from Sources files:\n$FILES" 
> possible bashism in ./usr/share/debian-cd/tools/grab_md5 line 53 (echo -e):
> echo -e "Using MD5 sums from Packages files:\n$FILES" 

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: debian-cd
Source-Version: 3.0.5

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

debian-cd_3.0.5.dsc
  to pool/main/d/debian-cd/debian-cd_3.0.5.dsc
debian-cd_3.0.5.tar.gz
  to pool/main/d/debian-cd/debian-cd_3.0.5.tar.gz
debian-cd_3.0.5_all.deb
  to pool/main/d/debian-cd/debian-cd_3.0.5_all.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 489562@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Steve McIntyre <93sam@debian.org> (supplier of updated debian-cd 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: Sun, 26 Oct 2008 16:23:04 -0700
Source: debian-cd
Binary: debian-cd
Architecture: source all
Version: 3.0.5
Distribution: unstable
Urgency: low
Maintainer: Debian CD Group <debian-cd@lists.debian.org>
Changed-By: Steve McIntyre <93sam@debian.org>
Description: 
 debian-cd  - Tools for building (Official) Debian CD set
Closes: 416230 470838 486360 489486 489562 491300 497270 497966
Changes: 
 debian-cd (3.0.5) unstable; urgency=low
 .
   [ Steve McIntyre ]
   * Pre-cache apt's package information in large lumps rather than walk
     through it one package at a time. It's much faster. Closes: #470838
     Needs a small re-arrangement: code from tools/add_packages now
     moves into tools/make_disc_trees.pl
   * Update lenny's isolinux.bin to the one from syslinux 3.63+dfsg-1.
     Should help a user with an old Adaptec SCSI card who has been reporting
     problems.
   * tools/imagesums: use newer features in latest mkisofs/genisoimage to grab
     pre-calculated MD5 and SHA1 checksums directly from the jigdo file.
 .
   [ Frans Pop ]
   * Remove wireless-tools-udeb from the lenny udeb_include files as it is not
     required for wireless support (see also #474708).
 .
   [ Joey Hess ]
   * Add support for a split syslinux.cfg file, which will later be provided
     to support the syslinux vesa menu.
 .
   [ Santiago Garcia ]
   * Start to add armel support.
 .
   [ Frans Pop ]
   * Drop the base-{etch,lenny} tasks files as make_disc_trees.pl already takes
     care of including packages needed by debootstrap.
   * generate_di+k_list: fix logic error in use of codenames.
     When building a CD for e.g. etch using D-I from e.g. sid, the script
     should update tasks/debian-installer+kernel-etch (instead of -sid), and
     that file should have '#include <debian-installer-sid>'. This means we'll
     no longer have to modify the Debian_* tasks in such cases.
 .
   [ Raphael Hertzog ]
   * Switch debhelper compatibility level to 5 and use debian/compat
     instead of DH_COMPAT.
   * Clean up debian/rules a bit.
 .
   [ Joey Hess ]
   * Add per-language desktop tasks to the task.list files.
   * Path to sparc daily builds has changed.
 .
   [ Frans Pop ]
   * Update tasksel tasks for Lenny Beta2.
 .
   [ Steve McIntyre ]
   * tools/update_tasks: Significant streamlining to make a huge difference in
     runtime. Now feasible to run this automatically per-build rather than
     cronning it.
 .
   [ Joey Hess ]
   * Update isolinux to version 3.63+dfsg-2 (featuring 'menu default64'
     support).
 .
   [ Steve McIntyre ]
   * tools/make-firmware-image: Simple script to create .tar.gz and
     .zip files containing non-free firmware and microcode packages.
   * Add support for dual-layer DVD, Blu-Ray and dual-layer Blu-Ray
     discs. Nothing special, just extra sizes recognised.
   * Fix bashism in tools/grab_md5. Closes: #489562
   * Change lynx dependency to use lynx-cur|lynx instead. Closes: #489486
   * Add sanity check for data passed to add_packages.
   * Workaround to avoid errors when building images for Etch+1/2 (using
     packages from stable but D-I/udebs from testing): don't include source
     for udebs in images. See also #490652.
 .
   [ Frans Pop ]
   * Support including packages from proposed-updates.
     This can be useful for pre-release testing of point releases, but should
     be used with care when CDs are intended to be published.
   * build.sh: after the earlier change in generate_di+k_list in this release,
     this script should also use the debian-installer+kernel task for the
     requested codename, and not the used D-I release.
   * build_all.sh: make setting of TASK consistent with build.sh.
 .
   [ Joey Hess ]
   * Disable zd1211-firmware as its dependencies currently
     break d-i. (See #492205)
 .
   [ Steve McIntyre ]
   * Applied patch from Robert Millan <rmh@aybabtu.com> to allow choice of
     desktop when building for win32-loader. Closes: #491300.
 .
   [ Frans Pop ]
   * Include packages needed for systems using loop-aes encryption on
     installation CDs. Closes: #486360.
 .
   [ Otavio Salvador ]
   * Include speakup and atl2 modules for i386 and amd64 medias.
   * Include multipath-tools-boot for all medias.
 .
   [ Steve McIntyre ]
   * Add the "-joliet-long" option when making images with Joliet names.
     Some packages with ridiculously long names are not unique in the
     first 64 characters, so normal Joliet sorting fails.
   * Cope better with source/binary package mismatches in
     tools/merge_package_lists.
   * Improve control over production of iso and jigdo files. Replace the
     simple DOJIGDO config option with MAXISOS and MAXJIGDOS, allowing
     more flexibility. Allows us to only produce a small number of iso
     images for less-popular architectures but still produce all the jigdo
     files, saving a lot of disk space.
   * Recommend syslinux-common instead of syslinux. Closes: #497966
   * Major licensing problem cleanup: (Closes: #497270)
     + Remove binary blobs from the package; pull those files from the
       .debs in the archive as needed (isolinux.bin, vesamenu.c32)
     + sbm.bin no longer needed, so dropped it
     + Pull out the code to find the right deb and put it in a new helper
       shell script (tools/which_deb), called from Makefile and boot-* as
       needed.
   * Support KERNEL_PARAMS for powerpc, alpha and sparc (Closes: #416230)
     Thanks to Vagrant Cascadian for the patch.
 .
   [ Frans Pop ]
   * Allow to skip generating MD5 and SHA1 sums files.
   * Add alsa-base and alsa-utils to forcd1 for Lenny as they are no longer
     key packages in the desktop task, but we do want them on the first CD.
   * Drop support for building Sarge CDs.
   * Allow multi-arch builds using D-I images from official mirrors or local
     D-I images.
   * Add easy-build.sh, a wrapper script for building images that takes care
     of a lot of the gotchas that can make debian-cd hard to use.
   * CONF.sh: change default CODENAME to lenny; change example paths to mirror
     to /srv/mirror.
   * Exclude 686-bigmem kernel udebs as they are only used for Xen netboot.
   * Stop including jfbterm and unifont on all images; jbfterm is no longer
     installed from D-I.
   * Update and cleanup the udeb exclusion lists for lenny (general list and
     lists for i386 and amd64).
Checksums-Sha1: 
 bb34411938c22d479f4251e5c5a1bf4d939a6e26 977 debian-cd_3.0.5.dsc
 7f3d4bc5b7bdecb066195f2d502ff7abf3ff3b93 783577 debian-cd_3.0.5.tar.gz
 245415bab9a25fb8defaa97a85495a747f59da5d 783880 debian-cd_3.0.5_all.deb
Checksums-Sha256: 
 7568acfdbaf6eb1b36af6d99895d83c4ec45d56d3456b9e3b91717586cbccb03 977 debian-cd_3.0.5.dsc
 4c9f0bec75a534c57aecb6850af2efc89bb628b930497b3e6a9a7e9bd7cb927a 783577 debian-cd_3.0.5.tar.gz
 07fd07ab03ad6f0110feb4e5e2f90da3eb0a74c21ff343f84fca7d61fc6d82ac 783880 debian-cd_3.0.5_all.deb
Files: 
 2edcac69424da096d9fd4535bef224a0 977 admin extra debian-cd_3.0.5.dsc
 a1c37c2584ac2130938c70056eed7e52 783577 admin extra debian-cd_3.0.5.tar.gz
 502f06131372063a4e3f83a6e12916fd 783880 admin extra debian-cd_3.0.5_all.deb

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

iD8DBQFJBP6ufDt5cIjHwfcRAmMBAJ9Tmg5gqAc+HqRw/IY4E+qgO8b0PwCeNXaj
DsJWFeikzD3oeS7eaMZdy3M=
=bfkx
-----END PGP SIGNATURE-----



--- End Message ---

Reply to: