Skip to contentSkip to conversation

ulogd: fix build failure

PATCH New Gustavo Zacarias 2 participants 4 messages Nov 4, 2012

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 ulogd: fix build failure

Fix Peter's reported build failure:
http://autobuild.buildroot.net/results/1dcf63799937f2216e3c712af8880ad431dbd47c/build-end.log
This is because the configure script uses absolute paths to detect dbi, mysql and pgsql, so forcibly disable them.
Also add the optional sqlite dependency when it's selected.

---
 package/ulogd/ulogd.mk |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)
package/ulogd/ulogd.mk +2 -1
index 7d31b73..4620bbd 100644
@@ -7,10 +7,11 @@
 ULOGD_VERSION = 2.0.1
 ULOGD_SOURCE = ulogd-$(ULOGD_VERSION).tar.bz2
 ULOGD_SITE = http://www.netfilter.org/projects/ulogd/files
+ULOGD_CONF_OPT = --with-dbi=no --with-mysql=no --with-pgsql=no
 ULOGD_AUTORECONF = YES
 ULOGD_DEPENDENCIES = host-pkgconf \
 	libmnl libnetfilter_acct libnetfilter_conntrack libnetfilter_log \
-	libnfnetlink
+	libnfnetlink $(if $(BR2_PACKAGE_SQLITE),sqlite)
 ULOGD_LICENSE = GPLv2
 ULOGD_LICENSE_FILES = COPYING
 
Signature · 1 line
1.7.8.6
#1 lore raw

PATCH 1/1 ulogd: fix build failure

Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Gustavo" == Gustavo Zacarias <[email protected]> writes:

Gustavo> Fix Peter's reported build failure:
Gustavo> http://autobuild.buildroot.net/results/1dcf63799937f2216e3c712af8880ad431dbd47c/build-end.log
Gustavo> This is because the configure script uses absolute paths to detect dbi, Gustavo> mysql and pgsql, so forcibly disable them.
Gustavo> Also add the optional sqlite dependency when it's selected.

Gustavo> Signed-off-by: Gustavo Zacarias <[email protected]> Gustavo> ---

 Gustavo>  package/ulogd/ulogd.mk |    3 ++-

Gustavo> 1 files changed, 2 insertions(+), 1 deletions(-)

Gustavo> diff --git a/package/ulogd/ulogd.mk b/package/ulogd/ulogd.mk Gustavo> index 7d31b73..4620bbd 100644
Gustavo> --- a/package/ulogd/ulogd.mk
Gustavo> +++ b/package/ulogd/ulogd.mk
Gustavo> @@ -7,10 +7,11 @@

 Gustavo>  ULOGD_VERSION = 2.0.1
 Gustavo>  ULOGD_SOURCE = ulogd-$(ULOGD_VERSION).tar.bz2
 Gustavo>  ULOGD_SITE = http://www.netfilter.org/projects/ulogd/files

Gustavo> +ULOGD_CONF_OPT = --with-dbi=no --with-mysql=no --with-pgsql=no

We do have mysql_client though - Any reason to not build support for it if enabled?

Signature · 1 line
Bye, Peter Korsgaard
#2 lore raw

PATCH 1/1 ulogd: fix build failure

Quoted 2 lines · Peter Korsgaard

We do have mysql_client though - Any reason to not build support for it if enabled?

I'm still running a fever from flu and i don't have the necessary brain power to test it right at this moment :)
Regards.

#3 lore raw

PATCH 1/1 ulogd: fix build failure

Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line

"Gustavo" == Gustavo Zacarias <[email protected]> writes:

Quoted 2 lines · Peter Korsgaard
Quoted 2 lines

We do have mysql_client though - Any reason to not build support for it if enabled?

Gustavo> I'm still running a fever from flu and i don't have the Gustavo> necessary brain power to test it right at this moment :)

Fair enough, I'll leave it for now then - Committed, thanks.

Signature · 1 line
Bye, Peter Korsgaard