Your message dated Sun, 12 Oct 2008 19:47:02 +0000 with message-id <[EMAIL PROTECTED]> and subject line Bug#499971: fixed in devscripts 2.10.39 has caused the Debian Bug report #499971, regarding [checkbashisms] Doesn't catch all brace expansions 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 [EMAIL PROTECTED] immediately.) -- 499971: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=499971 Debian Bug Tracking System Contact [EMAIL PROTECTED] with problems
--- Begin Message ---Package: devscripts Version: 2.10.37 Severity: normal Tags: patch Hi, checkbashisms doesn't notice a brace expansion if either of the components of the brace is empty. For example, % cat test.sh #!/bin/sh cp foo{,baz} % touch foo % dash test.sh cp: missing destination file operand after `foo{,baz}' Try `cp --help' for more information. % bash test.sh % checkbashisms test.sh % The attached patch against devscripts SVN appears to fix this problem for me. Regards, -- Jonny Lamb, UK [EMAIL PROTECTED] http://jonnylamb.com GPG: 0x2E039402Index: checkbashisms.pl =================================================================== --- checkbashisms.pl (revision 1649) +++ checkbashisms.pl (working copy) @@ -451,7 +451,7 @@ q<should be 'b = a'>, qr'\[\s+[^\]]+\s+==\s' => q<should be 'b = a'>, qr'\s\|\&' => q<pipelining is not POSIX>, - qr'[^\\\$]\{([^\s\\\}]+?,)+[^\\\}\s]+\}' => + qr'[^\\\$]\{([^\s\\\}]*?,)+[^\\\}\s]*\}' => q<brace expansion>, qr'(?:^|\s+)\w+\[\d+\]=' => q<bash arrays, H[0]>, $LEADIN . qr'read\s+(?:-[a-qs-zA-Z\d-]+)' => q<read with option other than -r>,
signature.asc
Description: Digital signature
--- End Message ---
--- Begin Message ---Source: devscripts Source-Version: 2.10.39 We believe that the bug you reported is fixed in the latest version of devscripts, which is due to be installed in the Debian FTP archive: devscripts_2.10.39.dsc to pool/main/d/devscripts/devscripts_2.10.39.dsc devscripts_2.10.39.tar.gz to pool/main/d/devscripts/devscripts_2.10.39.tar.gz devscripts_2.10.39_amd64.deb to pool/main/d/devscripts/devscripts_2.10.39_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 [EMAIL PROTECTED], and the maintainer will reopen the bug report if appropriate. Debian distribution maintenance software pp. Adam D. Barratt <[EMAIL PROTECTED]> (supplier of updated devscripts 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 [EMAIL PROTECTED]) -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Format: 1.8 Date: Sun, 12 Oct 2008 20:24:23 +0100 Source: devscripts Binary: devscripts Architecture: source amd64 Version: 2.10.39 Distribution: unstable Urgency: low Maintainer: Devscripts Devel Team <[EMAIL PROTECTED]> Changed-By: Adam D. Barratt <[EMAIL PROTECTED]> Description: devscripts - scripts to make the life of a Debian Package maintainer easier Closes: 499434 499971 500256 500585 501142 501162 501171 501447 501932 Changes: devscripts (2.10.39) unstable; urgency=low . [ Adam D. Barratt ] * bts: + Allow an explicit HELO to be specified when connecting via SMTP. + If a HELO is not specified then attempt to use the content of /etc/mailname instread. + Expand on the definition of "claim" in the manpage. (Closes: #501142) + Trim anything that doesn't look like an e-mail address from $DEBEMAIL or $EMAIL before using it for un/claim or un/subscribe. (Closes: #501162) + Update the documentation for "bts select" to make it clearer that specifying a key multiple times gives "or" semantics. (Closes: #499434) * checkbashisms: + Recognise brace expansions containing a leading or trailing empty value ("{,foo}", etc). Thanks Jonny Lamb. (Closes: #499971) + Recognise the use of "source" with quoted arguments. (Closes: #501171) * debdiff: Ensure exit status is always explicitly set (and set correctly). (Closes: #500256) * debian/rules: Create a link from the French pts-subscribe manpage as pts-unsubscribe, so long as the former exists (i.e. has been sufficiently translated). * licensecheck: Add support for fortran and Scilab files and for detecting the CeCILL and "SGI Free B" licenses. Thanks Sylvestre Ledru (Closes: #501447) * uscan: Allow --download-version to be more useful in situations where a current upstream version higher than that to be downloaded has been specified. Thanks James Westby. (Closes: #501932) . [ James Vega ] * pts-subscribe: When called as pts-unsubscribe, send an unsubscribe request to the PTS. (Closes: #500585) Checksums-Sha1: 576485dd4205d8de17ccf1c176f7426859cbf7e8 1386 devscripts_2.10.39.dsc 6bd990b52ee3f41682df55f2f696e3a48da7c269 613382 devscripts_2.10.39.tar.gz 9df8b1ea0895fa2d1038eff2542b933b5a66c035 531098 devscripts_2.10.39_amd64.deb Checksums-Sha256: eb8b465be292adf16b80b587ac030f1d6e4009e0f0dc33688ce1f29c3da808c7 1386 devscripts_2.10.39.dsc 605c3b626a68e45213c86428a65fa73cb90e7de563056953a3ae9adcb13ec402 613382 devscripts_2.10.39.tar.gz e9356d90b9c21369b1a12df7df506eb71da57318f81c09431b0a29ad871953d5 531098 devscripts_2.10.39_amd64.deb Files: 60f0d8c2d6b20a6f1cc8df3d55afb66b 1386 devel optional devscripts_2.10.39.dsc 6f7871c88ca59837dba0831ac5041049 613382 devel optional devscripts_2.10.39.tar.gz e040cdfa690fd7b89445f5f67482906b 531098 devel optional devscripts_2.10.39_amd64.deb -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.9 (GNU/Linux) iEYEARECAAYFAkjyUDgACgkQokcE1TReOoXAuwCfawQHMFDDjYsDABt1VNfqqY/l JU0An2pHwSlnjrKhK3JJcHglKvuF7IxL =08kM -----END PGP SIGNATURE-----
--- End Message ---
