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

Bug#1013417: lintian: dkms updates



Control: tag -1 + confirmed

Hi Andreas,

Andreas Beckmann wrote:
> Package: lintian
> Version: 2.115.1
[…]
> I recently split a dh-dkms package from the dkms binary package.

I'm aware of that. It is partially already supprted in Lintian since
commit cfafa44256133c458011e3f13440de6538dd6045 which is part of the
2.115.0 release already:

https://salsa.debian.org/lintian/lintian/-/commit/cfafa44256133c458011e3f13440de6538dd6045#78b2b372e63b40b17bfc9c37d820cbf5b2ebb24c_103_108

I'm just not sure if such a hard cut is a good way to actually handle
the old and new variant for a transition period. Then again, lintian
is probably the last place which should care about old packages. :-)

> Please update lintian's suggestions for the respective B-D:
> 
> for 'dh --with dkms', there should be a B-D: dh-dkms (instead of
> B-D: dkms) for 'dh_dkms', there should be a B-D: dh-dkms (instead of
> B-D: dkms)

I thought that's already done by the change mentioned above, but then
again a quick git grep also found this as of the current git HEAD:

lib/Lintian/Check/Debhelper.pm:    dh_dkms => 'dkms:any | dh-sequence-dkms:any',

> or (preferably) B-D: dh-sequence-dkms (lintian already knows about that).

Looking for that is how I found the line above. :-)

> And the I'd like to have all *-dkms binary packages to have
>   Testsuite: autopkgtest-pkg-dkms
> on their source package.

That's on my TODO list to figure out how it actually works for my own
dkms-based package. :-)

> Not sure how to model that in lintian.

I think I can figure out. :-)

> Either on the source package: B-D: dh-dkms | B-D: dh-sequence-dkms
> or on the binary package relationships: Depends: dkms
> or on the binary package content: /usr/src/*/dkms.conf

Indeed, package name matches /-dkms$/ is probably not enough.

Will think about it.

Thanks!

		Regards, Axel
-- 
 ,''`.  |  Axel Beckert <abe@debian.org>, https://people.debian.org/~abe/
: :' :  |  Debian Developer, ftp.ch.debian.org Admin
`. `'   |  4096R: 2517 B724 C5F6 CA99 5329  6E61 2FF9 CD59 6126 16B5
  `-    |  1024D: F067 EA27 26B9 C3FC 1486  202E C09E 1D89 9593 0EDE


Reply to: