Bug#496295: lintian: should produce all error messages and only then exit

2008-08-25 Thread Russ Allbery
Eugene V. Lyubimkin [EMAIL PROTECTED] writes: When I have supplied wrong file name to lintian, I have got the next output: -8- [EMAIL PROTECTED]:~/download/build/html2text-experimental$ lintian -ivI html2text_1.3.2a-7_amd64. N: Setting up lab in /tmp/q8OCuUwaoM ... error: cannot find

Bug#496295: setting package to lintian, tagging 496295

2008-08-25 Thread Russ Allbery
# Automatically generated email from bts, devscripts version 2.10.35 # via tagpending # # lintian (1.25.0) unstable; urgency=low # # * frontend/lintian: #+ [RA] Ensure we close the pipe to lintian-info before exiting, since # otherwise lintian-info may print the rest of our output after

Re: More patches

2008-08-25 Thread Raphael Geissert
Russ Allbery wrote: Raphael Geissert [EMAIL PROTECTED] writes: Attached is a git-am-friendly mbox file containing the patches making lintian check for embedded copies of jsjac and jsMath, and adds another tag checking for embedded copies of PHP PEAR modules (there are lots of those in the

[SCM] Debian package checker branch, master, updated. 1.24.4-62-gba62d53

2008-08-25 Thread Russ Allbery
The following commit has been merged in the master branch: commit ba62d53d81485ffd0b3b5724e5245e1120e177d3 Author: Raphael Geissert [EMAIL PROTECTED] Date: Sun Aug 24 13:58:13 2008 -0500 Detect jsjac and jsMath on the embedded js library check Signed-off-by: Raphael Geissert [EMAIL

Re: More patches

2008-08-25 Thread Russ Allbery
Raphael Geissert [EMAIL PROTECTED] writes: Attached is a git-am-friendly mbox file containing the patches making lintian check for embedded copies of jsjac and jsMath, This part is applied. Thanks! and adds another tag checking for embedded copies of PHP PEAR modules (there are lots of

[SCM] Debian package checker branch, master, updated. 1.24.4-63-g9a292ff

2008-08-25 Thread Russ Allbery
The following commit has been merged in the master branch: commit 9a292ff98547580f7af3649c5ae43d929073eb61 Author: Raphael Geissert [EMAIL PROTECTED] Date: Fri Aug 22 15:44:58 2008 -0500 Implement maintainer-also-in-uploaders * checks/fields{,.desc}: + [RA] Check for a

Re: set of patches

2008-08-25 Thread Russ Allbery
Raphael Geissert [EMAIL PROTECTED] writes: Attached are the following two patches in a git-friendly mbox format: lintian_enhanced_possibly-insecure-handling-of-tmp-files-in-maintainer-script.patch: Requires the tmp dir name to have a name thus reducing the number of false positives and

[SCM] Debian package checker branch, master, updated. 1.24.4-64-g81451d8

2008-08-25 Thread Russ Allbery
The following commit has been merged in the master branch: commit 81451d824cfc64614e10ac70d96b255e2861618d Author: Raphael Geissert [EMAIL PROTECTED] Date: Fri Aug 22 15:03:06 2008 -0500 Improve check for possibly insecure /tmp files * checks/scripts: + [RA] Improve the check

Re: More patches

2008-08-25 Thread Raphael Geissert
Russ Allbery wrote: Raphael Geissert [EMAIL PROTECTED] writes: and adds another tag checking for embedded copies of PHP PEAR modules (there are lots of those in the wild). I think I'm going to decline to apply this one in its current form, or with m/pear/, since the false positive risk

Re: set of patches

2008-08-25 Thread Raphael Geissert
Russ Allbery wrote: Raphael Geissert [EMAIL PROTECTED] writes: Attached are the following two patches in a git-friendly mbox format: lintian_enhanced_possibly-insecure-handling-of-tmp-files-in-maintainer-script.patch: Requires the tmp dir name to have a name thus reducing the number of

Re: More patches

2008-08-25 Thread Russ Allbery
Raphael Geissert [EMAIL PROTECTED] writes: That part was done in pourpose (not even /Filename is required on all cases) because some times some people do rename the files and add a comment or two but the code remains intact. Not even as an experimental tag using m,/pear/,? Oh, hey, good

Re: More patches

2008-08-25 Thread Russ Allbery
Raphael Geissert [EMAIL PROTECTED] writes: That part was done in pourpose (not even /Filename is required on all cases) because some times some people do rename the files and add a comment or two but the code remains intact. Not even as an experimental tag using m,/pear/,? Er, this string

Re: set of patches

2008-08-25 Thread Russ Allbery
Raphael Geissert [EMAIL PROTECTED] writes: Russ Allbery wrote: Thanks, applied with some changes to the long tag description and the addition of the Severity/Certainty tags. Never heard about some good reason to duplicate the information (as what I understood from the new description) but

Re: More patches

2008-08-25 Thread Raphael Geissert
Russ Allbery wrote: Raphael Geissert [EMAIL PROTECTED] writes: That part was done in pourpose (not even /Filename is required on all cases) because some times some people do rename the files and add a comment or two but the code remains intact. Not even as an experimental tag using

[SCM] Debian package checker branch, master, updated. 1.24.4-66-g0557ab1

2008-08-25 Thread Russ Allbery
The following commit has been merged in the master branch: commit 0557ab1b259879bdee0d813251ebbb454836e633 Author: Russ Allbery [EMAIL PROTECTED] Date: Mon Aug 25 22:28:10 2008 -0700 Fix t/runtests single test mode * t/runtests: + [RA] Fix single test mode. diff --git

Re: More patches

2008-08-25 Thread Russ Allbery
Raphael Geissert [EMAIL PROTECTED] writes: well, some files do have a string that would be matched, but probably m,/pear[/.], is a better regex. That's the main drawnback: there's no easy way to tell whether it is a PEAR module or not. PHP, the world of inconsistencies :) I'm currently