From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Thu, 16 Jul 2026 08:51:55 +0200 Received: from metis.whiteo.stw.pengutronix.de ([2a0a:edc0:2:b01:1d::104]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1wkFwp-004IgC-2E for lore@lore.pengutronix.de; Thu, 16 Jul 2026 08:51:55 +0200 Received: from [127.0.0.1] (helo=metis.whiteo.stw.pengutronix.de) by metis.whiteo.stw.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1wkFwn-0004rw-NE; Thu, 16 Jul 2026 08:51:53 +0200 Received: from mx1.white.stw.pengutronix.de ([2a0a:edc0:0:b01:1d::107]) by metis.whiteo.stw.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1wkFwW-0004rj-4k; Thu, 16 Jul 2026 08:51:36 +0200 Received: from drehscheibe.grey.stw.pengutronix.de (drehscheibe.grey.stw.pengutronix.de [IPv6:2a0a:edc0:0:c01:1d::a2]) (Authenticated sender: relay-from-drehscheibe.grey.stw.pengutronix.de) by mx1.white.stw.pengutronix.de (Postfix) with ESMTPSA id CC7D220225A; Thu, 16 Jul 2026 08:51:35 +0200 (CEST) Received: from pty.whiteo.stw.pengutronix.de ([2a0a:edc0:2:b01:1d::c5]) by drehscheibe.grey.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1wkFwV-0023iu-2O; Thu, 16 Jul 2026 08:51:35 +0200 Received: from mol by pty.whiteo.stw.pengutronix.de with local (Exim 4.98.2) (envelope-from ) id 1wkFwV-0000000EEz4-2eKt; Thu, 16 Jul 2026 08:51:35 +0200 Date: Thu, 16 Jul 2026 08:51:35 +0200 From: Michael Olbrich To: Sven =?utf-8?Q?P=C3=BCschel?= Message-ID: Mail-Followup-To: Sven =?utf-8?Q?P=C3=BCschel?= , ptxdist@pengutronix.de References: <20260714082724.446541-1-s.pueschel@pengutronix.de> <21eb1feb-efd4-4fc6-8342-069113dccce7@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <21eb1feb-efd4-4fc6-8342-069113dccce7@pengutronix.de> X-Sent-From: Pengutronix Hildesheim X-URL: http://www.pengutronix.de/ X-Accept-Language: de,en X-Accept-Content-Type: text/plain X-IRC: #ptxdist @freenode Subject: Re: [ptxdist] [PATCH v3] lvm2: move lvmdump and fsadm into separate options X-BeenThere: ptxdist@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Cc: ptxdist@pengutronix.de Sender: "ptxdist" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: ptxdist-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.whiteo.stw.pengutronix.de); SAEximRunCond expanded to false On Wed, Jul 15, 2026 at 01:06:11PM +0200, Sven Püschel wrote: > On 7/15/26 12:55 PM, Michael Olbrich wrote: > > On Tue, Jul 14, 2026 at 10:14:50AM +0200, Sven Püschel wrote: > > > Move lvmdump and the fsadm command into a separate option, as both > > > commands are bash scripts and therefore need bash as a dependency. > > > As these commands may not be used, we don't add bash as a dependency to > > > the much bigger LVM2_LVM_TOOLS option but instead add distinct options > > > for these commands. > > > > > > Signed-off-by: Sven Püschel > > > > > > --- > > > v3 > > > - added new options instead of adding bash to LVM2_LVM_TOOLS > > > - added RUNTIME flag to the bash dependency > > > v2 > > > - added > > > --- > > > rules/lvm2.in | 11 +++++++++++ > > > rules/lvm2.make | 7 ++++++- > > > 2 files changed, 17 insertions(+), 1 deletion(-) > > > > > > diff --git a/rules/lvm2.in b/rules/lvm2.in > > > index 3213d9c31..6eae0a2de 100644 > > > --- a/rules/lvm2.in > > > +++ b/rules/lvm2.in > > > @@ -8,6 +8,7 @@ menuconfig LVM2 > > > select READLINE > > > select NCURSES > > > select SYSTEMD if LVM2_SYSTEMD > > > + select BASH if (LVM2_FSADM || LVM2_LVMDUMP) && RUNTIME > > > prompt "lvm2 " > > > help > > > LVM2 refers to a new userspace toolset that provide logical volume management > > > @@ -32,6 +33,16 @@ config LVM2_STARTSCRIPT > > > depends on INITMETHOD_BBINIT > > > prompt "install /etc/init.d/lvm2" > > > +config LVM2_FSADM > > I think you need to select LVM2_LVM_TOOLS here. The scripts probably don't > > make sense without the actual binaries and the targetinstall stage will > > fail as well. > > These are guarded by the same `if·LVM2_LVM_TOOLS` that is also used for > `LVM2_STARTSCRIPT`. The diff just hides this fact. > > Therefore they cannot be selected if LVM2_LVM_TOOLS is disabled. Ah, right, that was not obvious from the diff context :-). > Btw. the makefile doesn't have the nested if, as it should be handled by the > kconfig. Correct. No need for nesting there. > > > > > + bool > > > + default y > > No 'default y' I think. These are tools typically used during development. > > And automatically enabling bash during an update is not so nice. > > My reasoning here was potential backwards compability (e.g. when you had all > tools installed previously you don't want to accidentially kick them out > during an update) and the fact that `LVM2_LVM_TOOLS` also has a `default y`. > > But I don't mind dropping this, if it's preferred to avoid accidentially > pulling bash as dependency on an update. I try to keep things compatible where it makes sense, but in this case, I'd rather trigger possible errors rather than pull in stuff silently. Michael > > > + prompt "install the fsadm command" > > > + > > > + > > > +config LVM2_LVMDUMP > > > + bool > > > + default y > > Same here. > > > > Michael > > > > > + prompt "install the lvmdump command" > > > endif > > > config LVM2_LIBDEVMAPPER_EVENT > > > diff --git a/rules/lvm2.make b/rules/lvm2.make > > > index 57773a1f7..b5aea9873 100644 > > > --- a/rules/lvm2.make > > > +++ b/rules/lvm2.make > > > @@ -110,10 +110,15 @@ $(STATEDIR)/lvm2.targetinstall: > > > @$(call install_copy, lvm2, 0, 0, 0755, -, /usr/sbin/dmsetup) > > > -ifdef PTXCONF_LVM2_LVM_TOOLS > > > +ifdef PTXCONF_LVM2_FSADM > > > @$(call install_copy, lvm2, 0, 0, 0755, -, /usr/sbin/fsadm) > > > +endif > > > + > > > +ifdef PTXCONF_LVM2_LVMDUMP > > > @$(call install_copy, lvm2, 0, 0, 0755, -, /usr/sbin/lvmdump) > > > +endif > > > +ifdef PTXCONF_LVM2_LVM_TOOLS > > > @$(call install_copy, lvm2, 0, 0, 0755, -, /usr/sbin/lvm) > > > @$(call install_link, lvm2, lvm, /usr/sbin/lvchange) > > > @$(call install_link, lvm2, lvm, /usr/sbin/lvconvert) > > > -- > > > 2.47.3 > > > > > > > > > > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |