[azure/aws/gcp/gke/kvm/T/X/Z/A][PATCH v2] Please only recommend or suggest initramfs-tools | linux-initramfs-tool for kernels able to boot without initramfs

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
18 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[azure/aws/gcp/gke/kvm/T/X/Z/A][PATCH v2] Please only recommend or suggest initramfs-tools | linux-initramfs-tool for kernels able to boot without initramfs

Marcelo Henrique Cerri
BugLink: http://bugs.launchpad.net/bugs/1700972

v2:

The post/pre installation/removal scripts also need to be update to skip
the initramfs generation.

For linux-azure and linux-kvm, the patches only contain the changes to the
scripts, since v1 was already applied (and not reverted).

v1:

Allow images to be created without the need of an initrd and also allow
users to run without an initrd if they want to.

The dependency to initramfs-tools was moved to Suggests for the cloud kernels,
and for the regular kernels it was moved to Recommends to be more conservative.

--
Marcelo Henrique Cerri (1):
  UBUNTU: [Debian] Don't depend on initramfs-tools

 debian.master/control.d/flavour-control.stub | 4 ++--
 debian/control-scripts/postinst              | 2 +-
 debian/control-scripts/postrm                | 2 ++
 debian/control-scripts/preinst               | 2 ++
 debian/control-scripts/prerm                 | 1 +
 5 files changed, 8 insertions(+), 3 deletions(-)

--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[xenial][PATCH] UBUNTU: [Debian] Don't depend on initramfs-tools

Marcelo Henrique Cerri
BugLink: http://bugs.launchpad.net/bugs/1700972

Allow images to be created without the need of an initrd and also allow
users to run without an initrd if they want to.

Signed-off-by: Marcelo Henrique Cerri <[hidden email]>
---
 debian.master/control.d/flavour-control.stub | 4 ++--
 debian/control-scripts/postinst              | 2 +-
 debian/control-scripts/postrm                | 2 ++
 debian/control-scripts/preinst               | 2 ++
 debian/control-scripts/prerm                 | 1 +
 5 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/debian.master/control.d/flavour-control.stub b/debian.master/control.d/flavour-control.stub
index e63a682dbd5b..5b7315a7dda6 100644
--- a/debian.master/control.d/flavour-control.stub
+++ b/debian.master/control.d/flavour-control.stub
@@ -27,8 +27,8 @@ Architecture: ARCH
 Section: kernel
 Priority: optional
 Provides: linux-image, fuse-module, =PROVIDES=${linux:rprovides}
-Depends: ${misc:Depends}, ${shlibs:Depends}, initramfs-tools | linux-initramfs-tool, kmod
-Recommends: BOOTLOADER
+Depends: ${misc:Depends}, ${shlibs:Depends}, kmod
+Recommends: BOOTLOADER, initramfs-tools | linux-initramfs-tool
 Suggests: fdutils, SRCPKGNAME-doc-PKGVER | SRCPKGNAME-source-PKGVER, SRCPKGNAME-tools, linux-headers-PKGVER-ABINUM-FLAVOUR
 Description: Linux kernel image for version PKGVER on DESC
  This package contains the Linux kernel image for version PKGVER on
diff --git a/debian/control-scripts/postinst b/debian/control-scripts/postinst
index 808575ce8a80..43de83fa5891 100644
--- a/debian/control-scripts/postinst
+++ b/debian/control-scripts/postinst
@@ -142,7 +142,7 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
-
+$initrd = '' if (! -x "$ramdisk");
 
 # For some versions of kernel-package, we had this warning in the
 # postinst, but the rules did not really interpolate the value in.
diff --git a/debian/control-scripts/postrm b/debian/control-scripts/postrm
index 227e351fc59b..9879178e2147 100644
--- a/debian/control-scripts/postrm
+++ b/debian/control-scripts/postrm
@@ -156,6 +156,8 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
+
 if ($link_in_boot) {
   $image_dest = "/$image_dir/";
   $image_dest =~ s|^/*|/|o;
diff --git a/debian/control-scripts/preinst b/debian/control-scripts/preinst
index 972377609fce..4c70e36b9631 100644
--- a/debian/control-scripts/preinst
+++ b/debian/control-scripts/preinst
@@ -161,6 +161,8 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
+
 $ENV{KERNEL_ARCH}=$kernel_arch if $kernel_arch;
 
 # About to upgrade this package from version $2 TO THIS VERSION.
diff --git a/debian/control-scripts/prerm b/debian/control-scripts/prerm
index 07e41461d3ab..755247f8e35e 100644
--- a/debian/control-scripts/prerm
+++ b/debian/control-scripts/prerm
@@ -163,6 +163,7 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
 
 $ENV{KERNEL_ARCH}=$kernel_arch if $kernel_arch;
 
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[trusty][PATCH] UBUNTU: [Debian] Don't depend on initramfs-tools

Marcelo Henrique Cerri
In reply to this post by Marcelo Henrique Cerri
BugLink: http://bugs.launchpad.net/bugs/1700972

Allow images to be created without the need of an initrd and also allow
users to run without an initrd if they want to.

Signed-off-by: Marcelo Henrique Cerri <[hidden email]>
---
 debian.master/control.d/flavour-control.stub | 4 ++--
 debian/control-scripts/postinst              | 2 +-
 debian/control-scripts/postrm                | 2 ++
 debian/control-scripts/preinst               | 2 ++
 debian/control-scripts/prerm                 | 1 +
 5 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/debian.master/control.d/flavour-control.stub b/debian.master/control.d/flavour-control.stub
index 3ecf4102d38e..31db80382da3 100644
--- a/debian.master/control.d/flavour-control.stub
+++ b/debian.master/control.d/flavour-control.stub
@@ -27,9 +27,9 @@ Section: kernel
 Priority: optional
 Pre-Depends: dpkg (>= 1.10.24)
 Provides: linux-image, linux-image-3.0, fuse-module, =PROVIDES=
-Depends: ${misc:Depends}, ${shlibs:Depends}, initramfs-tools (>= 0.36ubuntu6), module-init-tools (>= 3.3-pre11-4ubuntu3)
+Depends: ${misc:Depends}, ${shlibs:Depends}, module-init-tools (>= 3.3-pre11-4ubuntu3)
 Conflicts: hotplug (<< 0.0.20040105-1)
-Recommends: BOOTLOADER
+Recommends: BOOTLOADER, initramfs-tools (>= 0.36ubuntu6)
 Suggests: fdutils, SRCPKGNAME-doc-PKGVER | SRCPKGNAME-source-PKGVER, SRCPKGNAME-tools, linux-headers-PKGVER-ABINUM-FLAVOUR
 Description: Linux kernel image for version PKGVER on DESC
  This package contains the Linux kernel image for version PKGVER on
diff --git a/debian/control-scripts/postinst b/debian/control-scripts/postinst
index c4fc23f680c3..f8c884a96e81 100644
--- a/debian/control-scripts/postinst
+++ b/debian/control-scripts/postinst
@@ -142,7 +142,7 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
-
+$initrd = '' if (! -x "$ramdisk");
 
 # For some versions of kernel-package, we had this warning in the
 # postinst, but the rules did not really interpolate the value in.
diff --git a/debian/control-scripts/postrm b/debian/control-scripts/postrm
index 227e351fc59b..9879178e2147 100644
--- a/debian/control-scripts/postrm
+++ b/debian/control-scripts/postrm
@@ -156,6 +156,8 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
+
 if ($link_in_boot) {
   $image_dest = "/$image_dir/";
   $image_dest =~ s|^/*|/|o;
diff --git a/debian/control-scripts/preinst b/debian/control-scripts/preinst
index 972377609fce..4c70e36b9631 100644
--- a/debian/control-scripts/preinst
+++ b/debian/control-scripts/preinst
@@ -161,6 +161,8 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
+
 $ENV{KERNEL_ARCH}=$kernel_arch if $kernel_arch;
 
 # About to upgrade this package from version $2 TO THIS VERSION.
diff --git a/debian/control-scripts/prerm b/debian/control-scripts/prerm
index 07e41461d3ab..755247f8e35e 100644
--- a/debian/control-scripts/prerm
+++ b/debian/control-scripts/prerm
@@ -163,6 +163,7 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
 
 $ENV{KERNEL_ARCH}=$kernel_arch if $kernel_arch;
 
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[zesty][PATCH] UBUNTU: [Debian] Don't depend on initramfs-tools

Marcelo Henrique Cerri
In reply to this post by Marcelo Henrique Cerri
BugLink: http://bugs.launchpad.net/bugs/1700972

Allow images to be created without the need of an initrd and also allow
users to run without an initrd if they want to.

Signed-off-by: Marcelo Henrique Cerri <[hidden email]>
---
 debian.master/control.d/flavour-control.stub | 4 ++--
 debian/control-scripts/postinst              | 2 +-
 debian/control-scripts/postrm                | 2 ++
 debian/control-scripts/preinst               | 2 ++
 debian/control-scripts/prerm                 | 1 +
 5 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/debian.master/control.d/flavour-control.stub b/debian.master/control.d/flavour-control.stub
index 6318ad236538..b85b6cd001a2 100644
--- a/debian.master/control.d/flavour-control.stub
+++ b/debian.master/control.d/flavour-control.stub
@@ -27,8 +27,8 @@ Architecture: ARCH
 Section: kernel
 Priority: optional
 Provides: linux-image, fuse-module, =PROVIDES=${linux:rprovides}
-Depends: ${misc:Depends}, ${shlibs:Depends}, initramfs-tools | linux-initramfs-tool, kmod
-Recommends: BOOTLOADER
+Depends: ${misc:Depends}, ${shlibs:Depends}, kmod
+Recommends: BOOTLOADER, initramfs-tools | linux-initramfs-tool
 Breaks: flash-kernel (<< 3.0~rc.4ubuntu64) [arm64]
 Suggests: fdutils, SRCPKGNAME-doc-PKGVER | SRCPKGNAME-source-PKGVER, SRCPKGNAME-tools, linux-headers-PKGVER-ABINUM-FLAVOUR
 Description: Linux kernel image for version PKGVER on DESC
diff --git a/debian/control-scripts/postinst b/debian/control-scripts/postinst
index 808575ce8a80..43de83fa5891 100644
--- a/debian/control-scripts/postinst
+++ b/debian/control-scripts/postinst
@@ -142,7 +142,7 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
-
+$initrd = '' if (! -x "$ramdisk");
 
 # For some versions of kernel-package, we had this warning in the
 # postinst, but the rules did not really interpolate the value in.
diff --git a/debian/control-scripts/postrm b/debian/control-scripts/postrm
index 227e351fc59b..9879178e2147 100644
--- a/debian/control-scripts/postrm
+++ b/debian/control-scripts/postrm
@@ -156,6 +156,8 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
+
 if ($link_in_boot) {
   $image_dest = "/$image_dir/";
   $image_dest =~ s|^/*|/|o;
diff --git a/debian/control-scripts/preinst b/debian/control-scripts/preinst
index 972377609fce..4c70e36b9631 100644
--- a/debian/control-scripts/preinst
+++ b/debian/control-scripts/preinst
@@ -161,6 +161,8 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
+
 $ENV{KERNEL_ARCH}=$kernel_arch if $kernel_arch;
 
 # About to upgrade this package from version $2 TO THIS VERSION.
diff --git a/debian/control-scripts/prerm b/debian/control-scripts/prerm
index 07e41461d3ab..755247f8e35e 100644
--- a/debian/control-scripts/prerm
+++ b/debian/control-scripts/prerm
@@ -163,6 +163,7 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
 
 $ENV{KERNEL_ARCH}=$kernel_arch if $kernel_arch;
 
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[artful][PATCH] UBUNTU: [Debian] Don't depend on initramfs-tools

Marcelo Henrique Cerri
In reply to this post by Marcelo Henrique Cerri
BugLink: http://bugs.launchpad.net/bugs/1700972

Allow images to be created without the need of an initrd and also allow
users to run without an initrd if they want to.

Signed-off-by: Marcelo Henrique Cerri <[hidden email]>
---
 debian.master/control.d/flavour-control.stub | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/debian.master/control.d/flavour-control.stub b/debian.master/control.d/flavour-control.stub
index 6318ad236538..b85b6cd001a2 100644
--- a/debian.master/control.d/flavour-control.stub
+++ b/debian.master/control.d/flavour-control.stub
@@ -27,8 +27,8 @@ Architecture: ARCH
 Section: kernel
 Priority: optional
 Provides: linux-image, fuse-module, =PROVIDES=${linux:rprovides}
-Depends: ${misc:Depends}, ${shlibs:Depends}, initramfs-tools | linux-initramfs-tool, kmod
-Recommends: BOOTLOADER
+Depends: ${misc:Depends}, ${shlibs:Depends}, kmod
+Recommends: BOOTLOADER, initramfs-tools | linux-initramfs-tool
 Breaks: flash-kernel (<< 3.0~rc.4ubuntu64) [arm64]
 Suggests: fdutils, SRCPKGNAME-doc-PKGVER | SRCPKGNAME-source-PKGVER, SRCPKGNAME-tools, linux-headers-PKGVER-ABINUM-FLAVOUR
 Description: Linux kernel image for version PKGVER on DESC
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[azure][PATCH] UBUNTU: [Debian] Don't abort installation if update-initramfs is missing

Marcelo Henrique Cerri
In reply to this post by Marcelo Henrique Cerri
BugLink: http://bugs.launchpad.net/bugs/1700972

Change the control scripts to ignore the initramfs generation when
initramfs-tools is not available.

Signed-off-by: Marcelo Henrique Cerri <[hidden email]>
---
 debian/control-scripts/postinst | 2 +-
 debian/control-scripts/postrm   | 2 ++
 debian/control-scripts/preinst  | 2 ++
 debian/control-scripts/prerm    | 1 +
 4 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/debian/control-scripts/postinst b/debian/control-scripts/postinst
index 808575ce8a80..43de83fa5891 100644
--- a/debian/control-scripts/postinst
+++ b/debian/control-scripts/postinst
@@ -142,7 +142,7 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
-
+$initrd = '' if (! -x "$ramdisk");
 
 # For some versions of kernel-package, we had this warning in the
 # postinst, but the rules did not really interpolate the value in.
diff --git a/debian/control-scripts/postrm b/debian/control-scripts/postrm
index 227e351fc59b..9879178e2147 100644
--- a/debian/control-scripts/postrm
+++ b/debian/control-scripts/postrm
@@ -156,6 +156,8 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
+
 if ($link_in_boot) {
   $image_dest = "/$image_dir/";
   $image_dest =~ s|^/*|/|o;
diff --git a/debian/control-scripts/preinst b/debian/control-scripts/preinst
index 972377609fce..4c70e36b9631 100644
--- a/debian/control-scripts/preinst
+++ b/debian/control-scripts/preinst
@@ -161,6 +161,8 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
+
 $ENV{KERNEL_ARCH}=$kernel_arch if $kernel_arch;
 
 # About to upgrade this package from version $2 TO THIS VERSION.
diff --git a/debian/control-scripts/prerm b/debian/control-scripts/prerm
index 07e41461d3ab..755247f8e35e 100644
--- a/debian/control-scripts/prerm
+++ b/debian/control-scripts/prerm
@@ -163,6 +163,7 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
 
 $ENV{KERNEL_ARCH}=$kernel_arch if $kernel_arch;
 
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[kvm][PATCH] UBUNTU: [Debian] Don't abort installation if update-initramfs is missing

Marcelo Henrique Cerri
In reply to this post by Marcelo Henrique Cerri
BugLink: http://bugs.launchpad.net/bugs/1700972

Change the control scripts to ignore the initramfs generation when
initramfs-tools is not available.

Signed-off-by: Marcelo Henrique Cerri <[hidden email]>
---
 debian/control-scripts/postinst | 2 +-
 debian/control-scripts/postrm   | 2 ++
 debian/control-scripts/preinst  | 2 ++
 debian/control-scripts/prerm    | 1 +
 4 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/debian/control-scripts/postinst b/debian/control-scripts/postinst
index 808575ce8a80..43de83fa5891 100644
--- a/debian/control-scripts/postinst
+++ b/debian/control-scripts/postinst
@@ -142,7 +142,7 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
-
+$initrd = '' if (! -x "$ramdisk");
 
 # For some versions of kernel-package, we had this warning in the
 # postinst, but the rules did not really interpolate the value in.
diff --git a/debian/control-scripts/postrm b/debian/control-scripts/postrm
index 227e351fc59b..9879178e2147 100644
--- a/debian/control-scripts/postrm
+++ b/debian/control-scripts/postrm
@@ -156,6 +156,8 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
+
 if ($link_in_boot) {
   $image_dest = "/$image_dir/";
   $image_dest =~ s|^/*|/|o;
diff --git a/debian/control-scripts/preinst b/debian/control-scripts/preinst
index 972377609fce..4c70e36b9631 100644
--- a/debian/control-scripts/preinst
+++ b/debian/control-scripts/preinst
@@ -161,6 +161,8 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
+
 $ENV{KERNEL_ARCH}=$kernel_arch if $kernel_arch;
 
 # About to upgrade this package from version $2 TO THIS VERSION.
diff --git a/debian/control-scripts/prerm b/debian/control-scripts/prerm
index 07e41461d3ab..755247f8e35e 100644
--- a/debian/control-scripts/prerm
+++ b/debian/control-scripts/prerm
@@ -163,6 +163,7 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
 
 $ENV{KERNEL_ARCH}=$kernel_arch if $kernel_arch;
 
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[aws][PATCH] UBUNTU: [Debian] Don't depend on initramfs-tools

Marcelo Henrique Cerri
In reply to this post by Marcelo Henrique Cerri
BugLink: http://bugs.launchpad.net/bugs/1700972

Allow images to be created without the need of an initrd and also allow
users to run without an initrd if they want to.

Signed-off-by: Marcelo Henrique Cerri <[hidden email]>
---
 debian.aws/control.d/flavour-control.stub | 4 ++--
 debian/control-scripts/postinst           | 2 +-
 debian/control-scripts/postrm             | 2 ++
 debian/control-scripts/preinst            | 2 ++
 debian/control-scripts/prerm              | 1 +
 5 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/debian.aws/control.d/flavour-control.stub b/debian.aws/control.d/flavour-control.stub
index bdefd9b830c8..0140035e3ac6 100644
--- a/debian.aws/control.d/flavour-control.stub
+++ b/debian.aws/control.d/flavour-control.stub
@@ -27,9 +27,9 @@ Architecture: ARCH
 Section: kernel
 Priority: optional
 Provides: linux-image, fuse-module, =PROVIDES=${linux:rprovides}
-Depends: ${misc:Depends}, ${shlibs:Depends}, initramfs-tools | linux-initramfs-tool, kmod
+Depends: ${misc:Depends}, ${shlibs:Depends}, kmod
 Recommends: BOOTLOADER
-Suggests: fdutils, SRCPKGNAME-doc-PKGVER | SRCPKGNAME-source-PKGVER, SRCPKGNAME-tools, linux-headers-PKGVER-ABINUM-FLAVOUR
+Suggests: fdutils, SRCPKGNAME-doc-PKGVER | SRCPKGNAME-source-PKGVER, SRCPKGNAME-tools, linux-headers-PKGVER-ABINUM-FLAVOUR, initramfs-tools | linux-initramfs-tool
 Description: Linux kernel image for version PKGVER on DESC
  This package contains the Linux kernel image for version PKGVER on
  DESC.
diff --git a/debian/control-scripts/postinst b/debian/control-scripts/postinst
index 808575ce8a80..43de83fa5891 100644
--- a/debian/control-scripts/postinst
+++ b/debian/control-scripts/postinst
@@ -142,7 +142,7 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
-
+$initrd = '' if (! -x "$ramdisk");
 
 # For some versions of kernel-package, we had this warning in the
 # postinst, but the rules did not really interpolate the value in.
diff --git a/debian/control-scripts/postrm b/debian/control-scripts/postrm
index 227e351fc59b..9879178e2147 100644
--- a/debian/control-scripts/postrm
+++ b/debian/control-scripts/postrm
@@ -156,6 +156,8 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
+
 if ($link_in_boot) {
   $image_dest = "/$image_dir/";
   $image_dest =~ s|^/*|/|o;
diff --git a/debian/control-scripts/preinst b/debian/control-scripts/preinst
index 972377609fce..4c70e36b9631 100644
--- a/debian/control-scripts/preinst
+++ b/debian/control-scripts/preinst
@@ -161,6 +161,8 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
+
 $ENV{KERNEL_ARCH}=$kernel_arch if $kernel_arch;
 
 # About to upgrade this package from version $2 TO THIS VERSION.
diff --git a/debian/control-scripts/prerm b/debian/control-scripts/prerm
index 07e41461d3ab..755247f8e35e 100644
--- a/debian/control-scripts/prerm
+++ b/debian/control-scripts/prerm
@@ -163,6 +163,7 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
 
 $ENV{KERNEL_ARCH}=$kernel_arch if $kernel_arch;
 
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[gcp][PATCH] UBUNTU: [Debian] Don't depend on initramfs-tools

Marcelo Henrique Cerri
In reply to this post by Marcelo Henrique Cerri
BugLink: http://bugs.launchpad.net/bugs/1700972

Allow images to be created without the need of an initrd and also allow
users to run without an initrd if they want to.

Signed-off-by: Marcelo Henrique Cerri <[hidden email]>
---
 debian.gcp/control.d/flavour-control.stub | 4 ++--
 debian/control-scripts/postinst           | 2 +-
 debian/control-scripts/postrm             | 2 ++
 debian/control-scripts/preinst            | 2 ++
 debian/control-scripts/prerm              | 1 +
 5 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/debian.gcp/control.d/flavour-control.stub b/debian.gcp/control.d/flavour-control.stub
index d871ac82e6a5..2e97c52b039d 100644
--- a/debian.gcp/control.d/flavour-control.stub
+++ b/debian.gcp/control.d/flavour-control.stub
@@ -27,9 +27,9 @@ Architecture: ARCH
 Section: kernel
 Priority: optional
 Provides: linux-image, fuse-module, =PROVIDES=${linux:rprovides}
-Depends: ${misc:Depends}, ${shlibs:Depends}, initramfs-tools | linux-initramfs-tool, kmod
+Depends: ${misc:Depends}, ${shlibs:Depends}, kmod
 Recommends: BOOTLOADER
-Suggests: fdutils, SRCPKGNAME-doc-PKGVER | SRCPKGNAME-source-PKGVER, SRCPKGNAME-tools, linux-headers-PKGVER-ABINUM-FLAVOUR
+Suggests: fdutils, SRCPKGNAME-doc-PKGVER | SRCPKGNAME-source-PKGVER, SRCPKGNAME-tools, linux-headers-PKGVER-ABINUM-FLAVOUR, initramfs-tools | linux-initramfs-tool
 Description: Linux kernel image for version PKGVER on DESC
  This package contains the Linux kernel image for version PKGVER on
  DESC.
diff --git a/debian/control-scripts/postinst b/debian/control-scripts/postinst
index 808575ce8a80..43de83fa5891 100644
--- a/debian/control-scripts/postinst
+++ b/debian/control-scripts/postinst
@@ -142,7 +142,7 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
-
+$initrd = '' if (! -x "$ramdisk");
 
 # For some versions of kernel-package, we had this warning in the
 # postinst, but the rules did not really interpolate the value in.
diff --git a/debian/control-scripts/postrm b/debian/control-scripts/postrm
index 227e351fc59b..9879178e2147 100644
--- a/debian/control-scripts/postrm
+++ b/debian/control-scripts/postrm
@@ -156,6 +156,8 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
+
 if ($link_in_boot) {
   $image_dest = "/$image_dir/";
   $image_dest =~ s|^/*|/|o;
diff --git a/debian/control-scripts/preinst b/debian/control-scripts/preinst
index 972377609fce..4c70e36b9631 100644
--- a/debian/control-scripts/preinst
+++ b/debian/control-scripts/preinst
@@ -161,6 +161,8 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
+
 $ENV{KERNEL_ARCH}=$kernel_arch if $kernel_arch;
 
 # About to upgrade this package from version $2 TO THIS VERSION.
diff --git a/debian/control-scripts/prerm b/debian/control-scripts/prerm
index 07e41461d3ab..755247f8e35e 100644
--- a/debian/control-scripts/prerm
+++ b/debian/control-scripts/prerm
@@ -163,6 +163,7 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
 
 $ENV{KERNEL_ARCH}=$kernel_arch if $kernel_arch;
 
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[gke][PATCH] UBUNTU: [Debian] Don't depend on initramfs-tools

Marcelo Henrique Cerri
In reply to this post by Marcelo Henrique Cerri
BugLink: http://bugs.launchpad.net/bugs/1700972

Allow images to be created without the need of an initrd and also allow
users to run without an initrd if they want to.

Signed-off-by: Marcelo Henrique Cerri <[hidden email]>
---
 debian.gke/control.d/flavour-control.stub | 4 ++--
 debian/control-scripts/postinst           | 2 +-
 debian/control-scripts/postrm             | 2 ++
 debian/control-scripts/preinst            | 2 ++
 debian/control-scripts/prerm              | 1 +
 5 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/debian.gke/control.d/flavour-control.stub b/debian.gke/control.d/flavour-control.stub
index 6a9d11ba56fc..2dbb25c17958 100644
--- a/debian.gke/control.d/flavour-control.stub
+++ b/debian.gke/control.d/flavour-control.stub
@@ -27,9 +27,9 @@ Architecture: ARCH
 Section: kernel
 Priority: optional
 Provides: linux-image, fuse-module, =PROVIDES=${linux:rprovides}
-Depends: ${misc:Depends}, ${shlibs:Depends}, initramfs-tools | linux-initramfs-tool, kmod
+Depends: ${misc:Depends}, ${shlibs:Depends}, kmod
 Recommends: BOOTLOADER
-Suggests: fdutils, SRCPKGNAME-doc-PKGVER | SRCPKGNAME-source-PKGVER, SRCPKGNAME-tools, linux-headers-PKGVER-ABINUM-FLAVOUR
+Suggests: fdutils, SRCPKGNAME-doc-PKGVER | SRCPKGNAME-source-PKGVER, SRCPKGNAME-tools, linux-headers-PKGVER-ABINUM-FLAVOUR, initramfs-tools | linux-initramfs-tool
 Description: Linux kernel image for version PKGVER on DESC
  This package contains the Linux kernel image for version PKGVER on
  DESC.
diff --git a/debian/control-scripts/postinst b/debian/control-scripts/postinst
index 808575ce8a80..43de83fa5891 100644
--- a/debian/control-scripts/postinst
+++ b/debian/control-scripts/postinst
@@ -142,7 +142,7 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
-
+$initrd = '' if (! -x "$ramdisk");
 
 # For some versions of kernel-package, we had this warning in the
 # postinst, but the rules did not really interpolate the value in.
diff --git a/debian/control-scripts/postrm b/debian/control-scripts/postrm
index 227e351fc59b..9879178e2147 100644
--- a/debian/control-scripts/postrm
+++ b/debian/control-scripts/postrm
@@ -156,6 +156,8 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
+
 if ($link_in_boot) {
   $image_dest = "/$image_dir/";
   $image_dest =~ s|^/*|/|o;
diff --git a/debian/control-scripts/preinst b/debian/control-scripts/preinst
index 972377609fce..4c70e36b9631 100644
--- a/debian/control-scripts/preinst
+++ b/debian/control-scripts/preinst
@@ -161,6 +161,8 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
+
 $ENV{KERNEL_ARCH}=$kernel_arch if $kernel_arch;
 
 # About to upgrade this package from version $2 TO THIS VERSION.
diff --git a/debian/control-scripts/prerm b/debian/control-scripts/prerm
index 07e41461d3ab..755247f8e35e 100644
--- a/debian/control-scripts/prerm
+++ b/debian/control-scripts/prerm
@@ -163,6 +163,7 @@ if (-r "$CONF_LOC" && -f "$CONF_LOC"  ) {
   }
 }
 
+$initrd = '' if (! -x "$ramdisk");
 
 $ENV{KERNEL_ARCH}=$kernel_arch if $kernel_arch;
 
--
2.7.4


--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

ACK: [azure/aws/gcp/gke/kvm/T/X/Z/A][PATCH v2] Please only recommend or suggest initramfs-tools | linux-initramfs-tool for kernels able to boot without initramfs

Kamal Mostafa-2
In reply to this post by Marcelo Henrique Cerri
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [artful][PATCH] UBUNTU: [Debian] Don't depend on initramfs-tools

Seth Forshee
In reply to this post by Marcelo Henrique Cerri
On Tue, Aug 08, 2017 at 04:16:06PM -0300, Marcelo Henrique Cerri wrote:

> BugLink: http://bugs.launchpad.net/bugs/1700972
>
> Allow images to be created without the need of an initrd and also allow
> users to run without an initrd if they want to.
>
> Signed-off-by: Marcelo Henrique Cerri <[hidden email]>
> ---
>  debian.master/control.d/flavour-control.stub | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/debian.master/control.d/flavour-control.stub b/debian.master/control.d/flavour-control.stub
> index 6318ad236538..b85b6cd001a2 100644
> --- a/debian.master/control.d/flavour-control.stub
> +++ b/debian.master/control.d/flavour-control.stub
> @@ -27,8 +27,8 @@ Architecture: ARCH
>  Section: kernel
>  Priority: optional
>  Provides: linux-image, fuse-module, =PROVIDES=${linux:rprovides}
> -Depends: ${misc:Depends}, ${shlibs:Depends}, initramfs-tools | linux-initramfs-tool, kmod
> -Recommends: BOOTLOADER
> +Depends: ${misc:Depends}, ${shlibs:Depends}, kmod
> +Recommends: BOOTLOADER, initramfs-tools | linux-initramfs-tool
>  Breaks: flash-kernel (<< 3.0~rc.4ubuntu64) [arm64]
>  Suggests: fdutils, SRCPKGNAME-doc-PKGVER | SRCPKGNAME-source-PKGVER, SRCPKGNAME-tools, linux-headers-PKGVER-ABINUM-FLAVOUR
>  Description: Linux kernel image for version PKGVER on DESC

This one seems to be missing the changes to the control scripts.

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [artful][PATCH] UBUNTU: [Debian] Don't depend on initramfs-tools

Seth Forshee
On Thu, Aug 10, 2017 at 10:45:31AM -0500, Seth Forshee wrote:

> On Tue, Aug 08, 2017 at 04:16:06PM -0300, Marcelo Henrique Cerri wrote:
> > BugLink: http://bugs.launchpad.net/bugs/1700972
> >
> > Allow images to be created without the need of an initrd and also allow
> > users to run without an initrd if they want to.
> >
> > Signed-off-by: Marcelo Henrique Cerri <[hidden email]>
> > ---
> >  debian.master/control.d/flavour-control.stub | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/debian.master/control.d/flavour-control.stub b/debian.master/control.d/flavour-control.stub
> > index 6318ad236538..b85b6cd001a2 100644
> > --- a/debian.master/control.d/flavour-control.stub
> > +++ b/debian.master/control.d/flavour-control.stub
> > @@ -27,8 +27,8 @@ Architecture: ARCH
> >  Section: kernel
> >  Priority: optional
> >  Provides: linux-image, fuse-module, =PROVIDES=${linux:rprovides}
> > -Depends: ${misc:Depends}, ${shlibs:Depends}, initramfs-tools | linux-initramfs-tool, kmod
> > -Recommends: BOOTLOADER
> > +Depends: ${misc:Depends}, ${shlibs:Depends}, kmod
> > +Recommends: BOOTLOADER, initramfs-tools | linux-initramfs-tool
> >  Breaks: flash-kernel (<< 3.0~rc.4ubuntu64) [arm64]
> >  Suggests: fdutils, SRCPKGNAME-doc-PKGVER | SRCPKGNAME-source-PKGVER, SRCPKGNAME-tools, linux-headers-PKGVER-ABINUM-FLAVOUR
> >  Description: Linux kernel image for version PKGVER on DESC
>
> This one seems to be missing the changes to the control scripts.

The patch for zesty applies just fine though, I assume there weren't any
different changes needed for artful?

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [artful][PATCH] UBUNTU: [Debian] Don't depend on initramfs-tools

Marcelo Henrique Cerri
No. They are the same. I did a mistake when generating the patches.

--
Regards,
Marcelo

On Thu, Aug 10, 2017 at 10:47:43AM -0500, Seth Forshee wrote:

> On Thu, Aug 10, 2017 at 10:45:31AM -0500, Seth Forshee wrote:
> > On Tue, Aug 08, 2017 at 04:16:06PM -0300, Marcelo Henrique Cerri wrote:
> > > BugLink: http://bugs.launchpad.net/bugs/1700972
> > >
> > > Allow images to be created without the need of an initrd and also allow
> > > users to run without an initrd if they want to.
> > >
> > > Signed-off-by: Marcelo Henrique Cerri <[hidden email]>
> > > ---
> > >  debian.master/control.d/flavour-control.stub | 4 ++--
> > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/debian.master/control.d/flavour-control.stub b/debian.master/control.d/flavour-control.stub
> > > index 6318ad236538..b85b6cd001a2 100644
> > > --- a/debian.master/control.d/flavour-control.stub
> > > +++ b/debian.master/control.d/flavour-control.stub
> > > @@ -27,8 +27,8 @@ Architecture: ARCH
> > >  Section: kernel
> > >  Priority: optional
> > >  Provides: linux-image, fuse-module, =PROVIDES=${linux:rprovides}
> > > -Depends: ${misc:Depends}, ${shlibs:Depends}, initramfs-tools | linux-initramfs-tool, kmod
> > > -Recommends: BOOTLOADER
> > > +Depends: ${misc:Depends}, ${shlibs:Depends}, kmod
> > > +Recommends: BOOTLOADER, initramfs-tools | linux-initramfs-tool
> > >  Breaks: flash-kernel (<< 3.0~rc.4ubuntu64) [arm64]
> > >  Suggests: fdutils, SRCPKGNAME-doc-PKGVER | SRCPKGNAME-source-PKGVER, SRCPKGNAME-tools, linux-headers-PKGVER-ABINUM-FLAVOUR
> > >  Description: Linux kernel image for version PKGVER on DESC
> >
> > This one seems to be missing the changes to the control scripts.
>
> The patch for zesty applies just fine though, I assume there weren't any
> different changes needed for artful?

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team

signature.asc (484 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

APPLIED: [artful][PATCH] UBUNTU: [Debian] Don't depend on initramfs-tools

Seth Forshee
On Thu, Aug 10, 2017 at 03:25:50PM -0300, Marcelo Henrique Cerri wrote:
> No. They are the same. I did a mistake when generating the patches.

Applied to artful, using the patch for zesty. I've confirmed that the
error building linux-signed is now fixed. Thanks!

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

ACK: [azure/aws/gcp/gke/kvm/T/X/Z/A][PATCH v2] Please only recommend or suggest initramfs-tools | linux-initramfs-tool for kernels able to boot without initramfs

Seth Forshee
In reply to this post by Marcelo Henrique Cerri
On Tue, Aug 08, 2017 at 04:16:02PM -0300, Marcelo Henrique Cerri wrote:

> BugLink: http://bugs.launchpad.net/bugs/1700972
>
> v2:
>
> The post/pre installation/removal scripts also need to be update to skip
> the initramfs generation.
>
> For linux-azure and linux-kvm, the patches only contain the changes to the
> scripts, since v1 was already applied (and not reverted).
>
> v1:
>
> Allow images to be created without the need of an initrd and also allow
> users to run without an initrd if they want to.
>
> The dependency to initramfs-tools was moved to Suggests for the cloud kernels,
> and for the regular kernels it was moved to Recommends to be more conservative.

Acked-by: Seth Forshee <[hidden email]>

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

APPLIED[azure]: [azure][PATCH] UBUNTU: [Debian] Don't abort installation if update-initramfs is missing

Marcelo Henrique Cerri
In reply to this post by Marcelo Henrique Cerri
Acked-by: Marcelo Henrique Cerri <[hidden email]>

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Applied: [azure/aws/gcp/gke/kvm/T/X/Z/A][PATCH v2] Please only recommend or suggest initramfs-tools | linux-initramfs-tool for kernels able to boot without initramfs

Kamal Mostafa-2
In reply to this post by Marcelo Henrique Cerri
Applied to all requested kernels.

 -Kamal

On Tue, Aug 08, 2017 at 04:16:02PM -0300, Marcelo Henrique Cerri wrote:

> BugLink: http://bugs.launchpad.net/bugs/1700972
>
> v2:
>
> The post/pre installation/removal scripts also need to be update to skip
> the initramfs generation.
>
> For linux-azure and linux-kvm, the patches only contain the changes to the
> scripts, since v1 was already applied (and not reverted).
>
> v1:
>
> Allow images to be created without the need of an initrd and also allow
> users to run without an initrd if they want to.
>
> The dependency to initramfs-tools was moved to Suggests for the cloud kernels,
> and for the regular kernels it was moved to Recommends to be more conservative.
>
> --
> Marcelo Henrique Cerri (1):
>   UBUNTU: [Debian] Don't depend on initramfs-tools
>
>  debian.master/control.d/flavour-control.stub | 4 ++--
>  debian/control-scripts/postinst              | 2 +-
>  debian/control-scripts/postrm                | 2 ++
>  debian/control-scripts/preinst               | 2 ++
>  debian/control-scripts/prerm                 | 1 +
>  5 files changed, 8 insertions(+), 3 deletions(-)
>
> --
> 2.7.4
>
>
> --
> kernel-team mailing list
> [hidden email]
> https://lists.ubuntu.com/mailman/listinfo/kernel-team

--
kernel-team mailing list
[hidden email]
https://lists.ubuntu.com/mailman/listinfo/kernel-team
Loading...