[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: [PATCH] Nuke a few easily Lintian warnings



On Tue, Aug 17, 2010 at 01:04:51PM +0100, Ben Hutchings wrote:
> > empty-binary-package

> >         Resolved by adding the word virtual to the relevant package
> >         descriptions.

> I prefer 'metapackage'.  And I think that should go in the short
> description (as in the packages generated by linux-latest-2.6).

Sensible, since 'metapackage' is the correct word for it. :-)

> > After this patch it looks from
> > http://lintian.debian.org/maintainer/debian-kernel@lists.debian.org.html
> > like the remaining Lintian warnings would be (I only considered the
> > linux-2.6 source package):

> > linux-base - no-debconf-config
> >         (should be linux-base.config not linux-base.postinst?)

> We cannot make this a config script because it requires external tools
> just to work out whether it should ask any questions, and a config
> script may be run before the package dependencies are satisfied.  This
> warning should be overridden.

Which tools are these?  It's certainly possible to write a .config script
that prompts for debconf questions opportunistically, and the .config script
will always be run again at postinst time to ask any questions that it
didn't know to ask during apt pre-configuration.  This generally gives a
(slighty) better user experience overall, since users who already have the
requisite external tools installed then get the debconf prompts in a batch
up front.

Fixing this is probably not going to be high on anyone's priority list, but
I would argue against a lintian override in this case because I think
there's real room for improvement.

Cheers,
-- 
Steve Langasek                   Give me a lever long enough and a Free OS
Debian Developer                   to set it on, and I can move the world.
Ubuntu Developer                                    http://www.debian.org/
slangasek@ubuntu.com                                     vorlon@debian.org

Attachment: signature.asc
Description: Digital signature


Reply to: