From: "Darrick J. Wong" <djwong@kernel.org>
To: Bastian Germann <bage@debian.org>
Cc: linux-xfs@vger.kernel.org, Chris Hofstaedtler <zeha@debian.org>
Subject: Re: [PATCH 1/1] Install files into UsrMerged layout
Date: Wed, 12 Jun 2024 11:08:43 -0700 [thread overview]
Message-ID: <20240612180843.GE2764752@frogsfrogsfrogs> (raw)
In-Reply-To: <20240612173551.6510-2-bage@debian.org>
On Wed, Jun 12, 2024 at 07:35:05PM +0200, Bastian Germann wrote:
> From: Chris Hofstaedtler <zeha@debian.org>
>
> Signed-off-by: Chris Hofstaedtler <zeha@debian.org>
> Signed-off-by: Bastian Germann <bage@debian.org>
> ---
> configure.ac | 19 ++-----------------
> debian/local/initramfs.hook | 2 +-
> 2 files changed, 3 insertions(+), 18 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index da30fc5c..a532d90d 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -113,23 +113,8 @@ esac
> #
> test -n "$multiarch" && enable_lib64=no
>
> -#
> -# Some important tools should be installed into the root partitions.
> -#
> -# Check whether exec_prefix=/usr: and install them to /sbin in that
> -# case. If the user chooses a different prefix assume they just want
> -# a local install for testing and not a system install.
> -#
> -case $exec_prefix:$prefix in
> -NONE:NONE | NONE:/usr | /usr:*)
> - root_sbindir='/sbin'
> - root_libdir="/${base_libdir}"
> - ;;
> -*)
> - root_sbindir="${sbindir}"
> - root_libdir="${libdir}"
> - ;;
> -esac
> +root_sbindir="${sbindir}"
> +root_libdir="${libdir}"
Should we get rid of $root_sbindir, $root_libdir, PKG_ROOT_LIB_DIR, and
PKG_ROOT_SBIN_DIR while we're at it? That will break anyone who hasn't
done the /usr merge yet, but how many distros still want
/sbin/xfs_repair? opensuse and the rhel variants seem to have moved
that to /usr/sbin/ years ago.
--D
>
> AC_SUBST([root_sbindir])
> AC_SUBST([root_libdir])
> diff --git a/debian/local/initramfs.hook b/debian/local/initramfs.hook
> index 5b24eaec..eac7e79e 100644
> --- a/debian/local/initramfs.hook
> +++ b/debian/local/initramfs.hook
> @@ -45,7 +45,7 @@ rootfs_type() {
> . /usr/share/initramfs-tools/hook-functions
>
> if [ "$(rootfs_type)" = "xfs" ]; then
> - copy_exec /sbin/xfs_repair
> + copy_exec /usr/sbin/xfs_repair
> copy_exec /usr/sbin/xfs_db
> copy_exec /usr/sbin/xfs_metadump
> fi
> --
> 2.45.2
>
>
next prev parent reply other threads:[~2024-06-12 18:08 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-12 17:35 [PATCH 0/1] Forward: Install files into UsrMerged layout Bastian Germann
2024-06-12 17:35 ` [PATCH 1/1] " Bastian Germann
2024-06-12 18:08 ` Darrick J. Wong [this message]
2024-06-12 18:58 ` Chris Hofstaedtler
2024-06-12 20:07 ` Darrick J. Wong
2024-06-12 19:00 ` [PATCH v2 " Chris Hofstaedtler
2024-06-12 20:03 ` Darrick J. Wong
2024-06-12 20:10 ` Chris Hofstaedtler
2024-06-19 21:48 ` [PATCH v3] Remove support for split-/usr installs Chris Hofstaedtler
2024-06-20 18:34 ` Darrick J. Wong
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20240612180843.GE2764752@frogsfrogsfrogs \
--to=djwong@kernel.org \
--cc=bage@debian.org \
--cc=linux-xfs@vger.kernel.org \
--cc=zeha@debian.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox