Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] libcap: disable progs altogether
@ 2012-10-29  0:34 Gustavo Zacarias
  2012-10-30 22:55 ` Arnout Vandecappelle
  0 siblings, 1 reply; 2+ messages in thread
From: Gustavo Zacarias @ 2012-10-29  0:34 UTC (permalink / raw)
  To: buildroot

Disable progs in the Makefile directly instead of building and then
removing them.
This is because they use fork() and can cause build failures on nommu.

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
 package/libcap/libcap.mk |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/package/libcap/libcap.mk b/package/libcap/libcap.mk
index b922e5a..ac73dcf 100644
--- a/package/libcap/libcap.mk
+++ b/package/libcap/libcap.mk
@@ -12,7 +12,9 @@ LIBCAP_SOURCE = libcap2_$(LIBCAP_VERSION).orig.tar.gz
 LIBCAP_DEPENDENCIES = host-libcap
 LIBCAP_INSTALL_STAGING = YES
 
+# progs use fork()
 define LIBCAP_BUILD_CMDS
+	$(SED) '/-C progs/d' $(@D)/Makefile
 	$(TARGET_MAKE_ENV) $(TARGET_CONFIGURE_OPTS) $(MAKE) -C $(@D) \
 		LIBATTR=no BUILD_CC="$(HOSTCC)" BUILD_CFLAGS="$(HOST_CFLAGS)"
 endef
@@ -25,7 +27,6 @@ endef
 define LIBCAP_INSTALL_TARGET_CMDS
 	$(TARGET_MAKE_ENV) $(MAKE) -C $(@D) LIBATTR=no DESTDIR=$(TARGET_DIR) \
 		prefix=/usr lib=lib install
-	rm -f $(addprefix $(TARGET_DIR)/usr/sbin/,capsh getpcaps)
 endef
 
 define HOST_LIBCAP_BUILD_CMDS
-- 
1.7.8.6

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [Buildroot] [PATCH] libcap: disable progs altogether
  2012-10-29  0:34 [Buildroot] [PATCH] libcap: disable progs altogether Gustavo Zacarias
@ 2012-10-30 22:55 ` Arnout Vandecappelle
  0 siblings, 0 replies; 2+ messages in thread
From: Arnout Vandecappelle @ 2012-10-30 22:55 UTC (permalink / raw)
  To: buildroot

On 10/29/12 01:34, Gustavo Zacarias wrote:
> +# progs use fork()
>   define LIBCAP_BUILD_CMDS
> +	$(SED) '/-C progs/d' $(@D)/Makefile
>   	$(TARGET_MAKE_ENV) $(TARGET_CONFIGURE_OPTS) $(MAKE) -C $(@D) \

  Could you do this in a post-patch hook instead? It's more verbose, but
more appropriate I think.

  Regards,
  Arnout
-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
Essensium/Mind                                     http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium                BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-10-30 22:55 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-10-29  0:34 [Buildroot] [PATCH] libcap: disable progs altogether Gustavo Zacarias
2012-10-30 22:55 ` Arnout Vandecappelle

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox