[SRU][K][PATCH 0/8] UBUNTU: [Packaging] remove circular dep between modules and image

Stefan Bader stefan.bader at canonical.com
Wed Oct 26 08:35:19 UTC 2022


On 26.10.22 10:17, Dimitri John Ledkov wrote:
> 
> 
> On Wed, 26 Oct 2022, 10:09 Stefan Bader, <stefan.bader at canonical.com 
> <mailto:stefan.bader at canonical.com>> wrote:
> 
>     On 25.10.22 09:15, Dimitri John Ledkov wrote:
>      > Currently we have circular dependencies between linux-image and
>      > linux-modules/linux-modules-extra packages. This is redundant, and
>      > harmful when additional types of linux-images are provided
>      > (i.e. kernel.efi).
>      >
>      > Break the circular dependency by only keeping linux-image ->
>      > linux-modules dep, remove linux-modules -> linux-image dep, remove
>      > linux-modules-extra -> linux-image dep, add linux-modules-extra ->
>      > linux-modules dep.
>      >
>      > This has already been applied to generic kernel, these are patches for
>      > all the remaining kernels in kinetic.
>      >
>      > BugLink: https://bugs.launchpad.net/bugs/1989334
>     <https://bugs.launchpad.net/bugs/1989334>
>      > Signed-off-by: Dimitri John Ledkov <dimitri.ledkov at canonical.com
>     <mailto:dimitri.ledkov at canonical.com>>
>      >
>      > Dimitri John Ledkov (1):
>      >    [SRU][K linux-*][PATCH] UBUNTU: [Packaging] remove circular dep
>      >      between modules and image
>      >
>      >   debian.*/control.d/flavour-control.stub | 4 ++--
>      >   1 file changed, 2 insertion(+), 2 deletion(-)
>      >
> 
>     There are some sources which only get the modules-extra part changed. That
>     might
>     be because the dependency was already removed for modules or forgotten.
>     Impossible to tell from the patches. A note about it here would be helpful...
> 
> 
> True, maybe I should have increased context. It is correct, that each kernel got 
> different set of changes. Most only had incorrect modules-extra deps, and only 
> few had out of date modules package.
> 
> I can try to generate patches with longer context such that it it obvious which 
> packages getting which changes.
> 
Enough to have it here now. But as a reminder to think about reviewers context 
next time.


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


More information about the kernel-team mailing list