Bug#521257: Use of uninitialized value in bitwise or (|) at /usr/share/lintian/lib/Lintian/Command.pm line 308.

2009-03-27 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert > writes: > >> I found a difference on the results of the file-info parsing: >> >> diff --git a/t1 b/t2 >> index 14783a7..1e3fbb1 100644 >> --- a/t1 >> +++ b/t2 >> @@ -648,4 +648,4 @@ unit_tests/

Bug#521257: Use of uninitialized value in bitwise or (|) at /usr/share/lintian/lib/Lintian/Command.pm line 308.

2009-03-27 Thread Raphael Geissert
ease let me know if there is > anything else I could do to help debugging this. there might be several, unknown?, reasons why it can or can not be reproduced. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net file-info.mbox Description: application/mbox

Bug#521257: Use of uninitialized value in bitwise or (|) at /usr/share/lintian/lib/Lintian/Command.pm line 308.

2009-03-26 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: > >> --- a/lib/Lintian/Command.pm >> +++ b/lib/Lintian/Command.pm >> @@ -303,7 +303,7 @@ all the executions of kill_kill. >> =cut >> >> sub kill { >> -my $status; >> +my $status = 0; &g

Bug#521257: Use of uninitialized value in bitwise or (|) at /usr/share/lintian/lib/Lintian/Command.pm line 308.

2009-03-26 Thread Raphael Geissert
rther errors) > > I think the "commmand failed with error code 123" issue has been reported > before already, but the uninitialized value in line 308 seems new. The "command failed..." error was supposed to be fixed in 2.2.8, but looks like something is missing. Cheers, --

Re: Plans for squeeze

2009-03-25 Thread Raphael Geissert
Raphael Geissert wrote: > Russ Allbery wrote: > >> Raphael Geissert writes: >>> Russ Allbery wrote: >>>> I thought I responded to your patch with the specific details of what >>>> else needs to be done, and I hadn't seen a further response af

Re: Plans for squeeze

2009-03-24 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: >> Russ Allbery wrote: > >>> I'm not sure about this one, honestly. I think there's something to be >>> said about the simplicity of this mapping, and we do make the >>> additional informatio

Re: Plans for squeeze

2009-03-24 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: > >> Before I move into the main topic of this mail: is there any plan to >> participate in GSoC 2009? > > I don't have any time to be a mentor or to put together project ideas. I > think it would be great if some

Plans for squeeze

2009-03-23 Thread Raphael Geissert
s? suggestions? are those goals feasible? 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: Updating lintian.d.o

2009-03-17 Thread Raphael Geissert
Now that I think about it, the lab version should have been bumped when the changes to unpack-src-l1 were made :-/ Sorry for the mess. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to debian-lint-maint-requ...@lists.debian.org with

Updating lintian.d.o

2009-03-16 Thread Raphael Geissert
IMO should be "fixed" by replacing variables with glob masks 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: [SCM] Debian package checker branch, master, updated. 2.2.6-92-gd416c9d

2009-03-08 Thread Raphael Geissert
ould have been run before the upload. 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: [SCM] Debian package checker branch, master, updated. 2.2.6-82-g8478b06

2009-03-08 Thread Raphael Geissert
Russ Allbery wrote: > > debhelper compatibility level V5 is now deprecated s/V5/V4 The code change is fine, the changelog and commit entries are not :) Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to debian-lint-main

Documentation update

2009-03-08 Thread Raphael Geissert
Hi, Looking around at the different documentation files I noticed there were some bits missing. Attached mbox should bring them up to date. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian-doc.mbox Description: application/mbox

debhelper 7.2.3-related changes

2009-03-08 Thread Raphael Geissert
are asking on #-devel and #-mentors what to do due to the install* checks so I think they should be addressed ASAP. 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 "uns

Re: Parallel unpack

2009-03-08 Thread Raphael Geissert
nce a single write. I've tried to reproduce that bug with a test with over one thousand files but only managed to reproduce it a couple of times. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to debian-lint-maint-requ...@list

Bug#518873: [checks/{files,changelog-file}] correctly handle .gz but not gzip files

2009-03-08 Thread Raphael Geissert
Package: lintian Version: 2.2.6 Severity: minor Tags: patch Hi, Here are some of the findings while trying to test for the error messages of failing collection scripts. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian-gzip.mbox Description: application

Re: Bug#517650: lintian: changes made to collection scripts are not reflected unless a package changes

2009-03-08 Thread Raphael Geissert
t and found some bugs while at it, will file the reports in a moment. But error reporting seems to be working fine: warning: collect info strings about package binaries-embedded-zlib failed warning: skipping check of binary package binaries-embedded-zlib (added an exit 1 on collection/scripts) Che

Re: Parallel unpack

2009-03-08 Thread Raphael Geissert
t; I missed that one; surprisingly enough it anyway passed the whole test suite. 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: couple of patches

2009-03-08 Thread Raphael Geissert
wanted to do for a while. Then we won't need > those extra use libs that are easy to forget and will be closer to having > a real module structure. > Sure. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to d

Re: Bug#516687: lintian: copyright-should-refer-to-common-license-file-for-gfdl should check for version

2009-03-08 Thread Raphael Geissert
;> || m/GNU GENERAL PUBLIC LICENSE\s+Version 1/ >> || m/compatible\s+with\s+(?:the\s+)?(?:GNU\s+)?GPL/ > > The real problem is that the existing regex didn't match this particular > reference. I'm taking a look now. > OK :) Cheers, -- Raphael Geissert - Deb

Re: Bug#517650: lintian: changes made to collection scripts are not reflected unless a package changes

2009-03-08 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: > >> # info already available? >> -next if (-e "$base/$ci->{'output'}"); >> +next if (-l "$base/.${coll}$ci->{'version'}"); >> +system('sh&

Bug#516687: lintian: copyright-should-refer-to-common-license-file-for-gfdl should check for version

2009-03-04 Thread Raphael Geissert
tag 516687 patch thanks Attached mbox should do it. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian-gfdl.mbox Description: application/mbox

Bug#517650: lintian: changes made to collection scripts are not reflected unless a package changes

2009-03-04 Thread Raphael Geissert
the collection scripts based on their deps. This would also allow us to start moving bits out of unpack/unpack-* and in to collection/ scripts. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian-fine-grained_collection.mbox Description: application/mbox

Bug#517778: lintian: binaries-from-other-arch test fails in testing

2009-03-01 Thread Raphael Geissert
Package: lintian Version: 2.2.6 Tags: patch Hi, While making some changes I ran the whole test suite and failed on the binaries-from-other-arch test because of some changes now in testing's objdump. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net li

Bug#517650: lintian: changes made to collection scripts are not reflected unless a package changes

2009-03-01 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: > >> When a collection script is modified, the changes are not reflected even >> if the same package is re-unpacked or re-processed. This could lead to >> certain temporary miss behaviours. > > That's why you

Re: couple of patches

2009-03-01 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: > >> None of these require my previous changes to be applied, but would be >> great if the others could be applied as well. > > I applied the first patch. For the move of common_data, I agree that this > is an improve

Re: Couple of patches

2009-03-01 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: [...] > >> Do you handle that one? > > Not at the moment. I have lots of other things I want to work on first. > Implemented, see attached mbox. [...] > > The proof-of-concept looks like a reasonable approach. I had

Bug#517650: lintian: changes made to collection scripts are not reflected unless a package changes

2009-02-28 Thread Raphael Geissert
touch the .$collection_script$version file. * Skip a collection script if the .$collection_script$version file exists. If it is fine I'll implement it. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to debian-lint-maint-requ...@l

Bug#517191: Sorry for false submission

2009-02-26 Thread Raphael Geissert
st nor in the BTS, so attaching it now/again. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian_necf.mbox Description: application/mbox

Bug#514495: [lib/Spelling.pm] check the spelling of large texts in a more efficient way

2009-02-24 Thread Raphael Geissert
I'm afraid your message didn't make it to the newsgroup provided by gmane; so it is until now that I'm reading your email. On Sunday 08 February 2009 21:11:20 Russ Allbery wrote: > Raphael Geissert writes: > > There might indeed be a degradation on those texts where the

Bug#516530: [unpack,reporting] support multiple sections

2009-02-23 Thread Raphael Geissert
s file from and encode that information into the > index. > Ok; attached mbox contains it all. Please beware that I haven't tested the latest change (I don't have a local mirror, and my personal repository is partially broken). Cheers, -- Raphael Geissert - Debian Maintainer

jslib false negative patch

2009-02-23 Thread Raphael Geissert
Hi, As just pointed out by pabs on IRC. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian_js.mbox Description: application/mbox

Bug#516530: [unpack,reporting] support multiple sections

2009-02-23 Thread Raphael Geissert
On what indices should the area be specified? would bin and udebs be more than enough? 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". Troubl

Bug#302196: [new check] broken libtool .la files shipping in packages

2009-02-22 Thread Raphael Geissert
ouch). Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian_libtool.mbox Description: application/mbox

Bug#516530: [unpack,reporting] support multiple sections

2009-02-22 Thread Raphael Geissert
this a > section or category to call it an archive area instead, which is the > terminology used by the Social Contract and in the next version of Policy, > but that can be done later.) > Cloned :) Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net Bill W

Bug#516530: [unpack,reporting] support multiple sections

2009-02-22 Thread Raphael Geissert
tag 516530 patch thanks Attached mbox should do it. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian_sections.mbox Description: application/mbox

Bug#318104: [checks/binaries] check for embedded copies of zlib' from `lintian: warn if package includes a private copy of zlib

2009-02-22 Thread Raphael Geissert
tag 318104 patch thanks Attached mbox contains the necessary changes. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian_zlib.mbox Description: application/mbox

Re: [SCM] Debian package checker branch, master, updated. 2.2.5-61-g98be8ca

2009-02-21 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert > writes: > >>> +exists($archselftable{$arch}) or warn("Unknown architecture $arch"); >>> + >> >> argh; there should be two exceptions: all and lpia. > > Oh, thanks. I just got all sepearately. Sho

Re: [SCM] Debian package checker branch, master, updated. 2.2.5-61-g98be8ca

2009-02-21 Thread Raphael Geissert
[...] > > +exists($archselftable{$arch}) or warn("Unknown architecture $arch"); > + argh; there should be two exceptions: all and lpia. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to debian-lint-maint-requ.

Bug#468147: Check for correct lsb-base dependency for init script functions

2009-02-21 Thread Raphael Geissert
Hi everyone, Now that lenny is out even oldstable (i.e. etch) has an lsb-base > 3.0-6 Am I right to assume that this bug should therefor be closed? Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to debian-lint-maint-r

Bug#515689: duplicate-font-file X also in ttf-aenigma, bad advice?

2009-02-21 Thread Raphael Geissert
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

Bug#516221: Checks for dpkg-divert

2009-02-21 Thread Raphael Geissert
t;Internal error: \$mode has unknown value: $mode"; replace die with fail, and use Utils (fail); Once the check is merged in 'scripts' [^#] won't be needed anymore. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to debi

couple of patches

2009-02-21 Thread Raphael Geissert
Hi all, None of these require my previous changes to be applied, but would be great if the others could be applied as well. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian_profil3.mbox Description: application/mbox lintian_reorg.mbox Description

Bug#516067: lintian: false positives when checking the same package version from different archs

2009-02-21 Thread Raphael Geissert
Hi Frank, Nice to see you around :) 2009/2/21 Frank Lichtenheld : [...] > > lintian -v might be helpful. > -v isn't helpful at all, only -d is a bit more, but the issue is a bit obscure. Attached is the output of -d on just a subset of the packages. Cheers, -- Raphael Gei

Re: Couple of patches

2009-02-18 Thread Raphael Geissert
Some other optimisations and a new check/bug fix, found by somebody in #debian-mentors Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian-copyright_license_typo.mbox Description: application/mbox lintian-data_optim.mbox Description: application/mbox

Bug#513996: [checks/binaries] detect binaries from other architectures

2009-02-18 Thread Raphael Geissert
merge 236534 513996 tag 513996 patch thanks On Tuesday 03 February 2009 02:15:15 Russ Allbery wrote: > Raphael Geissert writes: > > While working on the check for prebuilt binaries in source tarballs I > > was thinking about making it severity: minor as having prebuilt binaries

Bug#516067: lintian: false positives when checking the same package version from different archs

2009-02-18 Thread Raphael Geissert
-dependency-on-libc needed by ./usr/bin/kgb There are 16 debs in deb/ one has a different version than the others. 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 "unsubs

Bug#515689: duplicate-font-file X also in ttf-aenigma, bad advice?

2009-02-17 Thread Raphael Geissert
ather add an override > instead? > > 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

Bug#515068: orig.tar.{bz2,lzma} support

2009-02-13 Thread Raphael Geissert
That reminds me of a change I saw it was needed the other day. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian-lzma.mbox Description: application/mbox

Bug#514951: [checks/binaries] check the output of strings for typos and mistakes

2009-02-13 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: >> Russ Allbery wrote: > >>> You have to run strings -a if you're going to implement that, at which >>> point I think chances are pretty high that you're going to get false >>> positives from the

Re: Couple of patches

2009-02-13 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: > >> At least the collection scripts could be run in parallel. All the >> collection scripts with the same Order should run in parallel just fine, >> and once the dependencies are done or all the scripts of the same Order &

Bug#514951: [checks/binaries] check the output of strings for typos and mistakes

2009-02-13 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: > >> Attached mbox contains patches adding a collection script to gather the >> output of the 'strings' command on the ELF binaries of a package, and >> spell checks its output. The collection script is also ne

Bug#515137: lintian: source-contains-prebuilt-binary false positive on files in test suite

2009-02-13 Thread Raphael Geissert
t just "looks like" one, but anything under tests.* could indeed be ignored. I'll leave that up the maintainers :) 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

Bug#515068: orig.tar.{bz2,lzma} support

2009-02-13 Thread Raphael Geissert
ase of gzip nor bzip2 there is no reason to do it for lzma. As I'm not a lintian maintainer I'm not marking this bug as blocked by #347636 (bug report against tar asking for lzma auto detection, CC'ed), as there is another solution: manually set --lzma if needed. Cheers, -- Raphael G

Bug#514951: [checks/binaries] check the output of strings for typos and mistakes

2009-02-11 Thread Raphael Geissert
elf files without having to look for them in the file-info index. Maybe it should die if nobody finds a use for it. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian-binaries_spelling.mbox Description: application/mbox signature.asc Description: This is a

Bug#509624: [checks/files] check icons for an incorrect size

2009-02-11 Thread Raphael Geissert
tag 509624 patch thanks Whatever the final severity is, attached patch implements it. Right now I'm lacking good arguments for the tag description, so it needs to be improved. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian-icons.mbox Descri

Bug#474590: [frontend] add a parsable reason for overrides and optionally require it

2009-02-11 Thread Raphael Geissert
"C"? another possibility would be to print them as "N" with some sort of format that harness could identify and parse. Suggestions? Comments? Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net signature.asc Description: This is a digitally signed message part.

Re: Couple of patches

2009-02-11 Thread Raphael Geissert
ating Tag and its second read of all the tag metadata by merging it > into the new Lintian::Tag::Info module. I'm slowly working on that. > Sure. I already tried to remove the add_tag and display_tags from frontend/lintian; but I ended up with something useless. Cheers, -- Raphael Ge

Bug#509624: lintian should check for wrong image size

2009-02-11 Thread Raphael Geissert
normal fit better? 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: Couple of patches

2009-02-11 Thread Raphael Geissert
Raphael Geissert wrote: [...] > I just started to optimise the code, reduce number of calls, etc based on > the profiling data (by moving some code I reduced over 2k function calls). > Will send some patches tomorrow. > Attached is the first set. We could speed up lintian a

Re: Couple of patches

2009-02-10 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: >> Russ Allbery wrote: [...] >> I remember seeing the other day some unneeded files being generated at >> unpack level 1 of binary packages. > > More detail would be good. The only things that level 1 unpack of binary &

Bug#514697: [lib/Lintian/Collect/Binary.pm] missing "use Lintian::Colection"

2009-02-10 Thread Raphael Geissert
/Lintian/Collect/Binary.pm @@ -26,6 +26,7 @@ use base 'Lintian::Collect'; use Carp qw(croak); use Util; +use Lintian::Relation; # Initialize a new binary package collect object. Takes the package name, # which is currently unused. Cheers, -- Raphael Geissert - Debian Maintai

Bug#514495: [lib/Spelling.pm] check the spelling of large texts in a more efficient way

2009-02-09 Thread Raphael Geissert
Raphael Geissert wrote: > Russ Allbery wrote: > [...] >> >> Have you benchmarked this? My intuition says that if this makes any >> difference at all, it will be a performance *degredation*. You're now >> walking the entire text for every typo we know abo

Bug#514497: lintian doesn't know about many virtual packages

2009-02-08 Thread Raphael Geissert
Attached mbox should do it. I had some problems with perl not liking 'unless ($seen{$pkg} lt 2 or ...', but I used a workaround. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian-virtual_packages.mbox Description: application/mbox

Bug#514495: [lib/Spelling.pm] check the spelling of large texts in a more efficient way

2009-02-07 Thread Raphael Geissert
.dfsg.1-1+lenny2_i386.deb [...] real0m6.311s user0m5.456s sys 0m0.600s (xlintian is an alias to lintian from my git repo). As you can see with the patch it is faster. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to debian

Bug#127494: [checks/cruft] precompiled binaries in the source

2009-02-07 Thread Raphael Geissert
tag 127494 patch thanks Attached mbox implements both parts. Maybe the check should later be extended to look for stuff other than ELF an PE objects. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian-src_binaries.mbox Description: application/mbox

Bug#514497: lintian doesn't know about many virtual packages

2009-02-07 Thread Raphael Geissert
archive, and the patch in the attached mbox does exactly that. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian-virtual_packages.mbox Description: application/mbox

Bug#514496: [t/runtests] some test fail due to rsync's time and size check -based behaviour

2009-02-07 Thread Raphael Geissert
Package: lintian Version: 2.2.1 Severity: normal Tags: patch Hi, This bug gave me something to do for over an hour, when change "caused" the failure of some unrelated tests. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian-rsync_fix.mbox D

Bug#514495: [lib/Spelling.pm] check the spelling of large texts in a more efficient way

2009-02-07 Thread Raphael Geissert
ssions and other operations so that > they are performed once for all the text, instead of doing it once on every > word. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian-spelling_optim.mbox Description: application/mbox

Another spelling patch

2009-02-07 Thread Raphael Geissert
Hi, Some more typos. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian-spelling.mbox Description: application/mbox

Bug#513053: [checks/descriptions] warn when first person is used in descriptions

2009-02-07 Thread Raphael Geissert
> I don't think that using "You" in a description is ok (not even lowercase "you", but that would be too picky); but I won't object to try to reach a concensus about it. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -

Bug#513053: [checks/descriptions] warn when first person is used in descriptions

2009-02-07 Thread Raphael Geissert
clone 513053 -1 retitle 513053 [checks/descriptions] include first-word-of-sentences-too-repetitive tag 513053 moreinfo thanks Russ Allbery wrote: > Raphael Geissert writes: > >> I originally used severity: normal, but later was unsure about the >> severity because if I fi

Re: DRAFT: Bits from the Lintian maintainers

2009-02-07 Thread Raphael Geissert
or less lintian clean (say less than 10 tags) should try --pedantic at least once every now and then to see what it reports and think about those issues. "You decide, but take a look anyway" would be the right slogan. [...] > > Upcoming Work > = > [...] > R

Re: Couple of patches

2009-02-07 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: > >> Attached are two mboxs, one adds a some more words to Spelling.pm; > > Applied. Thanks > >> the other mbox contains the necessary changes to generate an index of >> the source package, and file-index. >

Bug#514095: lintian: Should not report for duplicate files|fonts in binary packages when one of the packages is a udeb

2009-02-04 Thread Raphael Geissert
something when I wrote the check. > > I think that, if feasible, this test should not compare the content of > udeb packages with the content of deb packages. > Attached mbox contains the patch. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net

Couple of patches

2009-02-03 Thread Raphael Geissert
think about how to be friendly to the kernel's file caching, to speed lintian up. Because the more checks are added, the more info is collected, the more it slows down. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian_spelling.mbox Description: applic

Re: error messages in the raw lintian.log

2009-02-03 Thread Raphael Geissert
f topic, shouldn't the source-contains-* checks better be severity: pedantic? There's no much maintainers can do about those but repack the tarball and/or bug upstream, not to mention that they usually don't affect the final .deb and when they do another check should catch

Re: error messages in the raw lintian.log

2009-02-03 Thread Raphael Geissert
didn't see the tar ones. I don't know how we could suppress those, maybe a bug report against those packages is the only option, as tar doesn't seem to have a "ultra-quiet" option (and even if it had, it doesn't sound like the best solution here). Cheers, -- Raphael G

Bug#513996: [checks/binaries] detect binaries from other architectures

2009-02-02 Thread Raphael Geissert
arch directories should be ignored, but I can't think about any good reason to for example ship an i386 binary in an armel package. Oh, and by implementing such a check I would not need to rise the severity of the other check, not to mention that it isn't the right thing to do :). Chee

Bug#513946: lintian: should not tag copyright-refers-to-versionless-license-file when using 'GPL-2+'

2009-02-02 Thread Raphael Geissert
[No need to CC me] 2009/2/2 Evgeni Golov : > On Mon, 02 Feb 2009 12:26:37 -0600 Raphael Geissert wrote: > >> > >> > I suspect there's still arguably a bug in such a package, since this would >> > imply that the upstream license statement wasn't cut and

Bug#513946: lintian: should not tag copyright-refers-to-versionless-license-file when using 'GPL-2+'

2009-02-02 Thread Raphael Geissert
his tag isn't the > right one to apply. So this patch looks good to me. I'll go ahead and > apply it. > Yes, and a new check should be implemented. 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

Bug#513766: lintian should warn when it failed to parse the changelog

2009-01-31 Thread Raphael Geissert
ad side-effects when checking the source package. Well, I guess this bug should then be closed, don't you think? 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 "

Bug#513767: [checks/debhelper,checks/standards-version] lintian doesn't correctly handle NaN values

2009-01-31 Thread Raphael Geissert
Package: lintian Version: 2.2.0 Severity: normal Tags: patch Attached is an mbox containing several bug fixes, found thanks to lintian.log Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian_lintian.d.o_bugfixes.mbox Description: application/mbox

Bug#513766: lintian should warn when it failed to parse the changelog

2009-01-31 Thread Raphael Geissert
that you expected to produce > errors which didn't? Yes, attached changelog files is the culprit. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net batv-milter (0.3.0.dfsg-2) unstable; urgency=high * Build depend on m4 (FTBFS)Closes: #505755 -- Rich

Bug#513766: lintian should warn when it failed to parse the changelog

2009-01-31 Thread Raphael Geissert
'debfiles/changelog', '5', 'couldn\'t parse date Mon, 29 Dec 2008 85:45:00 -' ]] }, 'data' => [ bless( { [...] 'Date' => 'Mon, 29 Dec 2008 85:45:00 -',

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

2009-01-31 Thread Raphael Geissert
esent, I think we should still warn with severity: pedantic, since I > think the user should still be pointing to the version of the GPL stated > in the license, not some random later version. Attached mbox implements the checks. I checked the regex against lots of the installed packages a

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

2009-01-31 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: > >> So, what's the way to go here? > >> severity:minor when no "or any later version" is used? and >> severity:pedantic when it is? >> Or just the former with the severity of the latter? > >>

Re: [SCM] Debian package checker branch, master, updated. 2.2.0-29-gc2ef3fd

2009-01-31 Thread Raphael Geissert
nts-i386.gz \ > -| perl -n -w -E 'print lc $_ if > (s#^.+/([\w-]+\.(?:[to]tf|pfb|pcf))\s+\w+/([to]tf.+)$#$1=$2#i);' \ > -| sort > fonts > - > -mv fonts "$lintian_data/files/" > +| perl -n -w -E 'print lc $_ if > (s,^.+/([\w-]+\.(?:

Bug#127494: [checks/cruft] precompiled binaries in the source

2009-01-31 Thread Raphael Geissert
I'm too sleepy or there's no way to do it via the standard options of ls or find. The file index is necessary for #512265 and #472219 too. Any hints? Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to debian-lint-maint-requ

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

2009-01-31 Thread Raphael Geissert
On Sunday 31 August 2008 23:20:32 Russ Allbery wrote: > Raphael Geissert writes: > > 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-licen

Bug#513650: lintian: pedantic: check for fonts in non-font package

2009-01-31 Thread Raphael Geissert
ompromise between accuracy (or rather number of false negatives) and speed. 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: Bug#512265: retitle 512265 to [checks/rules] warn if a test suite exists but is not run

2009-01-30 Thread Raphael Geissert
It only sort of belongs in cruft; it's > different than what the rest of that check does. That sounds like a good idea :). Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net -- To UNSUBSCRIBE, email to debian-lint-maint-requ...@lists.debian.org with a

Bug#512265: retitle 512265 to [checks/rules] warn if a test suite exists but is not run

2009-01-30 Thread Raphael Geissert
te. What do you think? Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net signature.asc Description: This is a digitally signed message part.

Bug#513423: lintian: false positive: libmx4j-java source: missing-build-dependency-for-dh_-command

2009-01-28 Thread Raphael Geissert
iner I'm leaving the report open, although IMO there's nothing to change here (and the tag description sounds fine). 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

Bug#513336: lintian: dbg-package-missing-depends should work on the source package

2009-01-28 Thread Raphael Geissert
ertainty and reduces the number of false > positives by not expecting X from X-dbg to actually exist. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian_dbg-rework.mbox Description: application/mbox signature.asc Description: This is a digitally signed message part.

Re: Starting a new archive-wide run

2009-01-27 Thread Raphael Geissert
Raphael Geissert wrote: > > Theory: > mkdir bin > ln -s /bin/true bin/gpg > PATH="bin:$PATH" Implemented in the patch included in the mbox. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net harness_gpg.mbox Description: application/mbox

Re: Starting a new archive-wide run

2009-01-26 Thread Raphael Geissert
Russ Allbery wrote: > Raphael Geissert writes: > >> On a partially unrelated topic, what about disabling the pgp/gpg >> signature check on lintian.d.o? it just consumes time and space in >> lintian.log and won't find anything without the keyrings and gpg setup. &g

Re: Starting a new archive-wide run

2009-01-26 Thread Raphael Geissert
d anything without the keyrings and gpg setup. 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

Bug#513053: [checks/descriptions] warn when first person is used in descriptions

2009-01-25 Thread Raphael Geissert
a more wide diversity of words. > In case you have troubles with it an email to the english i18n mailing > list should help. Cheers, -- Raphael Geissert - Debian Maintainer www.debian.org - get.debian.net lintian_descriptions.mbox Description: application/mbox

Re: Starting a new archive-wide run

2009-01-25 Thread Raphael Geissert
t's doing those. I just took a quick look at the source-related checks and didn't find much, just some unused global variables and an unused pm. The attached mbox contains the patches reflecting the cleanup. Cheers, -- Raphael Geissert - Debian Maintai

<    2   3   4   5   6   7   8   9   >