fixeria submitted this change.

View Change

Approvals: Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve pespin: Looks good to me, approved
mobile: make LUA support configurable via --with-lua53

Change-Id: If440cee52410421034823a3749709e28c4136b95
---
M src/host/layer23/configure.ac
1 file changed, 13 insertions(+), 5 deletions(-)

diff --git a/src/host/layer23/configure.ac b/src/host/layer23/configure.ac
index f74efa2..50cb8ca 100644
--- a/src/host/layer23/configure.ac
+++ b/src/host/layer23/configure.ac
@@ -50,11 +50,19 @@


dnl optional dependencies
-PKG_CHECK_MODULES(LIBLUA, lua53, [
- WITH_LUA=1], [
- WITH_LUA=0])
-AC_SUBST([WITH_LUA])
-AM_CONDITIONAL([BUILD_LUA], test "x$WITH_LUA" = "x1")
+AC_ARG_WITH([lua53], [
+ AS_HELP_STRING([--with-lua53],
+ [Enable LUA scripting support @<:@default=check@:>@])
+])
+
+found_lua53=no
+AS_IF([test "x$with_lua53" != "xno"], [
+ PKG_CHECK_MODULES(LIBLUA, lua53, [found_lua53=yes], [found_lua53=no])
+ AS_IF([test "x$with_lua53" = "xyes" -a "x$found_lua53" = "xno"], [
+ AC_MSG_ERROR([lua53 support requested but pkg-config is unable to find it])
+ ])
+])
+AM_CONDITIONAL([BUILD_LUA], test "x$found_lua53" = "xyes")

dnl checks for header files
AC_HEADER_STDC

To view, visit change 30365. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: If440cee52410421034823a3749709e28c4136b95
Gerrit-Change-Number: 30365
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged