From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Wed, 15 Jul 2026 13:06:24 +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 1wjxRY-00415l-09 for lore@lore.pengutronix.de; Wed, 15 Jul 2026 13:06:24 +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 1wjxRV-0004PC-V2; Wed, 15 Jul 2026 13:06:21 +0200 Received: from ptz.office.stw.pengutronix.de ([2a0a:edc0:0:900:1d::77] helo=[127.0.0.1]) by metis.whiteo.stw.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1wjxRL-0004Le-Sl; Wed, 15 Jul 2026 13:06:11 +0200 Message-ID: <21eb1feb-efd4-4fc6-8342-069113dccce7@pengutronix.de> Date: Wed, 15 Jul 2026 13:06:11 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird To: Michael Olbrich References: <20260714082724.446541-1-s.pueschel@pengutronix.de> Content-Language: en-US From: =?UTF-8?Q?Sven_P=C3=BCschel?= In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit 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 Hi, 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. Btw. the makefile doesn't have the nested if, as it should be handled by the kconfig. > >> + 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. Sincerely     Sven > >> + 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 >> >> >>