Bug#691389: [checkbashisms] multiple bug fixes and new checks

[Raphael Geissert]

Package: devscripts
Version: 2.12.5
User: devscripts at packages.debian.org
Usertags: devscripts
Tag: patch
Severity: wishlist

Hi,

Attached is an mbox with a bunch of bug fixes and new checks, #687450 
included.

On IRC I mentioned a "regression" when checking autoconf, but the issue is 
now visible thanks to some bug fixes. In one configure script the bug fix 
actually revealed a bashism.

I tried to fix two other parsing issues but the way quoted strings are 
handled does not really allow them to be fixed without rewriting a good deal 
of checkbashisms.

Other than the visible issue with autoconf, there was no regression based on 
my test suite. I ran a version of checkbashisms based on a revision from a 
few days ago against the shell scripts in all .debs and there was no major 
regression.

As I type there is another run in progress based on the version of master 
plus the attached patches. Will report back if I find any major issue.

P.S. don't take too much of the function name regexes, beware :)

Cheers,
-- 
Raphael Geissert - Debian Developer
www.debian.org - get.debian.net
-------------- next part --------------
A non-text attachment was scrubbed...
Name: checkbashisms.mbox
Type: application/mbox
Size: 25497 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/devscripts-devel/attachments/20121024/79c2f9c1/attachment-0001.bin>