All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] libev: don't install event.h
@ 2014-08-26 11:36 Gustavo Zacarias
  2014-08-26 11:51 ` Vicente Olivert Riera
  2014-08-26 21:49 ` Peter Korsgaard
  0 siblings, 2 replies; 3+ messages in thread
From: Gustavo Zacarias @ 2014-08-26 11:36 UTC (permalink / raw)
  To: buildroot

It's a 'compatibility layer' with libevent which we:
1) Don't need
2) Possibly outdated with respect to libevent2
3) Causes build failures

Fixes:
http://autobuild.buildroot.net/results/451/4510fe02ef3497803ed27bf339dca07b3b073c10/

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
 package/libev/libev.mk | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/package/libev/libev.mk b/package/libev/libev.mk
index a6ff3a9..dc42858 100644
--- a/package/libev/libev.mk
+++ b/package/libev/libev.mk
@@ -10,4 +10,11 @@ LIBEV_INSTALL_STAGING = YES
 LIBEV_LICENSE = BSD-2c or GPLv2+
 LIBEV_LICENSE_FILES = LICENSE
 
+# The 'compatibility' event.h header conflicts with libevent
+# It's completely unnecessary for BR packages so remove it
+define LIBEV_DISABLE_EVENT_H_INSTALL
+	$(SED) 's/ event.h//' $(@D)/Makefile.in
+endef
+LIBEV_POST_PATCH_HOOKS += LIBEV_DISABLE_EVENT_H_INSTALL
+
 $(eval $(autotools-package))
-- 
1.8.5.5

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

* [Buildroot] [PATCH] libev: don't install event.h
  2014-08-26 11:36 [Buildroot] [PATCH] libev: don't install event.h Gustavo Zacarias
@ 2014-08-26 11:51 ` Vicente Olivert Riera
  2014-08-26 21:49 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Vicente Olivert Riera @ 2014-08-26 11:51 UTC (permalink / raw)
  To: buildroot

On 08/26/2014 12:36 PM, Gustavo Zacarias wrote:
> It's a 'compatibility layer' with libevent which we:
> 1) Don't need
> 2) Possibly outdated with respect to libevent2
> 3) Causes build failures
>
> Fixes:
> http://autobuild.buildroot.net/results/451/4510fe02ef3497803ed27bf339dca07b3b073c10/
>
> Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
> ---
>   package/libev/libev.mk | 7 +++++++
>   1 file changed, 7 insertions(+)
>
> diff --git a/package/libev/libev.mk b/package/libev/libev.mk
> index a6ff3a9..dc42858 100644
> --- a/package/libev/libev.mk
> +++ b/package/libev/libev.mk
> @@ -10,4 +10,11 @@ LIBEV_INSTALL_STAGING = YES
>   LIBEV_LICENSE = BSD-2c or GPLv2+
>   LIBEV_LICENSE_FILES = LICENSE
>
> +# The 'compatibility' event.h header conflicts with libevent
> +# It's completely unnecessary for BR packages so remove it
> +define LIBEV_DISABLE_EVENT_H_INSTALL
> +	$(SED) 's/ event.h//' $(@D)/Makefile.in
> +endef
> +LIBEV_POST_PATCH_HOOKS += LIBEV_DISABLE_EVENT_H_INSTALL
> +
>   $(eval $(autotools-package))
>

Tested-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>

-- 
Vincent

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

* [Buildroot] [PATCH] libev: don't install event.h
  2014-08-26 11:36 [Buildroot] [PATCH] libev: don't install event.h Gustavo Zacarias
  2014-08-26 11:51 ` Vicente Olivert Riera
@ 2014-08-26 21:49 ` Peter Korsgaard
  1 sibling, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2014-08-26 21:49 UTC (permalink / raw)
  To: buildroot

>>>>> "Gustavo" == Gustavo Zacarias <gustavo@zacarias.com.ar> writes:

 > It's a 'compatibility layer' with libevent which we:
 > 1) Don't need
 > 2) Possibly outdated with respect to libevent2
 > 3) Causes build failures

 > Fixes:
 > http://autobuild.buildroot.net/results/451/4510fe02ef3497803ed27bf339dca07b3b073c10/

 > Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>

Committed, thanks.

-- 
Bye, Peter Korsgaard

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

end of thread, other threads:[~2014-08-26 21:49 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-26 11:36 [Buildroot] [PATCH] libev: don't install event.h Gustavo Zacarias
2014-08-26 11:51 ` Vicente Olivert Riera
2014-08-26 21:49 ` 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.