Skip to contentSkip to conversation

libev: don't install event.h

PATCH New Gustavo Zacarias 3 participants 3 messages Aug 26, 2014

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 libev: don't install event.h

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/

---
 package/libev/libev.mk | 7 +++++++
 1 file changed, 7 insertions(+)
package/libev/libev.mk +7 -0
index a6ff3a9..dc42858 100644
@@ -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))
Signature · 1 line
1.8.5.5
#1 lore raw

PATCH 1/1 libev: don't install event.h

Quoted 20 lines · Gustavo Zacarias

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/

---
  package/libev/libev.mk | 7 +++++++
  1 file changed, 7 insertions(+)
package/libev/libev.mk +0 -0
index a6ff3a9..dc42858 100644
@@ -10,4 +10,11 @@ LIBEV_INSTALL_STAGING = YES
  LIBEV_LICENSE = BSD-2c or GPLv2+

package/libev/libev.mk LIBEV_INSTALL_STAGING = YES · line 20

Quoted 10 lines
package/libev/libev.mk +7 -0
  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))
Signature · 1 line
Vincent
#2 lore raw

PATCH 1/1 libev: don't install event.h

In reply to Gustavo Zacarias
Quoted 11 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Gustavo" == Gustavo Zacarias <[email protected]> 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/

Committed, thanks.

Signature · 1 line
Bye, Peter Korsgaard