Bug#731248: please add a check to check for the wrong --with autotools-dev

2013-12-03 Thread Joey Hess
Henrique de Moraes Holschuh wrote: (cc'ing the debhelper maintaner to get an authoritative answer about whether a dh module can have a - in the name or not). It's a perl module, and AFAIK you cannot have a dash in the name of a perl module. joey@darkstar:~perl -e 'use Foo::Bar-Baz' Can't

Bug#731248: please add a check to check for the wrong --with autotools-dev

2013-12-03 Thread Joey Hess
Matthias Klose wrote: Package: lintian Seen now in many packages, calling dh --with autotools-dev instead of autotools_dev, making this a nop. Actually, dh --with nonexistantthing is not a noop, it causes dh to fail. --with autotools-dev works because dh does a $mod=~s/-/_/g; --

Bug#672664: lintian: Please add a check on udeb non-emptiness

2012-05-12 Thread Joey Hess
Cyril Brulebois wrote: (There might legitimate cases where udebs ship no contents, and just contain maintainer scripts/debconf stuff, so I'm cc-ing debian-boot@ for input.) There are several such udebs. iso-scan is one. -- see shy jo signature.asc Description: Digital signature

Bug#664211: add check for wrong dependency on debconf

2012-03-16 Thread Joey Hess
Package: lintian Version: 2.5.5 Severity: normal cdebconf has gotten to be a quite usable replacement for debconf. However, in trying to remove debconf from my system, I found there were several packages with wrong dependencies, like this: Depends: debconf (= 0.5) | debconf-2.0, debconf That

Bug#664211: add check for wrong dependency on debconf

2012-03-16 Thread Joey Hess
Russ Allbery wrote: But, on this topic, when I looked at removing debconf from my system, the thing that seemed to be missing was debconf-set-selections. Does cdebconf provide that utility in some way that I'm not noticing? cdebconf depends on debconf to allow them to be co-installed for

Bug#656885: lintian: Separate tag for no d/compat

2012-01-22 Thread Joey Hess
Niels Thykier wrote: I noticed that you are making d/compat the only source of a debhelper compat[1]. Would you like us to add a tag for missing compat file (with use of debhelper)? Absolutely. There are potentially some packages that use DH_COMPAT to set their compat level, rather than

Bug#651572: check for executable, non-hashbanged files in debian/

2011-12-09 Thread Joey Hess
Package: lintian Version: 2.5.4 Severity: wishlist Surprisingly, many packages seem to have randomly executable files in the source package, like debian/changelog and debian/copyright. A lintian check seems like a good idea. -- see shy jo signature.asc Description: Digital signature

Bug#634112: debhelper deprecations

2011-07-16 Thread Joey Hess
Package: lintian Version: 2.5.1 Severity: normal Thank you for maintaining so much debhelper deprecation checks! I compared the lintian checks with debhelper's TODO list, and dh_installmanpages is deprecated but not shown by lintian. Also, debian/compress files are deprecated for a while. But

sr@latin locale debconf pre-depends issue

2010-11-20 Thread Joey Hess
As seen in #604153, debconf templates files with fields using a l...@modifier locale such as s...@latin will fail to work with debconf before 1.5.34. This needs to be fixed by the affected packages Pre-Depending (not Depending) on debconf (= 1.5.34) Only tasksel and keyboard-configuration are

Bug#588156: lintian: [checks/debhelper] logic doesn't work for dh = 7.9.1 and compat level 8

2010-07-05 Thread Joey Hess
gregor herrmann wrote: Right, I'm also not really sure about the status of compat level 8. Cc'ing Joey to get some input. Using compat level 8 before debhelper version 8.0 leaves a package open to breaking when I make more changes to that compat level. -- see shy jo signature.asc

Bug#576014: lintian: warn on files in /usr/lib/debug/usr/lib/pyshared/pythonX.Y

2010-03-31 Thread Joey Hess
Josselin Mouette wrote: Many python-*-dbg packages using python-support ship files in /usr/lib/debug/usr/lib/pyshared/pythonX.Y/, likely as a result of dh_strip'ping files in /usr/lib/pyshared/pythonX.Y/. However, this is wrong, as gdb will never look into that directory;

Re: DRAFT: Bits from the Lintian maintainers

2009-12-31 Thread Joey Hess
Russ Allbery wrote: debhelper and didn't have this dependency will now get a warning. We're currently deciding whether to teach Lintian that ${misc:Depends} isn't needed in this specific case or to just uniformly recommend the ${misc:Depends} dependency for all packages using debhelper. The

Re: Processed: Re: Bug#521960: lintian: desktop-mimetype-without-update-call false positive with CDBS

2009-03-31 Thread Joey Hess
Marc, your PATH is broken. It contains literally '~/bin' -- not /home/duck/bin. That will cause the shell to look in ./~bin/ in the current directory, which could even be a security risk. Just thought you should know that. j...@gnu:~PATH=~/bin:/usr/bin:/bin find ~/tmp/a -type f -execdir echo {}

Bug#517629: bogus can't break line warnings when run in narrow terminal

2009-02-28 Thread Joey Hess
Package: lintian Version: 2.2.6 Severity: normal j...@gnu:~/src/debhelperecho $COLUMNS 65 j...@gnu:~/src/debhelperlintian ../debhelper_7.0.52_all.deb W: debhelper: manpage-has-errors-from-man usr/share/man/man1/dh.1.gz 200: warning [p 2, 3.2i, div `an-div', 0.0i]: can't break line

Bug#515601: lintian: false positive changelog-file-not-compressed if corresponding file is small

2009-02-16 Thread Joey Hess
Russ Allbery wrote: I believe that if you install them with dh_installchangelogs, they'll be compressed. I think that's the way you're supposed to install the upstream changelog. No matter how the changelog is installed, dh_compress is the *only* thing in debhhelper that will compress it, and

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

2008-09-02 Thread Joey Hess
Raphael Geissert wrote: 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 intended syntax, working only by accident in certian configurations, and of

Re: Ideas wrt debconf templates checks

2008-07-05 Thread Joey Hess
Adam D. Barratt wrote: As far as I can see, such a test does not currently exist. It would be fairly simple to add, assuming that the following methodology would make sense (on a per-binary package basis): for each of config, preinst, postinst, prerm, postrm find uses of db_input,

Bug#484549: lintian: Check that patches in debian/patches/ do not modify files in the debian directory

2008-06-04 Thread Joey Hess
Raphael Hertzog wrote: This is always wrong as the files in the debian directory are provided by Debian. Unless they're provided by upstream... Furthermore those packages do not work with the 3.0 (quilt) source format as it needs to generate a diff between the current source directory and

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

2008-05-12 Thread Joey Hess
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 intended syntax, working only by accident in certian

Bug#477628: support debian/rules files using the dh utility

2008-04-23 Thread Joey Hess
Package: lintian Version: 1.23.46 Severity: normal As of debhelper version 7, there is a dh utility that runs a bunch of debhelper commands in sequence. Use of this breaks various lintian tests that check for certian dh_ commands being in the rules file. You can probably just check for ^\tdh and

Bug#473840: please stop warning about the triggers control file

2008-04-01 Thread Joey Hess
Package: lintian Version: 1.23.46 Severity: normal W: initramfs-tools: unknown-control-file triggers This is a new official control file used by dpkg triggers. -- System Information: Debian Release: lenny/sid APT prefers unstable APT policy: (500, 'unstable'), (500, 'stable'), (1,

Bug#469409: claims copyright-without-copyright-notice for valid copyright file

2008-03-04 Thread Joey Hess
Package: lintian Version: 1.23.45 Severity: normal W: sleepd: copyright-without-copyright-notice sleepd is copyright 2000-2008 by Joey Hess [EMAIL PROTECTED] under the terms of the GNU GPL, version 2 or higher. On Debian systems, the full text of the GPL is in /usr/share/common-licenses

Bug#452216: W: libaa1: unknown-control-file symbols

2007-11-20 Thread Joey Hess
Package: lintian Version: 1.23.36 Severity: normal The symbols control file is a dpkg-gensymbols file. (For bonus points, sanity check the file. ;-) -- System Information: Debian Release: lenny/sid APT prefers unstable APT policy: (500, 'unstable'), (500, 'testing'), (1, 'experimental')

Bug#358523: please add check for broken invoke-rc.d calls caused by debhelper bug

2006-03-22 Thread Joey Hess
Jeroen van Wolffelaar wrote: Certianly, especially in this case. Can you maybe mention two examples in current unstable, one being ok, one not being ok, but still having a similar (but correct) snippet? not ok: dbus 0.61-4 ok: sleepd 1.3.4 (in Incoming) -- see shy jo signature.asc

Bug#314680: /usr/share/menu

2005-06-17 Thread Joey Hess
Package: lintian Version: 1.23.8 Severity: normal W: bsdgames: postrm-has-useless-call-to-update-menus N: N: The postrm script calls the update-menus program though no file is N: installed in /usr/lib/menu or /etc/menu-methods This is incorrect; I use /usr/share/menu. -- System Information:

Bug#263377: usr-doc-symlink-without-dependency broken

2004-08-03 Thread Joey Hess
Package: lintian Version: 1.23.2 Severity: normal [EMAIL PROTECTED]:~packagedpkg --info aalib1-dev_1.4p5-21_i386.deb|grep Depends Depends: aalib1 (= 1.2-18), libx11-dev, slang1-dev ( 1.3.0-0), libncurses5-dev [EMAIL PROTECTED]:~packagelintian aalib1-dev_1.4p5-21_i386.deb E: aalib1-dev:

Bug#234608: does not know about title debconf template type

2004-02-24 Thread Joey Hess
Package: lintian Version: 1.22.10 Severity: normal E: base-config: unknown-template-type title For a while debconf has supported the title type used by the SETTITLE command. -- System Information: Found unknown policy: ('1', 'pool')Debian Release: testing/unstable APT prefers unstable APT