NACK Re: [SRU][f:linux-restricted-modules-hwe-5.15][PATCH 0/1] Add lrm autogenerated transitional independent of variants

[Timo Aaltonen]

Kleber Sacilotto de Souza kirjoitti 29.9.2022 klo 19.33:
> BugLink: https://bugs.launchpad.net/bugs/1991130
> 
> Note: this patch is supposed to be applied to all lrm packages (or at
> least to hwe and generic) but I'm proposing for f:lrm-hwe-5.15 only for
> now to fix the transition from f:oem-5.14. If this works well I will
> send it again to be applied to other lrm packages.
> 
> [Impact]
> 
> The kernel lrm packages have a mechanism to auto-generate transitional packages
> by adding a line to 'debian/package.config' in the following format
> 
> transitional <from> <to> <archs>
> 
> Example:
> 
> transitional 450-oem-20.04 450-generic amd64
> 
> When being processed by 'debian/scripts/gen-rules.lrm' the autogenerated
> contents are added to 'debian/control.d/transitionals-local'. However, this file
> is not included in the final 'debian/control' for packages which does not
> contain the default variant '--' (see debian/rules.in). This is intentional to
> prevent unwanted transitionals.
> 
> However, there are some cases where we want to transition packages to kernels
> which are not the default variant in a series (e.g. hwe kernels).
> 
> [Fix]
> 
> The proposed fix is to add a new command "migrate", which does exactly what
> "transitional" does but adding the content to 'debian/control.d/migrate-local'
> which would get added to the final 'debian/control' independently of the
> source's variants.
> 
> [Test Case]
> 
> Add "migrate" entries to 'debian/package.config' and check whether they get
> added to 'debian/control' even for non default variant packages.
> 
> [Where problems could occur]
> 
> If not used correctly, unwanted transitionals could be generated causing issues
> to the archive. However, this is a new command and should not impact any of the
> previously set transitionals and it needs to be explicitly used.
> 
> Kleber Sacilotto de Souza (1):
>    UBUNTU: [Packaging] add a new migrate autogenerate option
> 
>   debian/scripts/gen-rules.lrm | 21 +++++++++++++++++++++
>   1 file changed, 21 insertions(+)
> 

I'll send a revised (and rebased) version, and target also unstable so 
that we'll get this eventually in the distro kernel too.

-- 
t