Re: One less TODO item: remove old unbalance `' quotes from all tag descriptions

2009-01-24 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: > >> In the attached patch I replaced all "unbalanced quotes" with single >> quotation marks. > > Thanks! I think double quote marks are better and are what was more > widely used in Lintian already, so t

One less TODO item: remove old unbalance `' quotes from all tag descriptions

2009-01-22 Thread Raphael Geissert
Hi, In the attached patch I replaced all "unbalanced quotes" with single quotation marks. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian_TODO.mbox Description: application/mbox

Bug#512678: [t/] Add support for calling lintian with different command line arguments in a per-test basis

2009-01-22 Thread Raphael Geissert
ic" test I added to test this new feature of runtests I think it would be wise to add a similar test for every check (of course adding a test for every CLI option would be better). [1] The missing "use Utils" and "Needs-Info: source-control-data". Cheers, -- R

Bug#512560: [lib/] some scripts use fail but don't use Util

2009-01-21 Thread Raphael Geissert
unctions calling fail where actually used or not, but what I know is that in the feature if it is used it will... "fail to fail" :) IMO the patch should be applied and then the code reviewed to find out whether that function should remain around. Cheers, -- Raphael Geissert - Debian Maint

Bug#512563: [checks/standards-version.desc] missing Needs-Info on source-control-fields

2009-01-21 Thread Raphael Geissert
Package: lintian Version: 2.1.6 Tags: patch Hi again, Now I found the reason why Lintian::Collect::Source was calling fail, standards-version requires source-control-fields. This was leading to errors when calling lintian with -C standard-versions. Cheers, -- Raphael Geissert - Debian

Bug#512560: [lib/] some scripts use fail but don't use Util

2009-01-21 Thread Raphael Geissert
well. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian_missing_fail.mbox Description: application/mbox signature.asc Description: This is a digitally signed message part.

Re: Lintian cleanup - patch

2009-01-21 Thread Raphael Geissert
Adam D. Barratt wrote: > On Tue, 2009-01-20 at 22:06 -0600, Raphael Geissert wrote: >> Attached is an mbox containing a couple of changes. The first one gets rid >> of the 'Type' leftovers, and the second corrects a couple of spelling >> mistakes and typos. &

Bug#512478: [checks/debhelper] debhelper-but-no-misc-depends should check pre-depends as well and should warn when in suggests or recommends

2009-01-20 Thread Raphael Geissert
looks for pre-depends in addition to depends. The second patch re-introduces the check for suggests but when present in those cases, a tag is emitted (for the same reason I stated above). Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian_debhelper-misc.mbox

Lintian cleanup - patch

2009-01-20 Thread Raphael Geissert
Hi all, Attached is an mbox containing a couple of changes. The first one gets rid of the 'Type' leftovers, and the second corrects a couple of spelling mistakes and typos. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian_cleanup.mbox D

Bug#512265: lintian: please encourage running test suites at build time

2009-01-18 Thread Raphael Geissert
this check)? Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net signature.asc Description: This is a digitally signed message part.

Bug#512119: False positives for update menus and bashism checks

2009-01-17 Thread Raphael Geissert
gt;> What would be really handy here is a pointer to policy change history. > > Like a mention of /usr/share/doc/debian-policy/upgrading-checklist.txt.gz? > Or probably to the online version, as I'm sure not everyone has debian-policy installed (or not on every machine). C

Re: Bug#373767: pedantic tags - patch

2009-01-13 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: >> Raphael Geissert wrote: >> [...] >>> Attached is an mbox with the patches implementing the --pedantic >>> options/severity plus a few initial checks. >> >> ping > > It looked good to me o

Re: Bug#373767: pedantic tags - patch

2009-01-13 Thread Raphael Geissert
Raphael Geissert wrote: [...] > Attached is an mbox with the patches implementing the --pedantic > options/severity plus a few initial checks. ping Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to debian-lint-main

Bug#373767: pedantic tags - patch

2009-01-11 Thread Raphael Geissert
and a 'Weight'-per-tag idea later. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net pedantic.mbox Description: application/mbox

Bug#42936: lintian: does not detect incorrect bang paths in gziped files

2009-01-11 Thread Raphael Geissert
it is nice when all you do is gunzip and run it out of the box. So, shall I reopen and assign it to me? reopen 42936 owner 42936 Raphael Geissert block 42936 with 373767 Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net signature.asc Description: This is a digitally signed message part.

Re: Processed: owner 373767, block 497344 with 373767, block 497346 with 373767, block 474590 with 373767 ...

2009-01-06 Thread Raphael Geissert
heers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to debian-lint-maint-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Bug#497344: lintian: please warn about ${shlibs:Depends} when Architecture: all

2009-01-06 Thread Raphael Geissert
uld be added to --pedantic (still thinking on how to implement it). Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to debian-lint-maint-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Re: git gc and the Lintian repository

2009-01-06 Thread Raphael Geissert
an page or have enough prior experience to make any > recommendations? > What about asking the alioth admins to do that? I guess, at least, a GB or two could be saved if they setup some cron-triggered script that runs git gc on all dirs under /git with a .git dir. Cheers, -- Raphael Geisser

Re: Thoughts on the new Lintian test suite

2009-01-06 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: >> Russ Allbery wrote: > >> What about grouping together tests belonging to the same check? >> E.g. move all the cruft tests to t/tests/cruft/ > > We could do that -- it trades off directory size with more nesting. I

Re: Thoughts on the new Lintian test suite

2009-01-06 Thread Raphael Geissert
> adding a new stanza to debian/control.in. I agree that it is PITA to write new tests and that's why I've been still adding tests to my checks in the testset suite. I like the idea of one test per tag, as it allows for isolated and cleaner testing of every check; but we should als

Re: 2.1.1 and l.d.o

2008-12-16 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: > >> When is lintian.d.o going to be updated to use 2.1.1? >> It's been more than a week since it was released. > > As soon as I got a chance -- it's been at the top of my non-work priority > list for a few d

2.1.1 and l.d.o

2008-12-16 Thread Raphael Geissert
Hi, When is lintian.d.o going to be updated to use 2.1.1? It's been more than a week since it was released. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to debian-lint-maint-requ...@lists.debian.org with a subject of "u

Re: 2.1.1 upload?

2008-12-08 Thread Raphael Geissert
e upload if required, but didn't want to >> presume, being somewhat the "new boy" still :) > > Please go ahead. I'll take care of updating lintian.d.o after it's > uploaded. > what about including my latest patch before uploading? :D Cheers, -- Raphae

Bug#507917: debhelper + data-only packages = annoying lintian warning

2008-12-06 Thread Raphael Geissert
2008/12/6 Eugene V. Lyubimkin <[EMAIL PROTECTED]>: > Raphael Geissert wrote: >> Hi, >> >> Eugene V. Lyubimkin wrote: >> [...] >>> Lintian recently started to produce 'debhelper-but-no-misc-depends' >>> warnings. While it reveal true issu

More php libs

2008-12-05 Thread Raphael Geissert
e any more time on that :) Thanks in advance. Cheers, - -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.9 (GNU/Linux) iEYEARECAAYFAkk6GAgACgkQYy49rUbZzlraRwCeLVHn41E7B9MbGWK+tCIP56zR MuwAn3zUpE2fehyViHHdYmI/GXUHeZiW

Bug#507917: debhelper + data-only packages = annoying lintian warning

2008-12-05 Thread Raphael Geissert
ease give an example? Thanks, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Re: new set of changes

2008-11-29 Thread Raphael Geissert
Raphael Geissert wrote: > Frank Lichtenheld wrote: > >> On Wed, Jul 30, 2008 at 08:45:08PM -0500, Raphael Geissert wrote: >>> Hi all, >>> >>> Attached is a patch making the following changes: >>> * correctly parse line continuations >>&g

Bug#498875: lintian: please detect 2.x licences and warn

2008-11-29 Thread Raphael Geissert
packages won't help because it may cause false negatives. And of course, only old versions of PHP itself will trigger the warning (not even etch's version trigger it), so it should be just fine not to exclude anything. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.

Bug#505857: lintian: false positive debian-watch-file-should-mangle-version

2008-11-28 Thread Raphael Geissert
soon, which just like the checkbashisms as pm stuff will need to be integrated in lintian afterwards. So? [1]This is on my ToDo list for DEHS2 anyway. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subje

Bug#499737: [checks/debhelper] emit debhelper-but-no-miscDepends

2008-11-28 Thread Raphael Geissert
2008/11/28 Frank Lichtenheld <[EMAIL PROTECTED]>: > On Mon, Nov 03, 2008 at 08:19:40PM -0600, Raphael Geissert wrote: >> >> I'm not really sure, but I think you sent me an email about this and the >> patch >> no longer applying in master. I can't confi

more files for the embedded-code-copies checks

2008-11-24 Thread Raphael Geissert
Hi, Can anyone please apply the attached patches in git-am-friendly format? Thanks in advance. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian_files.1.mbox Description: application/mbox lintian_files.2.mbox Description: application/mbox

Bug#499737: [checks/debhelper] emit debhelper-but-no-miscDepends

2008-11-03 Thread Raphael Geissert
On Sunday 21 September 2008, you wrote: > On Sun, Sep 21, 2008 at 01:33:54PM -0500, Raphael Geissert wrote: > > Forwarding a patch I sent in [EMAIL PROTECTED] to the BTS, so it > > is noticed. > > This one is currently on my 2.1.0 TODO list. Which should not keep > anyone

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

2008-10-30 Thread Raphael Geissert
postrm:5 > /usr/bin/ocaml-md5sums > I: ocaml-compiler-libs: unused-override > copyright-should-refer-to-common-license-file-for-lgpl Right, I only tried the former but not the latter. > > Regards, > > Adam > Cheers, -- Raphael Geissert - Debian Maintainer www.debian.or

Bug#503875: [lintian] Lintian does not complain about missing binary-arch or binary-indep target in debian/rules

2008-10-28 Thread Raphael Geissert
formation. --- > Architecture: amd64 > Kernel: Linux 2.6.26-1-amd64 > > Debian Release: lenny/sid > 500 testing security.debian.org > 500 testing ftp.fr.debian.org > Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -

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

2008-10-09 Thread Raphael Geissert
Package: lintian Version: 2.0.0 Severity: minor Example: $ lintian --show-overrides -I xserver-xorg-video-openchrome_0.2.902+svn579-2_i386.deb | cut '-d ' -f-3 O: xserver-xorg-video-openchrome: non-dev-pkg-with-shlib-symlink O: xserver-xorg-video-openchrome: non-dev-pkg-with-shlib-symlink O: xse

Re: Bug#498875: lintian: please detect 2.x licences and warn

2008-09-27 Thread Raphael Geissert
Frank Lichtenheld wrote: > On Sat, Sep 13, 2008 at 06:44:27PM -0500, Raphael Geissert wrote: >> Raphael Geissert wrote: >> +if (m/(The\s+PHP\s+Licen[cs]e,?\s+version\s+2)/si) { >> +tag "copyright-refers-to-non-free-php-license", ""; >> +} >>

Re: Adding checks depending on the availability of information in the data/ files

2008-09-23 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert <[EMAIL PROTECTED]> > writes: > >> Today I had an idea on how could lintian help accomplish such kind of RG >> that require external information that is absolutely not going to be >> hardcoded. The idea for this spec

Adding checks depending on the availability of information in the data/ files

2008-09-23 Thread Raphael Geissert
Hi all, I believe everybody knows that there was a RG for lenny that was about making sure pre-/depends/recommends relationships in packages in main were available in main (i.e. not in contrib, non-free, 3rd party repository). Today I had an idea on how could lintian help accomplish such kind of

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

2008-09-23 Thread Raphael Geissert
Frank Lichtenheld wrote: > On Mon, Sep 22, 2008 at 06:40:15PM -0500, Raphael Geissert wrote: >> >> Hmm, right. Maybe output "formaters" should be able to indicate at what >> point of the process they want to act/be called. > > Nah, no need to expose t

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

2008-09-22 Thread Raphael Geissert
Jordà Polo wrote: > On Sun, Sep 21, 2008 at 12:51:44PM -0500, Raphael Geissert wrote: >> Btw, why would it be any useful to print them until the whole package is >> processed? I would find printing them after every check script is run >> better. > > The current lint

Bug#498883: Please add support for overridable tags

2008-09-21 Thread Raphael Geissert
2008/9/21 Russ Allbery <[EMAIL PROTECTED]>: > "Raphael Geissert" <[EMAIL PROTECTED]> writes: >> 2008/9/14 Russ Allbery <[EMAIL PROTECTED]>: > >>> I already explained why I think this is a bad idea. I don't think > >> Actually, what I

Bug#498883: Please add support for overridable tags

2008-09-21 Thread Raphael Geissert
2008/9/14 Russ Allbery <[EMAIL PROTECTED]>: > Raphael Geissert <[EMAIL PROTECTED]> writes: > >> The attached patch adds support for "overridable" tags. Any value in >> the .desc files can be overriden if the lowercase name is in its >> corresponding

On dropping 'Type'

2008-09-21 Thread Raphael Geissert
Hi, Now that the EWI code is generated via %codes, why is Type still around then? Cheers, -- Atomo64 - Raphael Please avoid sending me Word, PowerPoint or Excel attachments. See http://www.gnu.org/philosophy/no-word-attachments.html -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subjec

Bug#499737: [checks/debhelper] emit debhelper-but-no-miscDepends

2008-09-21 Thread Raphael Geissert
Package: lintian Version: 1.24.4 Severity: wishlist Tags: patch Forwarding a patch I sent in [EMAIL PROTECTED] to the BTS, so it is noticed. Cheers, -- Atomo64 - Raphael Please avoid sending me Word, PowerPoint or Excel attachments. See http://www.gnu.org/philosophy/no-word-attachments.html

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

2008-09-21 Thread Raphael Geissert
Jordà Polo wrote: ... > > One of the outputs I thought about involved printing after processing > the packages instead of printing as tags are found. This could also help avoid the usage of $warned{tag-name} and similar kind of checks to avoid issuing duplicated tags. Of course it also depends o

Bug#499074: [checks/patch-systems] depend on $(QUILT_STAMPFN), not patch

2008-09-16 Thread Raphael Geissert
Russ Allbery wrote: > > Having a rule in debian/rules that depends on patch when using the quilt > makefile fragment is almost always wrong, since patch is a phony target > that will force any rule that depends on it to be rebuilt. This often > results in repeated compiles on the buildds, wasting

Bug#498883: Please add support for overridable tags

2008-09-13 Thread Raphael Geissert
Package: lintian Version: 1.24.4 Severity: wishlist Tags: patch The attached patch adds support for "overridable" tags. Any value in the .desc files can be overriden if the lowercase name is in its corresponding Overridable field. Example: in .desc: Tag: test-tag Type: warning Severity: normal

Bug#498875: lintian: please detect 2.x licences and warn

2008-09-13 Thread Raphael Geissert
Raphael Geissert wrote: > Tags: patch > Now actually including the patch. Cheers, -- Atomo64 - Raphael Please avoid sending me Word, PowerPoint or Excel attachments. See http://www.gnu.org/philosophy/no-word-attachments.html diff --git a/checks/copyright-file b/checks/copyright-file

Bug#498876: lintian: please include version-refers-to-distribution

2008-09-13 Thread Raphael Geissert
Package: lintian Version: 1.24.4 Severity: wishlist Tags: patch The discussion is dead, so I'm now recording it as a report. --- Forwarded message Subject: yet another patch From: Raphael Geissert Date: Sun, 31 Aug 2008 21:05:48 -0500 Newsgroup: gmane.linux.debian.devel.lint.

Re: lintian_copyright-without-license-notice

2008-09-13 Thread Raphael Geissert
Raphael Geissert wrote: > Hi all, > > I've just implemented another check, this time for the copyright file > which basically tries to detect the ones that don't provide "any" licence > information at all. > > I've tried to make it as less prone to

Bug#498875: lintian: please detect 2.x licences and warn

2008-09-13 Thread Raphael Geissert
Package: lintian Version: 1.24.4 Severity: wishlist Tags: patch Since there has been no follow up, I'm now filing it as a bug report to have a record. --- Forwarded message Subject: Re: Some more patches From: Raphael Geissert Date: Sat, 28 Jun 2008 22:38:40 -0500 News

Re: another patch/tag

2008-09-13 Thread Raphael Geissert
Frank Lichtenheld wrote: > On Sat, Sep 13, 2008 at 12:02:20AM -0500, Raphael Geissert wrote: >> I'm attaching an mbox which contains three commits of mine that are >> debhelper-related mixed with some other commits that I merged. I know it >> is a mess, but I'm fall

Re: testset/runtests patch

2008-09-13 Thread Raphael Geissert
Adam D. Barratt wrote: > 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 " only runs a single test, with the build >> > directory being "debian/tests". >> >> You mean "debian/rules runtests

Re: testset/runtests patch

2008-09-12 Thread Raphael Geissert
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, which can happen if a wrong test

Re: another patch/tag

2008-09-12 Thread Raphael Geissert
Frank Lichtenheld wrote: > > Sure, patches to move that stuff to data/ welcome. > Right, but the major problem I can see with debhelper-related data is that, for example, in my case I get: $ grep -l misc:Depends /usr/bin/dh_* /usr/bin/dh_gconf /usr/bin/dh_gtkmodules /usr/bin/dh_installcatalogs

Re: Bug#381485: any news on the lintian check?

2008-09-12 Thread Raphael Geissert
Frank Lichtenheld wrote: > On Thu, Sep 11, 2008 at 09:38:25PM -0500, Raphael Geissert wrote: >> On Monday 25 August 2008, Holger Levsen wrote: >> > downgrading, as all blocking bugs for 438885 have been fixed. >> > >> > Any news on implementing the lintian ch

testset/runtests patch

2008-09-11 Thread Raphael Geissert
Heya, Finally some patch for something outside checks/ :) Attached mbox/patch prints a warning to STDERR when no checks were processed, which can happen if a wrong testset directory was specified (and it will hopefully save me some minutes next time I happen to write 'checks' instead of 'testset'

Bug#381485: any news on the lintian check?

2008-09-11 Thread Raphael Geissert
tag 381485 patch thanks On Monday 25 August 2008, Holger Levsen wrote: > severity 438885 important > thanks > > Hi, Hi Holger, > > downgrading, as all blocking bugs for 438885 have been fixed. > > Any news on implementing the lintian check? ;) As promised, attached is a patch in a mbox implemen

Re: one more

2008-09-11 Thread Raphael Geissert
Frank Lichtenheld wrote: > On Thu, Sep 04, 2008 at 03:14:52PM -0500, Raphael Geissert wrote: >> Tag: maintainer-script-ignores-errors >> Type: warning >> Severity: normal >> Certainty: certain >> Ref: policy 3.9 >> Info: The maintainer script doesn't

debhelper: please set an empty value for misc:Depends when not defined (was: Re: another patch/tag)

2008-09-11 Thread Raphael Geissert
Package: debhelper Severity: wishlist Version: 7.0.17 Hi Joey, Sorry for the large reference, but hope it is useful. Frank Lichtenheld wrote: > On Thu, Sep 11, 2008 at 03:31:06PM -0700, Russ Allbery wrote: >> Frank Lichtenheld <[EMAIL PROTECTED]> writes: >> >> > I like the general idea of this

Re: another patch/tag

2008-09-11 Thread Raphael Geissert
Frank Lichtenheld wrote: > On Sun, Sep 07, 2008 at 08:13:30PM -0500, Raphael Geissert wrote: >> Tag: debhelper-but-no-miscDepends >> Type: warning >> Severity: normal >> Certainty: certain >> Ref: debhelper(7) >> Info: The source package uses debhelper but

another patch/tag

2008-09-07 Thread Raphael Geissert
Hi all, I must be boring all of you, but here's another mbox/patch: Tag: debhelper-but-no-miscDepends Type: warning Severity: normal Certainty: certain Ref: debhelper(7) Info: The source package uses debhelper but it does not have ${misc:Depends} on the given binary package debian/control entry.

Bug#498197: lintian: source-field-malformed would never be triggered

2008-09-07 Thread Raphael Geissert
Package: lintian Version: 1.24.4 Tags: patch The attached mbox contains a patch fixing a bug where source-field-malformed would never be triggered. Although I hardly see how could such a situation ever occur, at least now lintian will emit the tag if appropriate. Cheers, -- Atomo64 - Raphael

Bug#498096: lintian: doesn't ignore udebs on no-standards-version-field

2008-09-06 Thread Raphael Geissert
Package: lintian Version: 1.24.4 checks/standards-version: > # udebs aren't required to conform to policy, so they don't need > # Standards-Version. (If they have it, though, it should be valid.) > my $version = $info->field('standards-version'); > if (not defined $version) { > tag 'no-standar

Re: new set of changes

2008-09-06 Thread Raphael Geissert
Frank Lichtenheld wrote: > On Wed, Jul 30, 2008 at 08:45:08PM -0500, Raphael Geissert wrote: >> Hi all, >> >> Attached is a patch making the following changes: >> * correctly parse line continuations >> * avoid trying to perform any extra check on vers

yet another check

2008-09-05 Thread Raphael Geissert
Hi, Tag: ORed-enhances Type: warning Severity: minor Certainty: certain Info: ORed relations in the Enhances field are incorrect as the package does not enhance one package or the other, it enhances both. The enhanced packages should be instead separated by commas. Credits: edos-debcheck Cheer

Re: two more tags

2008-09-05 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: > >> Why false positives? I mean, I see no reason to ship files that only >> make sense when developing under a Windows system even under >> usr/share/doc; some files are even generated by Visual Studio which have >> a

Re: two more tags

2008-09-05 Thread Raphael Geissert
Frank Lichtenheld wrote: > On Thu, Sep 04, 2008 at 02:26:32PM -0500, Raphael Geissert wrote: >> Tag: windows-devel-file-in-package >> Type: warning >> Severity: normal >> Certainty: certain >> Info: This package appears to contain development files only meaningf

Re: two more tags

2008-09-05 Thread Raphael Geissert
Frank Lichtenheld wrote: > On Thu, Sep 04, 2008 at 05:13:43PM -0500, Raphael Geissert wrote: >> Attached is an updated mbox (btw, I didn't like how the link to walters' >> guide looked like when displayed via Refs:). > > Hmm, someone should probably contact him

Re: two more tags

2008-09-04 Thread Raphael Geissert
Jordà Polo wrote: > On Thu, Sep 04, 2008 at 02:26:32PM -0500, Raphael Geissert wrote: >> Hi there, >> >> Attached are two mbox files each adding one of the following tags/checks: >> >> Tag: extended-description-is-probably-too-short >> Type: warning &g

one more

2008-09-04 Thread Raphael Geissert
Tag: maintainer-script-ignores-errors Type: warning Severity: normal Certainty: certain Ref: policy 3.9 Info: The maintainer script doesn't seem to set the -e flag which ensures that the script's execution is aborted when any executed command fails. -- Atomo64 - Raphael Please avoid sending m

two more tags

2008-09-04 Thread Raphael Geissert
Hi there, Attached are two mbox files each adding one of the following tags/checks: Tag: extended-description-is-probably-too-short Type: warning Severity: normal Certainty: certain Info: The extended description (the lines after the first line of the "Description:" field) is consists on just on

lintian_copyright-without-license-notice

2008-09-03 Thread Raphael Geissert
Hi all, I've just implemented another check, this time for the copyright file which basically tries to detect the ones that don't provide "any" licence information at all. I've tried to make it as less prone to FPs as possible and even ran lintian on over 16410 packages which are either availa

Bug#480939: check for brace bashism in debhelper config files

2008-09-02 Thread Raphael Geissert
tag 480939 patch thanks Hi Joey, 2008/9/2 Joey Hess <[EMAIL PROTECTED]>: > > It doesn't matter if bash or perl is providing this, it's still > completly undocumented and works only in some situations due to a detail > of implementation. You didn't answer my question, so I assume you still want

Bug#480939: check for brace bashism in debhelper config files

2008-09-01 Thread Raphael Geissert
On Monday 12 May 2008, Joey Hess wrote: > Package: lintian > Version: 1.23.48 > Severity: wishlist > > It seems that some people may decide that using brace expansion (ie, > docs/{foo,bar}) in a debian/package.docs file is a good idea. Despite it > not being documented anywhere, not being an intend

Re: yet another patch

2008-09-01 Thread Raphael Geissert
Frank Lichtenheld wrote: > On Mon, Sep 01, 2008 at 08:39:14PM -0500, Raphael Geissert wrote: >> Russ Allbery wrote: >> > Frank Lichtenheld <[EMAIL PROTECTED]> >> > writes: >> >> I'm not sure this check is worth including. I find even the >>

Bug#497347: lintian: warn about incomplete copyright sentences

2008-09-01 Thread Raphael Geissert
Russ Allbery wrote: > "Raphael Geissert" <[EMAIL PROTECTED]> > writes: > >> You said it: "at all"; in this case I propose to add a check for that >> specific bit of the copyright files. > > The problem here is that we encourage people to cut

Re: yet another patch

2008-09-01 Thread Raphael Geissert
Russ Allbery wrote: > Frank Lichtenheld <[EMAIL PROTECTED]> writes: > >> I'm not sure this check is worth including. I find even the aesthetical >> impact vey low. > > Yeah, I don't think it would do anything beyond just annoy people; there's > no actual problem here. It's just enforcing a codi

Bug#497347: lintian: warn about incomplete copyright sentences

2008-09-01 Thread Raphael Geissert
tag 497347 patch thanks 2008/9/1 Raphael Geissert <[EMAIL PROTECTED]>: > > Will submit a patch implementing the check in an hour or so, hope you like it. > Attached > > Cheers, Cheers, -- Atomo64 - Raphael Please avoid sending me Word, PowerPoint or Excel attachments. S

dots in command-with-path-in-maintainer-script

2008-09-01 Thread Raphael Geissert
Hi, Please apply attached patch to allow dots in paths found by command-with-path-in-maintainer-script so the python folks don't complain about /usr/bin/python2.5 being displayed as /usr/bin/python2 Cheers, -- Atomo64 - Raphael Please avoid sending me Word, PowerPoint or Excel attachments. See

Re: yet another patch

2008-09-01 Thread Raphael Geissert
Frank Lichtenheld wrote: > On Sun, Aug 31, 2008 at 09:05:48PM -0500, Raphael Geissert wrote: >> Tag: version-refers-to-distribution >> Type: warning >> Severity: minor >> Certainty: certain >> Info: The latest changelog version appears to mention the name o

Bug#497346: lintian should warn about references to symlinked common-licenses files

2008-09-01 Thread Raphael Geissert
clone 497346 -1 reassign -1 debian-policy blocks 497346 -1 thanks 2008/8/31 Russ Allbery <[EMAIL PROTECTED]>: > Raphael Geissert <[EMAIL PROTECTED]> writes: > >> The following snippet is pretty common and always, IMHO, wrong: >>> On Debian systems, the complete t

Bug#497347: lintian: warn about incomplete copyright sentences

2008-09-01 Thread Raphael Geissert
clone 497347 -1 reassign -1 dh-make 0.46 severity -1 normal thanks 2008/8/31 Russ Allbery <[EMAIL PROTECTED]>: > Raphael Geissert <[EMAIL PROTECTED]> writes: > >>> The Debian packaging is (C) 2008, John Doe <[EMAIL PROTECTED]> and >>> is licensed under t

Bug#497351: lintian doesn't emit new-package-should-close-itp-bug when bogus Closes is present

2008-08-31 Thread Raphael Geissert
Package: lintian Version: 1.24.4 > my $entry = $entries[0]; > if (@entries == 1 and $entry->Version =~ /-1$/) { > tag 'new-package-should-close-itp-bug' > unless @{ $entry->Closes }; > } The above code is fine, but fails to detect cases like this one: > $ dpkg-par

Bug#497350: lintian doesn't warn about the bogus (one number) Closes

2008-08-31 Thread Raphael Geissert
Package: lintian Version: 1.24.4 lintian doesn't emit wrong-bug-number-in-closes on the following case: > $ dpkg-parsechangelog > Source: foo > Version: 0.1-1 > Distribution: unstable > Urgency: low > Maintainer: John Doe <[EMAIL PROTECTED]> > Date: Fri, 18 Jul 2008 19:26:03 +0200 > Closes: 0 > C

Bug#497348: lintian: please warn about duplicated Section, Priority, Homepage, etc fields in control file

2008-08-31 Thread Raphael Geissert
Package: lintian Version: 1.24.4 Severity: wishlist Example follows. > Source: foo-bar > Section: x11 > Priority: optional > Maintainer: foo bar <[EMAIL PROTECTED]> > Standards-Version: 3.8.0 > Build-Depends: debhelper (>= 3) > Homepage: http://bar.moo > > Package: foo-bar > Architecture: all > S

Bug#497347: lintian: warn about incomplete copyright sentences

2008-08-31 Thread Raphael Geissert
Package: lintian Version: 1.24.4 Severity: wishlist > The Debian packaging is (C) 2008, John Doe <[EMAIL PROTECTED]> and > is licensed under the GPL, see above. The above example should either say: > ...is Copyright (C) 2008... The '(C)' part being optional, or > ...is © 2008... Cheers, -- Atom

Bug#497346: lintian should warn about references to symlinked common-licenses files

2008-08-31 Thread Raphael Geissert
Package: lintian Version: 1.24.4 Severity: wishlist The following snippet is pretty common and always, IMHO, wrong: > On Debian systems, the complete text of the GNU General > Public License can be found in `/usr/share/common-licenses/GPL'. It should either refer to GPL-2 or GPL-3. The same appli

Bug#497344: lintian: please warn about ${shlibs:Depends} when Architecture: all

2008-08-31 Thread Raphael Geissert
Package: lintian Version: 1.24.4 Severity: wishlist Self reminder. Cheers, -- Atomo64 - Raphael Please avoid sending me Word, PowerPoint or Excel attachments. See http://www.gnu.org/philosophy/no-word-attachments.html -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscr

yet another patch

2008-08-31 Thread Raphael Geissert
Hi all, Tag: version-refers-to-distribution Type: warning Severity: minor Certainty: certain Info: The latest changelog version appears to mention the name of one of the distributions. Unless this portion of the version is part of upstream it should be removed. This warning may also appear if

Re: more patches

2008-08-30 Thread Raphael Geissert
Frank Lichtenheld wrote: > On Sat, Aug 30, 2008 at 05:23:34PM -0500, Raphael Geissert wrote: >> Frank Lichtenheld wrote: >> > On Sat, Aug 30, 2008 at 03:52:51PM -0500, Raphael Geissert wrote: >> >> lintian_embedded-php-library: >> >> yet another PHP-rela

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

2008-08-30 Thread Raphael Geissert
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 JavaScript files, using such lib

Re: more patches

2008-08-30 Thread Raphael Geissert
W: djpig should-not-send-copy-of-message to atomo64 Frank Lichtenheld wrote: > On Sat, Aug 30, 2008 at 03:52:51PM -0500, Raphael Geissert wrote: >> lintian_embedded-php-library: >> yet another PHP-related check for embedded copies, none making part of >> PEAR, based on the

more patches

2008-08-30 Thread Raphael Geissert
Hi all, lintian_xinha: adds a, currently commented, entry for xinha, which is already packaged but not yet in the archive; so it can be kept as a reminder to enable it as soon as the package reaches the archive. lintian_embedded-php-library: yet another PHP-related check for embedded copies, none

Re: yet another patch

2008-08-30 Thread Raphael Geissert
W: lintian-maint-cced-mail-sender :) Frank Lichtenheld wrote: > On Fri, Aug 29, 2008 at 09:37:17PM -0500, Raphael Geissert wrote: >> Attached is a mbox for git-am making lintian check for embedded copies of >> tinymce and tinymce2. I also made a change so the package names are use

yet another patch

2008-08-29 Thread Raphael Geissert
Hi all, Attached is a mbox for git-am making lintian check for embedded copies of tinymce and tinymce2. I also made a change so the package names are used as regexes to ease the case of tinymce where both packages provide the same files but just different major versions. Cheers, -- Atomo64 - Rap

Re: More patches

2008-08-26 Thread Raphael Geissert
Russ Allbery wrote: > 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

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. N

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 di

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 fo

<    3   4   5   6   7   8   9   >