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

2013-02-24 Thread Raphael Geissert
Hi, As I mentioned on IRC, the new check for $_ introduced some false positives. Attached mbox fixes the regressions found so far. Cheers, -- Raphael Geissert - Debian Developer www.debian.org - get.debian.net underscore-FPs-fixes.mbox Description: application/mbox

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

2012-11-10 Thread Benjamin Drung
Am Donnerstag, den 01.11.2012, 14:27 -0600 schrieb Raphael Geissert: > On Thursday 25 October 2012 16:44:03 Benjamin Drung wrote: > > Am Mittwoch, den 24.10.2012, 21:15 -0500 schrieb Raphael Geissert: > > > Attached is an mbox with a bunch of bug fixes and new checks, #687450 > > > included. > >

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

2012-11-01 Thread Raphael Geissert
On Thursday 25 October 2012 16:44:03 Benjamin Drung wrote: > Am Mittwoch, den 24.10.2012, 21:15 -0500 schrieb Raphael Geissert: > > Attached is an mbox with a bunch of bug fixes and new checks, #687450 > > included. > > Thanks. I have applied your patches 1 up to 14. I grabbed the working > test c

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

2012-10-26 Thread Benjamin Drung
Am Freitag, den 26.10.2012, 12:13 -0400 schrieb James McCoy: > On Fri, Oct 26, 2012 at 02:31:34PM +0200, Benjamin Drung wrote: > > Am Donnerstag, den 25.10.2012, 17:33 -0500 schrieb Raphael Geissert: > > > > Please do not sign off your own patches. Your patches will be signed off > > > > by the per

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

2012-10-26 Thread James McCoy
On Fri, Oct 26, 2012 at 02:31:34PM +0200, Benjamin Drung wrote: > Am Donnerstag, den 25.10.2012, 17:33 -0500 schrieb Raphael Geissert: > > > Please do not sign off your own patches. Your patches will be signed off > > > by the person that applies your patches. > > > > Perhaps we differ on what the

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

2012-10-26 Thread Benjamin Drung
Am Donnerstag, den 25.10.2012, 17:33 -0500 schrieb Raphael Geissert: > Hi, > > On Thursday 25 October 2012 16:44:03 Benjamin Drung wrote: > > Am Mittwoch, den 24.10.2012, 21:15 -0500 schrieb Raphael Geissert: > > > Attached is an mbox with a bunch of bug fixes and new checks, #687450 > > > include

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

2012-10-25 Thread Raphael Geissert
Hi, On Thursday 25 October 2012 16:44:03 Benjamin Drung wrote: > Am Mittwoch, den 24.10.2012, 21:15 -0500 schrieb Raphael Geissert: > > Attached is an mbox with a bunch of bug fixes and new checks, #687450 > > included. > > Thanks. I have applied your patches 1 up to 14. I grabbed the working > t

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

2012-10-25 Thread Benjamin Drung
Am Mittwoch, den 24.10.2012, 21:15 -0500 schrieb Raphael Geissert: > Attached is an mbox with a bunch of bug fixes and new checks, #687450 > included. Thanks. I have applied your patches 1 up to 14. I grabbed the working test cases from your git repository and added them to devscripts. After appl

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

2012-10-24 Thread Raphael Geissert
Package: devscripts Version: 2.12.5 User: devscri...@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