Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

meson: install all binaries in $PREFIX/bin #138

Merged
merged 1 commit into from
Jan 24, 2025
Merged

Conversation

rjarry
Copy link
Collaborator

@rjarry rjarry commented Jan 23, 2025

Most distros are now merging /usr/sbin into /usr/bin. Let's not make it more confusing by installing in /usr/sbin upstream and have a different install location when packaged downstream.

This has the benefit of fixing packaging on Fedora which recently changed their %{_sbindir} macro to point to /usr/bin. Unfortunately, the install_dir in meson.build was wrongfully set to explicit 'sbin' instead of get_option('sbindir').

Link: https://bugzilla.redhat.com/show_bug.cgi?id=2340579
Link: https://koji.fedoraproject.org/koji/taskinfo?taskID=127983504
Link: https://fedoraproject.org/wiki//Changes/Unify_bin_and_sbin

Most distros are now merging /usr/sbin into /usr/bin. Let's not make it
more confusing by installing in /usr/sbin upstream and have a different
install location when packaged downstream.

This has the benefit of fixing packaging on Fedora which recently
changed their %{_sbindir} macro to point to /usr/bin. Unfortunately, the
install_dir in meson.build was wrongfully set to explicit 'sbin' instead
of get_option('sbindir').

Link: https://bugzilla.redhat.com/show_bug.cgi?id=2340579
Link: https://koji.fedoraproject.org/koji/taskinfo?taskID=127983504
Link: https://fedoraproject.org/wiki//Changes/Unify_bin_and_sbin
Signed-off-by: Robin Jarry <[email protected]>
@rjarry rjarry merged commit cd843af into DPDK:main Jan 24, 2025
4 checks passed
@rjarry rjarry deleted the no-sbin branch January 24, 2025 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants