Re: more files for the embedded-code-copies checks

2008-11-26 Thread Adam D. Barratt
On Mon, 2008-11-24 at 19:50 -0600, Raphael Geissert wrote: Can anyone please apply the attached patches in git-am-friendly format? Applied. Regards, Adam -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]

Bug#506793: lintian: description spell suggestion: Subversion instead of subversion

2008-11-24 Thread Adam D. Barratt
Hi, On Mon, 2008-11-24 at 21:29 +0100, Sandro Tosi wrote: please suggest Subversion instead of subversion in packages descriptions. My only objection is that doing so will mean adding an override to devscripts. ;-) I have to admit I was surprised that there aren't currently any VCSen in the

[SCM] Debian package checker branch, master, updated. 2.0.0-48-gbdb64da

2008-11-24 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit bdb64da5dd8a5b306b5fff53b77f7995912f4a7f Author: Adam D. Barratt [EMAIL PROTECTED] Date: Mon Nov 24 23:37:44 2008 + Ignore square bracket delimited text when running picky spelling checks. lib/Spelling.pm

Bug#506793: lintian: description spell suggestion: Subversion instead of subversion

2008-11-24 Thread Adam D. Barratt
On Mon, 2008-11-24 at 15:04 -0800, Russ Allbery wrote: Adam D. Barratt [EMAIL PROTECTED] writes: On Mon, 2008-11-24 at 21:29 +0100, Sandro Tosi wrote: please suggest Subversion instead of subversion in packages descriptions. My only objection is that doing so will mean adding

Bug#506271: lintian: weirdness when checking manpages with nroff ignore commands

2008-11-20 Thread Adam D. Barratt
Hi, Thanks for your report and for indirectly forcing me to learn a little more about *roff while investigating it. On Thu, 2008-11-20 at 03:03 +0100, Morten Kjeldgaard wrote: I was not able to get rid of lintian warnings in some man pages that made extensive use of the nroff ignore (.ig)

[SCM] Debian package checker branch, master, updated. 2.0.0-45-g5780cc0

2008-11-18 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 5780cc0962eaad5f62132d43fda60ed4e63a8296 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Tue Nov 18 08:59:06 2008 + Add quilt-patch-missing-description to the list of new patches diff --git a/debian/changelog b/debian

2.1.0 upload?

2008-11-18 Thread Adam D. Barratt
Hi, It's nearly two months since we released lintian 2.0.0 and lintian.d.o is still running ~rc2, so I thought I'd raise the question of releasing 2.1.0. Personally, I think that Frank's Lintian::Command and Lintian::Output work is more than worth an upload. In terms of lintian.d.o, there's also

[SCM] Debian package checker branch, master, updated. 2.0.0-42-gfba7268

2008-11-17 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit fba7268990cb472839fc58f5f43278bac1c6b182 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Mon Nov 17 20:12:28 2008 + Only issue one of the symbols-file-contains*debian-revision tags for a symbol. * checks/shared

[SCM] Debian package checker branch, master, updated. 2.0.0-43-g1fdbc19

2008-11-17 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 1fdbc1995439918cd4126fd61c19f92eae025c3a Author: Adam D. Barratt [EMAIL PROTECTED] Date: Mon Nov 17 21:39:58 2008 + Flag shlibs which call {_,}exit() (Closes: #504257) * checks/shared-libs{,.desc}: + Add

Bug#504257: lintian: check for exit() or _exit() calls in shared libraries

2008-11-17 Thread Adam D. Barratt
On Mon, 2008-11-03 at 15:20 +0900, Paul Wise wrote: On Sun, 2008-11-02 at 21:21 -0800, Russ Allbery wrote: I'm not opposed to adding it, but I think we should probably start with it marked as experimental, and I'd expect a lot of false positives. Agreed, definitely an info-level complaint

[SCM] Debian package checker branch, master, updated. 2.0.0-44-g1bfbfea

2008-11-17 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 1bfbfea74fc2a6be8398788992f8fb27994bef75 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Mon Nov 17 22:11:34 2008 + Drop the severity of shlib-calls-exit to wishlist diff --git a/checks/shared-libs.desc b/checks/shared

Bug#504257: lintian: check for exit() or _exit() calls in shared libraries

2008-11-17 Thread Adam D. Barratt
On Mon, 2008-11-17 at 14:03 -0800, Russ Allbery wrote: Adam D. Barratt [EMAIL PROTECTED] writes: I wasn't entirely sure about the Severity and Certainty, so suggestions / objections welcome. I would tend towards wishlist / possible, but since it's experimental, it doesn't matter a great

[SCM] Debian package checker branch, master, updated. 2.0.0-39-g9c2a249

2008-11-11 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 9c2a249661ea9da0f8dd1be8757cf0c9b248ef92 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Tue Nov 11 20:13:12 2008 + Add quilt-patch-without-description (Closes: #498892) checks/patch-systems{,.desc

[SCM] Debian package checker branch, master, updated. 2.0.0-41-ga512f42

2008-11-11 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit a512f426527d82392f4a75c69d7ae0f171564f27 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Tue Nov 11 21:36:32 2008 + Add an acknowledgement for the previous change diff --git a/debian/changelog b/debian/changelog index

[SCM] Debian package checker branch, master, updated. 2.0.0-40-gc60734a

2008-11-11 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit c60734a3b3be2e130e3988a1abe27c1aab4d2420 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Tue Nov 11 21:21:27 2008 + Don't treat literal use of ? in descriptions as part of a question. Don't detect the use

[SCM] Debian package checker branch, master, updated. 2.0.0-38-g8daaf81

2008-11-10 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 8daaf81362ed8e74edfe9499fdc5e64be5db96a0 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Mon Nov 10 23:27:32 2008 + Indicate that the patch-systems check script needs diffstat information checks/patch

Bug#503695: lintian: Add support for checking empty DP:

2008-11-08 Thread Adam D. Barratt
On Thu, 2008-10-30 at 18:30 +, Adam D. Barratt wrote: I can't see any reason why the check shouldn't be extended to require a dpatch comment line containing at least one non-space character and will do so unless someone can suggest a good reason. fwiw, a quick grep through lintian.d.o's

[SCM] Debian package checker branch, master, updated. 2.0.0-36-g64c17bd

2008-11-07 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 64c17bda8481d6e51355bea107a5ebe61314d67f Author: Adam D. Barratt [EMAIL PROTECTED] Date: Fri Nov 7 19:26:55 2008 + Don't emit unused-override in situations where we can't know it's unused Don't complain about

lintian.d.o updating issues

2008-11-03 Thread Adam D. Barratt
Hi, As you may (or may not) have noticed, lintian.d.o has been having a few issues in terms of updating each day over the past couple of weeks. In short, the ftp.d.o mirror on the machine is provided via nfs and the mount sometimes appears to have problems when the nfs server is rebooted under

Bug#504302: lintian: Rename --show-overrides to a --display-* option

2008-11-02 Thread Adam D. Barratt
On Sun, 2008-11-02 at 18:43 +0200, jaalto wrote: For canonical option names like these: --display-info --display-experimental ... it would be expected to find similarly named: --display-overrides Renaming an option that's existed for nearly eight years is

Bug#501722: lintian: unused-override is emitted when an override exists for a check which is not -C check

2008-11-01 Thread Adam D. Barratt
On Fri, 2008-10-31 at 15:33 +0100, Frank Lichtenheld wrote: On Fri, Oct 31, 2008 at 06:27:54AM +, Adam D. Barratt wrote: [...] adduser --- The package includes three unused overrides for maintainer-script-needs-depends-on-adduser and lintian issues a number of tags from

Bug#501722: lintian: unused-override is emitted when an override exists for a check which is not -C check

2008-10-31 Thread Adam D. Barratt
On Thu, 2008-10-30 at 18:43 +, Adam D. Barratt wrote: On Thu, 2008-10-09 at 15:12 -0500, Raphael Geissert wrote: $ lintian --show-overrides -C scripts -I xserver-xorg-video-openchrome_0.2.902+svn579-2_i386.deb | cut '-d ' -f-3 I: xserver-xorg-video-openchrome: unused-override I

[SCM] Debian package checker branch, master, updated. 2.0.0-35-g68f9119

2008-10-31 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 68f91190ccbf89274926f99060334681f30b7e1a Author: Adam D. Barratt [EMAIL PROTECTED] Date: Fri Oct 31 19:24:38 2008 + Don't count ## DP: as a dpatch description. Don't include dpatch comment lines containing

Bug#503695: lintian: Add support for checking empty DP:

2008-10-30 Thread Adam D. Barratt
On Mon, 2008-10-27 at 18:32 +0200, jaalto wrote: Given following debian/patches/10_example.dpatch: ## All lines beginning with `## DP:' are a description of the patch. ## DP: Please add report W: for empty patch description. (likewise for DP: ) The tag already exists -

Bug#501722: lintian: unused-override is emitted when an override exists for a check which is not -C check

2008-10-30 Thread Adam D. Barratt
On Thu, 2008-10-09 at 15:12 -0500, Raphael Geissert wrote: $ lintian --show-overrides -C scripts -I xserver-xorg-video-openchrome_0.2.902+svn579-2_i386.deb | cut '-d ' -f-3 I: xserver-xorg-video-openchrome: unused-override I: xserver-xorg-video-openchrome: unused-override I:

[SCM] Debian package checker branch, master, updated. 2.0.0-33-g7628956

2008-10-16 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 02229eb07a8082b81fc0a27c29455888ec0dbb36 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Fri Oct 17 06:32:46 2008 +0100 Remove no longer used variable diff --git a/collection/file-info b/collection/file-info index 4ca3718

[SCM] Debian package checker branch, master, updated. 2.0.0-33-g7628956

2008-10-16 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 7628956a56258e5c0f7c5096cc81ef9e021aecce Author: Adam D. Barratt [EMAIL PROTECTED] Date: Fri Oct 17 06:34:45 2008 +0100 s/catched/caught/ diff --git a/lib/Lintian/Command.pm b/lib/Lintian/Command.pm index 5d882d8..3c46461

[SCM] Debian package checker branch, master, updated. 2.0.0-17-g866d6b1

2008-10-03 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 866d6b19e835eccd383015adfaac410837f3e21e Author: Adam D. Barratt [EMAIL PROTECTED] Date: Fri Oct 3 07:50:58 2008 +0100 Fix typo (s/Knew/New/) diff --git a/debian/changelog b/debian/changelog index d3221d0..fc861bd 100644

Bug#500720: lintian: versioned-dependency-satisfied-by-perl triggered by conflicts and replaces

2008-09-30 Thread Adam D. Barratt
Hi, On Tue, 2008-09-30 at 22:04 +0300, Niko Tyni wrote: Looking at lintian.debian.org, the new versioned-dependency-satisfied-by-perl check incorrectly triggers on the perl and perl-modules packages due to their Conflicts and Replaces fields. The attached patch makes the check simply ignore

Re: Fwd: Cron [EMAIL PROTECTED] cd /org/lintian.debian.org root/reporting/harness -i

2008-09-30 Thread Adam D. Barratt
On Tue, 2008-09-30 at 13:43 -0700, Russ Allbery wrote: I tried to figure out what was causing this a while back and failed. Does anyone have any ideas? I think it's related to the local mirror not updating as quickly as one might desire, but I'm not sure. rm: cannot remove

Re: Fwd: Cron [EMAIL PROTECTED] cd /org/lintian.debian.org root/reporting/harness -i

2008-09-30 Thread Adam D. Barratt
On Tue, 2008-09-30 at 22:56 +0100, Adam D. Barratt wrote: On Tue, 2008-09-30 at 13:43 -0700, Russ Allbery wrote: I tried to figure out what was causing this a while back and failed. Does anyone have any ideas? I think it's related to the local mirror not updating as quickly as one might

Re: Upload and Blog-Post

2008-09-29 Thread Adam D. Barratt
Frank Lichtenheld wrote: On Sun, Sep 28, 2008 at 07:12:41PM +0100, Adam D. Barratt wrote: This appears to be fairly simply fixable by only using readelf if the error returned by objdump was File format not recognized, which accounts for all the 64-bit false positives which the readelf code

[SCM] Debian package checker branch, master, updated. 2.0.0-14-g6c73347

2008-09-29 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 6c73347d83c75d83dd5c01f6308ecfccc4fa793b Author: Adam D. Barratt [EMAIL PROTECTED] Date: Mon Sep 29 18:42:53 2008 +0100 Don't try to readelf-parse files for which objdump returns invalid operation. Only use readelf

Re: Upload and Blog-Post

2008-09-28 Thread Adam D. Barratt
On Wed, 2008-09-24 at 10:56 +0200, Frank Lichtenheld wrote: On Mon, Sep 22, 2008 at 01:20:58AM +0200, Frank Lichtenheld wrote: I've uploaded an rc2 to experimental. I hope I can find some time this week to do a full run with it. I really would like to avoid having to do a 2.0.1 directly

Re: Upload and Blog-Post

2008-09-25 Thread Adam D. Barratt
On Thu, 2008-09-25 at 22:42 +0200, Frank Lichtenheld wrote: The full run is finished and the results are online at lintian.debian.org including the full log with STDERR output. Unless someone finds a serious issue with the results, I plan to do an unstable upload on the weekend. Thanks. The

Re: Upload and Blog-Post

2008-09-24 Thread Adam D. Barratt
Frank Lichtenheld wrote: On Mon, Sep 22, 2008 at 01:20:58AM +0200, Frank Lichtenheld wrote: I've uploaded an rc2 to experimental. I hope I can find some time this week to do a full run with it. I really would like to avoid having to do a 2.0.1 directly after 2.0.0. It's just numbers, but anyway

[SCM] Debian package checker branch, master, updated. 2.0.0-rc2-2-gd89ad45

2008-09-24 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit d89ad45eaf84da97ba23a2477bdf0151d02f685d Author: Adam D. Barratt [EMAIL PROTECTED] Date: Wed Sep 24 19:40:27 2008 +0100 Detect {,foo} and {foo,} as brace expansions in bashism checks. Detect brace expansions

Re: [RFC/PATCH] New Lintian::Output module

2008-09-21 Thread Adam D. Barratt
On Sat, 2008-09-20 at 13:57 +0200, Frank Lichtenheld wrote: A patch series to bundle all the output logic of lintian into modules and to make it easier to define new output formats. Comments welcome. Sounds good to me. Adam -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of

Re: testset/runtests patch

2008-09-13 Thread Adam D. Barratt
On Fri, 2008-09-12 at 19:58 -0500, Raphael Geissert wrote: Frank Lichtenheld wrote: On Thu, Sep 11, 2008 at 09:45:32PM -0500, Raphael Geissert wrote: Finally some patch for something outside checks/ :) Attached mbox/patch prints a warning to STDERR when no checks were processed,

[SCM] Debian package checker branch, master, updated. 1.24.4-117-g99615b2

2008-09-13 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 99615b21f0923c6638d2792d4b2b6d355d7b64c3 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Sat Sep 13 10:48:57 2008 +0100 Ensure that the schedule hash contains arch when processing lab package lists When processing

Re: [SCM] Debian package checker branch, master, updated. 1.24.4-117-g99615b2

2008-09-13 Thread Adam D. Barratt
On Sat, 2008-09-13 at 09:49 +, Adam D. Barratt wrote: diff --git a/lib/Lintian/Schedule.pm b/lib/Lintian/Schedule.pm index 3c36363..a0d2dc6 100644 --- a/lib/Lintian/Schedule.pm +++ b/lib/Lintian/Schedule.pm @@ -47,6 +47,12 @@ sub add_file { } else { ($pkg, $ver, $arch

Re: [SCM] Debian package checker branch, master, updated. 1.24.4-117-g99615b2

2008-09-13 Thread Adam D. Barratt
On Sat, 2008-09-13 at 17:04 +0200, Frank Lichtenheld wrote: On Sat, Sep 13, 2008 at 10:52:36AM +0100, Adam D. Barratt wrote: [...] This should possibly be another call to get_deb_info(), but that seemed a little overkill when we already had the rest of the information; more than happy

Re: testset/runtests patch

2008-09-13 Thread Adam D. Barratt
On Sat, 2008-09-13 at 17:10 +0200, Frank Lichtenheld wrote: On Sat, Sep 13, 2008 at 10:38:23AM +0100, Adam D. Barratt wrote: debian/rules runtests test only runs a single test, with the build directory being debian/tests. You mean debian/rules runtests onlyrun=test Yes, I do. Thanks

[SCM] Debian package checker branch, master, updated. 1.24.4-119-g2275056

2008-09-13 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 239e818660f112ba38512b36ceb4f3a37a9b1f65 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Sat Sep 13 17:15:22 2008 +0100 Revert Ensure that the schedule hash contains arch when processing lab package lists

[SCM] Debian package checker branch, master, updated. 1.24.4-119-g2275056

2008-09-13 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 2275056d3a6eae487539ccb53feaf13a94c00c30 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Sat Sep 13 17:17:06 2008 +0100 Fix an uninitialized value warning in add_file() Currently neither the laboratory files

Re: [SCM] Debian package checker branch, master, updated. 1.24.4-117-g99615b2

2008-09-13 Thread Adam D. Barratt
On Sat, 2008-09-13 at 17:51 +0200, Frank Lichtenheld wrote: On Sat, Sep 13, 2008 at 04:30:24PM +0100, Adam D. Barratt wrote: I'm not sure if it actually causes any problems, but when I was running the harness using the new code the other day it was throwing an uninitialized value warning

[SCM] Debian package checker branch, master, updated. 1.24.4-124-gcf36c56

2008-09-13 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit cf36c56cad0d0138a2fe4ff7f44e69cacf836f62 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Sat Sep 13 18:56:49 2008 +0100 If a package only has I/X tags, don't include it on the main page summary * reporting/templates

[SCM] Debian package checker branch, master, updated. 1.24.4-125-g0e4f3ef

2008-09-13 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 0e4f3ef33e14db57c374b565c844c868a615273f Author: Adam D. Barratt [EMAIL PROTECTED] Date: Sat Sep 13 21:05:05 2008 +0100 Fix inverted logic in commit cf36c56cad0d0138a2fe4ff7f44e69cacf836f62 Packages should only

Re: [RFC/PATCH] Unify handling of known shell names

2008-09-13 Thread Adam D. Barratt
On Sat, 2008-09-13 at 22:47 +0200, Frank Lichtenheld wrote: Several places search for known shell names. As one can see in the code they diverged over time. Move the regex to common_data.pm to avoid that happening in the future. Also add mksh while at it. [...] 1) Is it correct to change

Re: Upload this weekend

2008-09-12 Thread Adam D. Barratt
On Fri, 2008-09-12 at 21:07 +0200, Frank Lichtenheld wrote: Hi. Since I can see no real reason not to, I will upload the current git state to experimental during the weekend (will try to tackle some of the whishlist bugs first). If you have any comments or objections about this, please

Bug#498617: package with only I tags is displayed on normal report and PTS

2008-09-11 Thread Adam D. Barratt
Hi, On Thu, 2008-09-11 at 16:16 +0200, Thijs Kinkhorst wrote: I have a package (php-net-socket) that currently just has two info-level lintian tags. On the page http://lintian.debian.org/maintainer/[EMAIL PROTECTED] it is listed in the bulleted list near the top but the anchor doesn't go

Bug#498617: package with only I tags is displayed on normal report and PTS

2008-09-11 Thread Adam D. Barratt
On Thu, 2008-09-11 at 19:23 +0100, Adam D. Barratt wrote: On Thu, 2008-09-11 at 16:16 +0200, Thijs Kinkhorst wrote: I have a package (php-net-socket) that currently just has two info-level lintian tags. On the page http://lintian.debian.org/maintainer/[EMAIL PROTECTED] it is listed

Bug#497887: lintian: check for unnecessary versioned dependencies on Perl core modules

2008-09-05 Thread Adam D. Barratt
On Fri, 2008-09-05 at 11:15 -0700, Russ Allbery wrote: Niko Tyni [EMAIL PROTECTED] writes: The implementation could use corelist(1) and/or Module::Corelist to check if the version in the dependency is already in the core. I think just transforming libfoo-bar-perl to Foo::Bar would catch

[SCM] Debian package checker branch, master, updated. 1.24.4-101-g0448bc1

2008-09-04 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 0448bc101e4ef395a34d0f4173b4d8779e0d15a5 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Thu Sep 4 08:11:18 2008 +0100 Fix whitespace damage diff --git a/debian/changelog b/debian/changelog index 97f55fa..7fe536d 100644

Bug#497864: lintian: please warn if a manpage contains a POD ERRORS section

2008-09-04 Thread Adam D. Barratt
On Thu, 2008-09-04 at 17:02 -0700, Russ Allbery wrote: Enrico Zini [EMAIL PROTECTED] writes: I noticed that Pod::Man likes to invent a new POD ERRORS manpage section instead of using stderr. This new default behavior was chosen by the Pod::Simple maintainer. It can be turned off in

[SCM] Debian package checker branch, master, updated. 1.24.4-89-geafd9db

2008-09-02 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit eafd9db010abcfbf81579681180f938ff2e4f80c Author: Adam D. Barratt [EMAIL PROTECTED] Date: Tue Sep 2 19:22:14 2008 +0100 Fix a typo in the etch binutils comments without objdump, not without objcopy diff --git

[SCM] Debian package checker branch, master, updated. 1.24.4-84-g6601a34

2008-09-01 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 6601a349a547e37879005047505c8fabbae82998 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Mon Sep 1 23:15:13 2008 +0100 Strip trailing whitespace from template names before checking if they are used Avoid flagging

[SCM] Debian package checker branch, master, updated. 1.24.4-82-gaa599a5

2008-08-31 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit aa599a58238b20784a34ac53fd3dbd4293f62f2d Author: Adam D. Barratt [EMAIL PROTECTED] Date: Sun Aug 31 11:24:45 2008 +0100 Fix some false-positives in the embedded Javascript library check Re-add an accidentally removed

Bug#497215: lintian: False positives for prototype js warnings

2008-08-31 Thread Adam D. Barratt
On Sun, 2008-08-31 at 02:21 +0200, Frank Lichtenheld wrote: On Sun, Aug 31, 2008 at 01:37:54AM +0200, Carl Fürstenberg wrote: On Sun, Aug 31, 2008 at 01:25, Raphael Geissert [EMAIL PROTECTED] wrote: Carl Fürstenberg wrote: the check for embedded prototype javacsript library is prone to

Bug#497215: setting package to lintian, tagging 497215

2008-08-31 Thread Adam D . Barratt
#lintian (2.0.0~rc1) experimental; urgency=low # # * checks/files{,.desc}: #+ [ADB] Re-add an accidentally removed slash to the check for embedded # Javascript libraries, removing some false positives. (Closes: #497215) # package lintian tags 497215 + pending -- To UNSUBSCRIBE,

[SCM] Debian package checker branch, master, updated. 1.24.4-73-g9d359a4

2008-08-30 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 9d359a4eb50dc4b2eb81389ed5aefe3fdc6d25f9 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Sat Aug 30 12:40:06 2008 +0100 Only complain at Info about obsolete build-deps that have alternatives. Don't issue an error tag

[SCM] Debian package checker branch, master, updated. 1.24.4-74-g6af5f53

2008-08-30 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 6af5f5300d0dd5950637e1a5190fdd22205b Author: Adam D. Barratt [EMAIL PROTECTED] Date: Sat Aug 30 12:41:19 2008 +0100 Remove inadverantly committed backup file diff --git a/checks/fields.desc.bak b/checks/fields.desc.bak

[SCM] Debian package checker branch, master, updated. 1.24.4-75-g5ceb970

2008-08-30 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 5ceb9708ee2ee22c917b26bca37f427ab8f2d9ec Author: Adam D. Barratt [EMAIL PROTECTED] Date: Sat Aug 30 12:47:28 2008 +0100 The embedded Javascript libraries are only examples, not an exhaustive list. Rework

[SCM] Debian package checker branch, master, updated. 1.24.4-76-gdd41a1e

2008-08-30 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit dd41a1e5941381126bf5b1d23376e5bbd500d07b Author: Adam D. Barratt [EMAIL PROTECTED] Date: Sat Aug 30 12:49:40 2008 +0100 Fix a number of typoes and spelling mistakes in tag descriptions. diff --git a/checks/binaries.desc b

Re: RFC: lintian laboratory format changes

2008-08-30 Thread Adam D. Barratt
On Sat, 2008-08-30 at 00:32 +0200, Frank Lichtenheld wrote: Solving this would probably be very straightforward (by adding more levels of subdirectories in the laboratory), but it will require some semantic descisions about when a specific version/architecture will be removed. But before

Re: Full lintian run started

2008-08-30 Thread Adam D. Barratt
On Sat, 2008-08-30 at 12:44 +0200, Frank Lichtenheld wrote: I've just started a full lintian run with 1.24.4 on gluck, Cool. this time I remembered to use -c, so that the improvements in the objdump collection script get actually used. Thanks. ;-) Adam -- To UNSUBSCRIBE, email to [EMAIL

Bug#496826: setting package to lintian, tagging 496875, tagging 496826

2008-08-30 Thread Adam D . Barratt
#lintian (1.25.0) unstable; urgency=low # # * checks/debhelper{,.desc}: #+ [FL] Do not require versioned build-depends for compat level 5 # anymore. Sarge is not supported anymore. (Closes: #496826) # * checks/fields{,.desc}: #+ [ADB] Don't issue an error tag for obsolete packages

Re: Full lintian run started

2008-08-30 Thread Adam D. Barratt
On Sat, 2008-08-30 at 21:52 +0200, Frank Lichtenheld wrote: On Sat, Aug 30, 2008 at 01:03:34PM +0100, Adam D. Barratt wrote: On Sat, 2008-08-30 at 12:44 +0200, Frank Lichtenheld wrote: I've just started a full lintian run with 1.24.4 on gluck, Cool. this time I remembered to use

Bug#497215: lintian: False positives for prototype js warnings

2008-08-30 Thread Adam D. Barratt
On Sun, 2008-08-31 at 00:51 +0200, Carl Fürstenberg wrote: Package: lintian Version: 1.24.4 Severity: important the check for embedded prototype javacsript library is prone to many false positives, as prototype is a pretty common word in js-space, and also that it can be common for

Re: more patches

2008-08-30 Thread Adam D. Barratt
On Sun, 2008-08-31 at 01:04 +0200, Frank Lichtenheld wrote: I've no idea against which tree you make these patches but it certainly is not master. From your patch (checks/files.desc): Please depend on the respective PEAR package providing the module and make sure the library can be

Bug#496284: notfound 495176 in 1.24.2.1, notfound 495176 in 1.24.2, found 495176 in 1.24.3 ...

2008-08-24 Thread Adam D . Barratt
# Automatically generated email from bts, devscripts version 2.10.34 # The check wasn't introduced until 1.24.3... notfound 495176 1.24.2.1 notfound 495176 1.24.2 found 495176 1.24.3 found 496284 1.24.3 merge 496284 495176 retitle 495176 Please make at least command-with-path-in-maintainer-script

Bug#495176: found 495176 in 1.24.2

2008-08-23 Thread Adam D . Barratt
# Automatically generated email from bts, devscripts version 2.10.34 # Mark as found in a version from which the sid package is descended found 495176 1.24.2 -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]

[SCM] Debian package checker branch, master, updated. 1.24.3-37-gba7d34b

2008-08-15 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit ba7d34b827dda64b3dfd21f56a198ce38b82612b Author: Adam D. Barratt [EMAIL PROTECTED] Date: Fri Aug 15 07:00:51 2008 +0100 Fix whitespace in a changelog entry diff --git a/debian/changelog b/debian/changelog index 09956ce

Bug#494979: command-with-path-in-maintainer-script cuts command names

2008-08-13 Thread Adam D. Barratt
merge 494979 494723 thanks Loïc Minier wrote, Wednesday, August 13, 2008 4:16 PM this is very minor but I got: W: libpango1.0-common: command-with-path-in-maintainer-script prerm:10 /usr/bin/defoma W: libpango1.0-common: command-with-path-in-maintainer-script postinst:12 /usr/sbin/update

Re: Interesting readelf failures from lintian.d.o

2008-08-12 Thread Adam D. Barratt
On Tue, 2008-08-12 at 13:27 -0700, Russ Allbery wrote: readelf: Warning: Virtual address 0x970c not located in any PT_LOAD segment. readelf: Error: Out of memory allocating 0x8c1f bytes for symbols Needless to say, none of my test cases for the readelf code produced any issues like that.

[SCM] Debian package checker branch, master, updated. 1.24.3-27-g2c578ef

2008-08-12 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 2c578efa4eb1e6e851fbf755b6d653891dc4bb34 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Tue Aug 12 23:43:55 2008 +0100 Correctly parse a readelf symbol version block containing the definition of a single symbol

Re: [SCM] Debian package checker branch, master, updated. 1.24.3-12-g7d05c56

2008-08-11 Thread Adam D. Barratt
On Mon, 2008-08-11 at 03:24 +, Frank Lichtenheld wrote: The following commit has been merged in the master branch: commit bbf9ff16bf55d7df1b48bfa22ef446173070b8e7 Merge: 46ebc81ce27ce438f6a030dfc67cb2a5ddd8ee14 7e7f3d2ea43d67e89b39d0b7ce87b3d36e0e7c9e Author: Frank Lichtenheld [EMAIL

Re: lintian.debian.org - git

2008-08-10 Thread Adam D. Barratt
On Sat, 2008-08-09 at 06:18 +0200, Frank Lichtenheld wrote: I just installed 1.24.3 on lintian.debian.org and started a full run. Thanks for that; it's good to see experimental tags working. :-) One thing I noticed is that the results for apparently-corrupted-elf-binary displayed on lintian.d.o

Re: lintian.debian.org - git

2008-08-10 Thread Adam D. Barratt
On Sun, 2008-08-10 at 19:27 +0200, Frank Lichtenheld wrote: D'oh, all my fault. While I did do a full lintian run, I did not instruct it to remove the laboratory. So your improvements in the collection script didn't actually get used... Yeah, that would explain it. :-) Thanks for looking in to

Bug#494577: Please add exemptions for embedded-javascript-library tag.

2008-08-10 Thread Adam D. Barratt
tags 494577 + pending thanks Hi, On Sun, 2008-08-10 at 19:58 +0100, Chris Lamb wrote: (My own fault, but) I think lintian should have exemptions for package/file pairs in the embedded-javascript-library tag. The lack of this feature is currently causing ~10 false positives where the

Re: Upload ... now

2008-08-09 Thread Adam D. Barratt
On Fri, 2008-08-08 at 17:36 -0700, Russ Allbery wrote: Frank Lichtenheld [EMAIL PROTECTED] writes: I was pondering when best to do the next upload and I came to the conclusion that there really will be no point where it is any more feasible than at any other point. We currently have a

[SCM] Debian package checker branch, master, updated. 1.24.3-2-g12016c1

2008-08-09 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 12016c1135f14011576421b550b6fe7ae28f5a15 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Sat Aug 9 16:25:27 2008 +0100 Fix uninitialized value warnings from the read bashism test (Closes: #494450) Split the read

Bug#494450: setting package to lintian, tagging 494450

2008-08-09 Thread Adam D . Barratt
# Automatically generated email from bts, devscripts version 2.10.33 # via tagpending # # lintian (1.24.4) UNRELEASED; urgency=low # # * checks/scripts: #+ [ADB] Split the read without variable or with invalid options bashism # test in two, thus enabling the checks to be more accurate

[SCM] Debian package checker branch, master, updated. 1.24.3-5-g64b29c0

2008-08-09 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 64b29c0010a90d1f8c3b346866db23744adc7657 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Sat Aug 9 19:32:40 2008 +0100 Improve the tested cases for read bashisms diff --git a/testset/maintainer-scripts/debian/prerm b

Bug#494450: [checks/scripts] problems with the read bashism test

2008-08-09 Thread Adam D. Barratt
On Sat, 2008-08-09 at 11:23 -0300, Frank Lichtenheld wrote: checks/scripts throws Perl warnings: $ lintian -Cscripts tcpquota_1.6.15-12.1_all.deb [...] Use of uninitialized value $match in concatenation (.) or string at /usr/share/lintian/checks/scripts line 693, C line 47. W: tcpquota:

[VAC] August 16th - 30th

2008-08-09 Thread Adam D. Barratt
Hi, I'll be away from the 16th of this month (next Saturday) until the 30th, enjoying the slower pace of life on the canals of Southern England (and the River Thames for a while). I'm not sure how frequent or reliable my 'net access will be during that period as we'll be drifting (pun intended)

Re: ancient man-db version causes false positives

2008-08-08 Thread Adam D. Barratt
On Fri, 2008-08-08 at 20:59 +0200, Carsten Hey wrote: [...] Adam D. Barratt [EMAIL PROTECTED] writes: It appears to be caused by the line .SH SKŁADNIA (synopsis). I'm assuming this is an encoding issue, as I can reproduce it on etch but only with the locale set to C; in utf-8

[SCM] Debian package checker branch, master, updated. 1.24.2-92-gb9e47a1

2008-08-08 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit b9e47a18e89f2fb729bbfef8e903967daabccd0f Author: Adam D. Barratt [EMAIL PROTECTED] Date: Fri Aug 8 23:03:04 2008 +0100 Apply wording, grammar and punctuation fix to some tag descriptions. Replace references

Bug#493903: lintian: please provide a way of limiting the checks run by lintian based on tags

2008-08-07 Thread Adam D. Barratt
On Wed, 06 Aug 2008 12:40:26 -0700, Russ Allbery wrote: Frank Lichtenheld [EMAIL PROTECTED] writes: On Wed, Aug 06, 2008 at 07:05:41AM +0100, Adam D. Barratt wrote: The main difference afaics is that one version quietens the emitting of tags unless one explicitly asked for them, whereas

Re: usage of lintian as an automated and forced qa check tool

2008-08-07 Thread Adam D. Barratt
Joerg Jaspert wrote, Wednesday, August 06, 2008 2:33 PM For the ftpmaster usage we will just run lintian, possibly as lintian --show-overrides --fail-on-warnings --tagfile /path/to/file [with --tagfile to be implemented] and then parse the output for the tags we get back. fwiw, depending on

Bug#491685: debian-copyright-line-too-long is inconvenient

2008-08-07 Thread Adam D. Barratt
Hi, Noah Slater wrote, Thursday, August 07, 2008 2:52 PM I would like to weigh in on this issue. [...] I do not think lintian should be raising any notices about debian/copyright if only because the majority of it's content is beyond our control. /Please/ read the bug logs before weighing

Bug#493903: lintian: please provide a way of limiting the checks run by lintian based on tags

2008-08-06 Thread Adam D. Barratt
On Wed, 2008-08-06 at 02:05 +0100, Adam D. Barratt wrote: Hi, On Tue, 2008-08-05 at 21:40 +0200, Joerg Jaspert wrote: please provide a way to limit the checks that get run by providing an input file listing tags, one per line. I've attached a first-pass patch implementing a --tags-file

Bug#493920: severity of 493290 is important, severity of 493920 is wishlist

2008-08-06 Thread Adam D . Barratt
# Automatically generated email from bts, devscripts version 2.10.34 # Sorry, typoed the bug number severity 493290 important severity 493920 wishlist -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]

Bug#493921: lintian: file-in-unusual-dir redundant in /opt due to dir-or-file-in-opt

2008-08-06 Thread Adam D. Barratt
On Tue, 2008-08-05 at 18:08 -0400, Scott Kitterman wrote: Since any file in /opt will already trigger the Error dir-or-file-in-opt, also firing off the Warning file-in-unusual-dir seems redundant and distracting. Yeah, there's a bunch of directories for which this will occur; for instance,

Bug#493903: lintian: please provide a way of limiting the checks run by lintian based on tags

2008-08-06 Thread Adam D. Barratt
Hi, On Tue, 2008-08-05 at 21:40 +0200, Joerg Jaspert wrote: please provide a way to limit the checks that get run by providing an input file listing tags, one per line. I've attached a first-pass patch implementing a --tags-file option. It only runs those checks which are necessary to test the

[SCM] Debian package checker branch, master, updated. 1.24.2-77-ga6d7e1e

2008-08-06 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit a6d7e1e7b4f502a7d9bcb518c518544fe1a3aa96 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Wed Aug 6 02:06:43 2008 +0100 Fix a typo in ::Collect::index use. It's $info-index, not $info-{index} diff --git a/checks

[SCM] Debian package checker branch, master, updated. 1.24.2-86-gf626e1f

2008-08-06 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit f626e1f3668bed5bd2f8e909c5d85c7e69ee8cc6 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Wed Aug 6 10:53:19 2008 +0100 Don't issue file-in-unusual-dir for /usr/tmp or /var/www Add /usr/tmp and /var/www to the list

Bug#493903: lintian: please provide a way of limiting the checks run by lintian based on tags

2008-08-06 Thread Adam D. Barratt
On Wed, 6 Aug 2008 17:57:40 +0200, Frank Lichtenheld wrote: On Wed, Aug 06, 2008 at 07:05:41AM +0100, Adam D. Barratt wrote: The main difference afaics is that one version quietens the emitting of tags unless one explicitly asked for them, whereas the other potentially raises all tags from

Re: [PATCH/RFC] Ignore dh for sake of binary-arch-rules-but-pkg-is-arch-indep

2008-08-06 Thread Adam D. Barratt
On Wed, 2008-08-06 at 19:58 +0200, Frank Lichtenheld wrote: After a discussion in the Debian Perl Group that we want to unify our debian/rules files as much as possible with the help of dh I tried to come up with a minimal version. My current proposal can be seen at

[SCM] Debian package checker branch, master, updated. 1.24.2-87-g2d99551

2008-08-06 Thread Adam D. Barratt
The following commit has been merged in the master branch: commit 2d99551b9610edb5c048c18174ef3b62e4fe50c1 Author: Adam D. Barratt [EMAIL PROTECTED] Date: Wed Aug 6 21:02:50 2008 +0100 Add Lintian::Collect::Binary::scripts diff --git a/debian/changelog b/debian/changelog index 9ea62f4

<    2   3   4   5   6   7   8   9   >