All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [alternative PATCH] fakeroot: fix build when libcap-dev is installed
@ 2016-01-04 18:33 Ricardo Martincoski
  2016-01-04 21:30 ` Arnout Vandecappelle
  2016-01-04 21:41 ` Peter Korsgaard
  0 siblings, 2 replies; 3+ messages in thread
From: Ricardo Martincoski @ 2016-01-04 18:33 UTC (permalink / raw)
  To: buildroot

Fixes https://bugs.busybox.net/show_bug.cgi?id=8541

Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com>
---
Alternative patch to https://patchwork.ozlabs.org/patch/562348/

We don't need libcap for buildroot, as Gustavo Zacarias explained in the past.
It seems reasonable to me to disable this detection too.

I can reproduce the original bug with libcap-dev in my Ubuntu 14.04.
Using this patch I cannot reproduce the bug anymore.
---
 package/fakeroot/fakeroot.mk | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/package/fakeroot/fakeroot.mk b/package/fakeroot/fakeroot.mk
index 837c2b9..168cd13 100644
--- a/package/fakeroot/fakeroot.mk
+++ b/package/fakeroot/fakeroot.mk
@@ -10,7 +10,10 @@ FAKEROOT_SITE = http://snapshot.debian.org/archive/debian/20141005T221953Z/pool/
 # Force capabilities detection off
 # For now these are process capabilities (faked) rather than file
 # so they're of no real use
-HOST_FAKEROOT_CONF_ENV = ac_cv_func_capset=no
+HOST_FAKEROOT_CONF_ENV = \
+	ac_cv_header_sys_capability_h=no \
+	ac_cv_func_capset=no
+
 FAKEROOT_LICENSE = GPLv3+
 FAKEROOT_LICENSE_FILES = COPYING
 
-- 
1.9.1

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

* [Buildroot] [alternative PATCH] fakeroot: fix build when libcap-dev is installed
  2016-01-04 18:33 [Buildroot] [alternative PATCH] fakeroot: fix build when libcap-dev is installed Ricardo Martincoski
@ 2016-01-04 21:30 ` Arnout Vandecappelle
  2016-01-04 21:41 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Arnout Vandecappelle @ 2016-01-04 21:30 UTC (permalink / raw)
  To: buildroot

On 04-01-16 19:33, Ricardo Martincoski wrote:
> Fixes https://bugs.busybox.net/show_bug.cgi?id=8541
> 
> Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com>

Acked-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
Tested-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be>
 (various cases with and without host- and system-libcap)

 In addition, this patch fixes the issue that the build of host-fakeroot will be
different depending on whether host-libcap was built before or not.

 Regards,
 Arnout

> ---
> Alternative patch to https://patchwork.ozlabs.org/patch/562348/
> 
> We don't need libcap for buildroot, as Gustavo Zacarias explained in the past.
> It seems reasonable to me to disable this detection too.
> 
> I can reproduce the original bug with libcap-dev in my Ubuntu 14.04.
> Using this patch I cannot reproduce the bug anymore.
> ---
>  package/fakeroot/fakeroot.mk | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/package/fakeroot/fakeroot.mk b/package/fakeroot/fakeroot.mk
> index 837c2b9..168cd13 100644
> --- a/package/fakeroot/fakeroot.mk
> +++ b/package/fakeroot/fakeroot.mk
> @@ -10,7 +10,10 @@ FAKEROOT_SITE = http://snapshot.debian.org/archive/debian/20141005T221953Z/pool/
>  # Force capabilities detection off
>  # For now these are process capabilities (faked) rather than file
>  # so they're of no real use
> -HOST_FAKEROOT_CONF_ENV = ac_cv_func_capset=no
> +HOST_FAKEROOT_CONF_ENV = \
> +	ac_cv_header_sys_capability_h=no \
> +	ac_cv_func_capset=no
> +
>  FAKEROOT_LICENSE = GPLv3+
>  FAKEROOT_LICENSE_FILES = COPYING
>  
> 


-- 
Arnout Vandecappelle                          arnout at mind be
Senior Embedded Software Architect            +32-16-286500
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:  7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF

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

* [Buildroot] [alternative PATCH] fakeroot: fix build when libcap-dev is installed
  2016-01-04 18:33 [Buildroot] [alternative PATCH] fakeroot: fix build when libcap-dev is installed Ricardo Martincoski
  2016-01-04 21:30 ` Arnout Vandecappelle
@ 2016-01-04 21:41 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2016-01-04 21:41 UTC (permalink / raw)
  To: buildroot

>>>>> "Ricardo" == Ricardo Martincoski <ricardo.martincoski@gmail.com> writes:

 > Fixes https://bugs.busybox.net/show_bug.cgi?id=8541
 > Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com>
> ---
 > Alternative patch to https://patchwork.ozlabs.org/patch/562348/

Committed, thanks.

-- 
Bye, Peter Korsgaard

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

end of thread, other threads:[~2016-01-04 21:41 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-01-04 18:33 [Buildroot] [alternative PATCH] fakeroot: fix build when libcap-dev is installed Ricardo Martincoski
2016-01-04 21:30 ` Arnout Vandecappelle
2016-01-04 21:41 ` Peter Korsgaard

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.