[Unstable][PATCH] UBUNTU: [Packaging] Add module versions to the ABI

Juerg Haefliger juerg.haefliger at canonical.com
Tue Apr 20 10:38:03 UTC 2021


On Mon, 19 Apr 2021 15:25:56 -0500
Seth Forshee <seth.forshee at canonical.com> wrote:

> On Tue, Apr 13, 2021 at 02:36:38PM +0200, Juerg Haefliger wrote:
> > Add module versions to the ABI (or '-' if the module doesn't report a
> > version) and make sure the module check only looks at the module name
> > and ignores the trailing version.
> > 
> > This info ends up in the buildinfo package which can be used by meta
> > packages so that they don't have to build-depend on the huge modules
> > package to determine versions of in-tree DKMS modules.
> > 
> > Signed-off-by: Juerg Haefliger <juergh at canonical.com>  
> 
> This doesn't seem like a very extensible way of doing things if we ever
> need to add additional information from modinfo later.

We could turn this into a CSV list and add more fields as necessary but I
agree it's not very flexible.


> We're already
> inclding the firmware information separately in the fwinfo file. I
> wonder how much it would bloat the abi size to include all of the
> modinfo data, similar to what's in modules.builtin.modinfo

Huh? Where does this file live?


> already
> (which we would probably also want to pull in as part of this
> information). It could probably replace the fwinfo and flavour.modules
> files though as they would now contain redundant information.

Yes, the more I think about it the more I believe we should include all
modinfo data and let the consumer decide which data to use rather than trying
to be 'smart' at buildinfo build time (just to realize later that we should
also have included XYZ). I'll play around with it some more.

...Juerg

 
> Seth

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20210420/214fc48b/attachment.sig>


More information about the kernel-team mailing list