diff mbox series

[1/8] iputils: just set bindir, not prefix

Message ID 20251210183308.4022909-1-ross.burton@arm.com
State Accepted, archived
Commit a4e0a71c391ee6a411c817186f38559c58c3c21b
Headers show
Series [1/8] iputils: just set bindir, not prefix | expand

Commit Message

Ross Burton Dec. 10, 2025, 6:33 p.m. UTC
We just want to put the binaries into base_bindir and nothing else, so
set bindir instead of prefix.

This is currently moot as iputils only installs binaries, but this is
the right thing to do and allows further cleanups.

Signed-off-by: Ross Burton <ross.burton@arm.com>
---
 meta/recipes-extended/iputils/iputils_20250605.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Mathieu Dubois-Briand Dec. 11, 2025, 2:35 p.m. UTC | #1
On Wed Dec 10, 2025 at 7:33 PM CET, Ross Burton via lists.openembedded.org wrote:
> We just want to put the binaries into base_bindir and nothing else, so
> set bindir instead of prefix.
>
> This is currently moot as iputils only installs binaries, but this is
> the right thing to do and allows further cleanups.
>
> Signed-off-by: Ross Burton <ross.burton@arm.com>
> ---

Hi Ross,

We got a configure error on iputils:

oeqa.utils.subprocesstweak.OETestCalledProcessError: Command '. /srv/pokybuild/yocto-worker/pkgman-non-rpm/build/build/tmp/work/qemux86_64-poky-linux/core-image-minimal/1.0/testimage-sdk/environment-setup-x86-64-v3-poky-linux > /dev/null; meson setup --warnlevel 1 /srv/pokybuild/yocto-worker/pkgman-non-rpm/build/build/tmp/work/qemux86_64-poky-linux/core-image-minimal/1.0/testimage-sdk/iputilsfzhd8meu/build /srv/pokybuild/yocto-worker/pkgman-non-rpm/build/build/tmp/work/qemux86_64-poky-linux/core-image-minimal/1.0/testimage-sdk/iputilsfzhd8meu/iputils-20250605 ;' returned non-zero exit status 1.
...
tmp/work/qemux86_64-poky-linux/core-image-minimal/1.0/testimage-sdk/iputilsfzhd8meu/iputils-20250605/meson.build:103:14: ERROR: C shared or static library 'cap' not found

https://autobuilder.yoctoproject.org/valkyrie/#/builders/67/builds/2868

Can you have a loot at this?

Thanks,
Mathieu
Ross Burton Dec. 11, 2025, 5:56 p.m. UTC | #2
On 11 Dec 2025, at 14:35, Mathieu Dubois-Briand <mathieu.dubois-briand@bootlin.com> wrote:
> We got a configure error on iputils:
> 
> oeqa.utils.subprocesstweak.OETestCalledProcessError: Command '. /srv/pokybuild/yocto-worker/pkgman-non-rpm/build/build/tmp/work/qemux86_64-poky-linux/core-image-minimal/1.0/testimage-sdk/environment-setup-x86-64-v3-poky-linux > /dev/null; meson setup --warnlevel 1 /srv/pokybuild/yocto-worker/pkgman-non-rpm/build/build/tmp/work/qemux86_64-poky-linux/core-image-minimal/1.0/testimage-sdk/iputilsfzhd8meu/build /srv/pokybuild/yocto-worker/pkgman-non-rpm/build/build/tmp/work/qemux86_64-poky-linux/core-image-minimal/1.0/testimage-sdk/iputilsfzhd8meu/iputils-20250605 ;' returned non-zero exit status 1.
> ...
> tmp/work/qemux86_64-poky-linux/core-image-minimal/1.0/testimage-sdk/iputilsfzhd8meu/iputils-20250605/meson.build:103:14: ERROR: C shared or static library 'cap' not found

That will teach me to test in a core-image-base instead of -minimal.

v2 sent.

Ross
diff mbox series

Patch

diff --git a/meta/recipes-extended/iputils/iputils_20250605.bb b/meta/recipes-extended/iputils/iputils_20250605.bb
index 31eb51e56d4..baeda556592 100644
--- a/meta/recipes-extended/iputils/iputils_20250605.bb
+++ b/meta/recipes-extended/iputils/iputils_20250605.bb
@@ -26,7 +26,7 @@  PACKAGECONFIG[docs] = "-DBUILD_HTML_MANS=true -DBUILD_MANS=true,-DBUILD_HTML_MAN
 
 inherit meson update-alternatives pkgconfig
 
-EXTRA_OEMESON += "--prefix=${root_prefix}/ -DSKIP_TESTS=true"
+EXTRA_OEMESON += "--bindir=${base_bindir} -DSKIP_TESTS=true"
 
 ALTERNATIVE_PRIORITY = "100"