Control: tags -1 + patch

On Thu, Dec 19, 2024 at 03:52:55PM -0800, Tianon Gravi wrote:
>      > Regarding tasksel vs. Priority, the latter has a potential for a much
>      > wider impact: lots of Debian system are installed without d-i and/or
>      > tasksel, and most if not all would get the package via Priority.
>      (Think
>      > of all the tools building Debian images, chroots, containers, etc., on
>      > top of debootstrap/mmdebstrap/etc.)
> 
>      I'm not sure it's the case that most of those other systems install
>      Priority: standard.  Debootstrap certainly doesn't by itself, and I
>      don't think the debuerreotype tool for building OCI images does either.
>      In any case, your point still stands.  I'll re-assign this to general
>      for now, and we can discuss the options in a broader context.
> 
>      FWIW, we have added linux-sysctl-defaults to the sid/trixie VM images
>      built by the cloud team.
> 
>      noah
> 
>    I think "Priority: important" is probably appropriate here? 
>    
> [2]https://salsa.debian.org/installer-team/debootstrap/-/blob/c019a546a37b9284f0503c955d0f198044a7e2f0/scripts/debian-common#L29
>    Then it's installed by default in standard / d-i, but not part of minbase,
>    buildd, etc.
>    (It wouldn't affect/do anything in most container images or runtimes
>    anyhow, which is also an argument for it being harmless to include. 🤷)

Hi kernel team.  Per the earlier discussion in this bug, please raise
the priority of linux-sysctl-defaults to important in order to ensure
that it's present by default in typical Debian installations.

Patch is attached.

Thanks
noah

>From e88af10427c974c74bceefc1f44fa5be81f878e6 Mon Sep 17 00:00:00 2001
From: Noah Meyerhans <fr...@morgul.net>
Date: Sun, 22 Dec 2024 09:36:15 -0500
Subject: [PATCH] Set Priority: important for linux-sysctl-defaults

linux-sysctl-defaults should be installed by default on the typical Debian
system.  Although it is recommended by several components of the default
install, Recommends are not applied during debootstrap, so it isn't installed.
Raising the priority to important ensures that it's installed in expected
places.

Closes: #1090811
---
 debian/control | 1 +
 1 file changed, 1 insertion(+)

diff --git a/debian/control b/debian/control
index cc95624..a6f42bc 100644
--- a/debian/control
+++ b/debian/control
@@ -21,6 +21,7 @@ Description: Linux image base package
 
 Package: linux-sysctl-defaults
 Architecture: all
+Priority: important
 Depends: ${misc:Depends}
 Multi-Arch: foreign
 Description: default sysctl configuration for Linux
-- 
2.39.5

Reply via email to