Skip to content

Support bin-sbin merge on Fedora#18373

Merged
behlendorf merged 1 commit intoopenzfs:masterfrom
Lalufu:lalufu/fedora-unified-bin-sbin
Mar 31, 2026
Merged

Support bin-sbin merge on Fedora#18373
behlendorf merged 1 commit intoopenzfs:masterfrom
Lalufu:lalufu/fedora-unified-bin-sbin

Conversation

@Lalufu
Copy link
Copy Markdown
Contributor

@Lalufu Lalufu commented Mar 25, 2026

Starting with Fedora 42, Fedora has been working on merging /bin and /sbin directories. See
https://fedoraproject.org/wiki/Changes/Unify_bin_and_sbin

To support this, make sure we do not put files into */sbin directories on these distributions by respecting the distribution set value of %{_sbindir}.

In addition, explicitly set mounthelperdir, which affects the placement of mount.zfs, which does not respect %{_sbindir} by default. Making it point to %{_sbindir} will allow it to work correctly on Fedora, while keeping its previous directory of /sbin on all other distributions.

Note that files that used to reside in */sbin directories on Fedora will stay accessible under these paths, as the distribution maintains symlinks. No changes are needed to external scripts invoking these binaries.

Starting with Fedora 42, Fedora has been working on merging /bin and
/sbin directories. See
https://fedoraproject.org/wiki/Changes/Unify_bin_and_sbin

To support this, make sure we do not put files into */sbin directories
on these distributions by respecting the distribution set value of
%{_sbindir}.

In addition, explicitly set `mounthelperdir`, which affects the
placement of `mount.zfs`, which does not respect %{_sbindir} by
default. Making it point to %{_sbindir} will allow it to work correctly
on Fedora, while keeping its previous directory of `/sbin` on all other
distributions.

Note that files that used to reside in */sbin directories on Fedora will
stay accessible under these paths, as the distribution maintains
symlinks. No changes are needed to external scripts invoking these
binaries.

Signed-off-by: Ralf Ertzinger <ralf@skytale.net>
Copy link
Copy Markdown
Contributor

@behlendorf behlendorf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the heads up on this change.

@behlendorf behlendorf added the Status: Accepted Ready to integrate (reviewed, tested) label Mar 31, 2026
@behlendorf behlendorf merged commit 869d805 into openzfs:master Mar 31, 2026
46 of 57 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status: Accepted Ready to integrate (reviewed, tested)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants