diff mbox series

[SRU,K,linux-aws] UBUNTU: [Packaging] remove circular dep between modules and image

Message ID 20221025071604.702-2-dimitri.ledkov@canonical.com
State New
Headers show
Series [SRU,K,linux-aws] UBUNTU: [Packaging] remove circular dep between modules and image | expand

Commit Message

Dimitri John Ledkov Oct. 25, 2022, 7:15 a.m. UTC
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.

BugLink: https://bugs.launchpad.net/bugs/1989334
Signed-off-by: Dimitri John Ledkov <dimitri.ledkov@canonical.com>
---
 debian.aws/control.d/flavour-control.stub | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Dimitri John Ledkov Nov. 16, 2022, 1:49 p.m. UTC | #1
Applied to kinetic:linux-aws master-next branch

On Tue, 25 Oct 2022 at 08:16, Dimitri John Ledkov
<dimitri.ledkov@canonical.com> 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.
>
> BugLink: https://bugs.launchpad.net/bugs/1989334
> Signed-off-by: Dimitri John Ledkov <dimitri.ledkov@canonical.com>
> ---
>  debian.aws/control.d/flavour-control.stub | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/debian.aws/control.d/flavour-control.stub b/debian.aws/control.d/flavour-control.stub
> index 9ef2c56669f3..daadcffefcb2 100644
> --- a/debian.aws/control.d/flavour-control.stub
> +++ b/debian.aws/control.d/flavour-control.stub
> @@ -46,13 +46,13 @@ Description: Linux kernel image for version PKGVER on DESC
>
>  Package: linux-modules-PKGVER-ABINUM-FLAVOUR
>  Build-Profiles: <!stage1>
>  Architecture: ARCH
>  Section: kernel
>  Priority: optional
> -Depends: ${misc:Depends}, ${shlibs:Depends}, linux-image-PKGVER-ABINUM-FLAVOUR | linux-image-unsigned-PKGVER-ABINUM-FLAVOUR
> +Depends: ${misc:Depends}, ${shlibs:Depends}
>  Built-Using: ${linux:BuiltUsing}
>  Description: Linux kernel extra modules for version PKGVER on DESC
>   Contains the corresponding System.map file, the modules built by the
>   packager, and scripts that try to ensure that the system is not left in an
>   unbootable state after an update.
>   .
> @@ -66,13 +66,13 @@ Description: Linux kernel extra modules for version PKGVER on DESC
>
>  Package: linux-modules-extra-PKGVER-ABINUM-FLAVOUR
>  Build-Profiles: <!stage1>
>  Architecture: ARCH
>  Section: kernel
>  Priority: optional
> -Depends: ${misc:Depends}, ${shlibs:Depends}, linux-image-PKGVER-ABINUM-FLAVOUR | linux-image-unsigned-PKGVER-ABINUM-FLAVOUR, wireless-regdb
> +Depends: ${misc:Depends}, ${shlibs:Depends}, wireless-regdb, linux-modules-PKGVER-ABINUM-FLAVOUR
>  Description: Linux kernel extra modules for version PKGVER on DESC
>   This package contains the Linux kernel extra modules for version PKGVER on
>   DESC.
>   .
>   Supports SUPPORTED processors.
>   .
> --
> 2.34.1
>
diff mbox series

Patch

diff --git a/debian.aws/control.d/flavour-control.stub b/debian.aws/control.d/flavour-control.stub
index 9ef2c56669f3..daadcffefcb2 100644
--- a/debian.aws/control.d/flavour-control.stub
+++ b/debian.aws/control.d/flavour-control.stub
@@ -46,13 +46,13 @@  Description: Linux kernel image for version PKGVER on DESC
 
 Package: linux-modules-PKGVER-ABINUM-FLAVOUR
 Build-Profiles: <!stage1>
 Architecture: ARCH
 Section: kernel
 Priority: optional
-Depends: ${misc:Depends}, ${shlibs:Depends}, linux-image-PKGVER-ABINUM-FLAVOUR | linux-image-unsigned-PKGVER-ABINUM-FLAVOUR
+Depends: ${misc:Depends}, ${shlibs:Depends}
 Built-Using: ${linux:BuiltUsing}
 Description: Linux kernel extra modules for version PKGVER on DESC
  Contains the corresponding System.map file, the modules built by the
  packager, and scripts that try to ensure that the system is not left in an
  unbootable state after an update.
  .
@@ -66,13 +66,13 @@  Description: Linux kernel extra modules for version PKGVER on DESC
 
 Package: linux-modules-extra-PKGVER-ABINUM-FLAVOUR
 Build-Profiles: <!stage1>
 Architecture: ARCH
 Section: kernel
 Priority: optional
-Depends: ${misc:Depends}, ${shlibs:Depends}, linux-image-PKGVER-ABINUM-FLAVOUR | linux-image-unsigned-PKGVER-ABINUM-FLAVOUR, wireless-regdb
+Depends: ${misc:Depends}, ${shlibs:Depends}, wireless-regdb, linux-modules-PKGVER-ABINUM-FLAVOUR
 Description: Linux kernel extra modules for version PKGVER on DESC
  This package contains the Linux kernel extra modules for version PKGVER on
  DESC.
  .
  Supports SUPPORTED processors.
  .