public inbox for linux-xfs@vger.kernel.org
 help / color / mirror / Atom feed
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
> 
> 

  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