Bug#464026: [checks/scripts] Sync bashism tests with checkbashisms

2008-02-04 Thread Russ Allbery
"Adam D. Barratt" <[EMAIL PROTECTED]> writes: > That's what I've changed checkbashsims to use. Unless I'm missing > something (which I'm more than happy to have pointed out :) "echo echo > -e" matches "\s+echo\s+-e" as the expression isn't (in that case) > anchored to the start of the line. Oh, d

Bug#464026: [checks/scripts] Sync bashism tests with checkbashisms

2008-02-04 Thread Adam D. Barratt
On Mon, 2008-02-04 at 16:06 -0800, Russ Allbery wrote: > "Adam D. Barratt" <[EMAIL PROTECTED]> writes: > > > The first two certainly should be prefixed by something to avoid them > > matching strings that happen to end in "echo -e". I've updated > > checkbashisms to do so for all three. > > > > As

Bug#464026: [checks/scripts] Sync bashism tests with checkbashisms

2008-02-04 Thread Russ Allbery
"Adam D. Barratt" <[EMAIL PROTECTED]> writes: > The first two certainly should be prefixed by something to avoid them > matching strings that happen to end in "echo -e". I've updated > checkbashisms to do so for all three. > > As far as I can see that won't stop "echo echo -e" matching, however.

Bug#464026: [checks/scripts] Sync bashism tests with checkbashisms

2008-02-04 Thread Adam D. Barratt
On Mon, 2008-02-04 at 14:50 -0800, Russ Allbery wrote: > "Adam D. Barratt" <[EMAIL PROTECTED]> writes: > > > As discussed recently, please find attached a patch (against lintian and > > devscripts SVN) to checks/scripts that adds the bashism tests that > > devscripts's checkbashisms performs but l

Bug#464026: [checks/scripts] Sync bashism tests with checkbashisms

2008-02-04 Thread Russ Allbery
"Adam D. Barratt" <[EMAIL PROTECTED]> writes: > As discussed recently, please find attached a patch (against lintian and > devscripts SVN) to checks/scripts that adds the bashism tests that > devscripts's checkbashisms performs but lintian does not. Thank you! You beat me to doing the merger. O

Bug#464026: [checks/scripts] Sync bashism tests with checkbashisms

2008-02-04 Thread Adam D. Barratt
Package: lintian Version: 1.2.42 Tags: patch X-Debbugs-Cc: [EMAIL PROTECTED] Hi, As discussed recently, please find attached a patch (against lintian and devscripts SVN) to checks/scripts that adds the bashism tests that devscripts's checkbashisms performs but lintian does not. The only test I'v