[Buildroot] [PATCH 1/2] package/stress-ng: fix install commande
Peter Korsgaard
peter at korsgaard.com
Wed Sep 7 21:00:58 UTC 2016
>>>>> "Romain" == Romain Naour <romain.naour at gmail.com> writes:
> Don't use make install otherwise stress-ng will be rebuild without
> required link libraries if any. Furthermore, using INSTALL allow to
> set the file permission correcly on the target.
> Fixes:
> http://autobuild.buildroot.net/results/bba/bba4d0b635ab6849f969b8a0363ebe9f5fafd82e
> Signed-off-by: Romain Naour <romain.naour at gmail.com>
> ---
> package/stress-ng/stress-ng.mk | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
> diff --git a/package/stress-ng/stress-ng.mk b/package/stress-ng/stress-ng.mk
> index 6015a1e..e12895a 100644
> --- a/package/stress-ng/stress-ng.mk
> +++ b/package/stress-ng/stress-ng.mk
> @@ -15,8 +15,11 @@ define STRESS_NG_BUILD_CMDS
> $(TARGET_CONFIGURE_OPTS) $(MAKE) -C $(@D)
> endef
> +# Don't use make install otherwise stress-ng will be rebuild without
> +# required link libraries if any. Furthermore, using INSTALL allow to
> +# set the file permission correcly on the target.
> define STRESS_NG_INSTALL_TARGET_CMDS
> - $(TARGET_CONFIGURE_OPTS) $(MAKE) DESTDIR=$(TARGET_DIR) -C $(@D) install
> + $(INSTALL) -m 0755 -D $(@D)/stress-ng $(TARGET_DIR)/usr/bin/stress-ng
> endef
It is a bit funky that this happens, but I guess it is because of the
touch stress-ng.c in the rule for stress-vecmath.o. I'm not sure why
they do that.
Committed, thanks.
--
Bye, Peter Korsgaard
More information about the buildroot
mailing list