[lxc-devel] [PATCH] configure.ac: Make our configure more consistent
Stéphane Graber
stgraber at ubuntu.com
Sun Dec 15 23:48:38 UTC 2013
This commit does the following changes:
- Disable rpath by default
- Switch all of our options to --enable-FEATURE in the help
- Add auto-detection of libcap availability
- Add auto-detection of python3 availability
- Always specify the default value in --help
- Add a configuration overview at the end
Signed-off-by: Stéphane Graber <stgraber at ubuntu.com>
---
configure.ac | 152 ++++++++++++++++++++++++++++++++++++++---------------------
1 file changed, 98 insertions(+), 54 deletions(-)
diff --git a/configure.ac b/configure.ac
index 575493f..c8a08a9 100644
--- a/configure.ac
+++ b/configure.ac
@@ -72,13 +72,13 @@ AM_CONDITIONAL([DISTRO_UBUNTU], [test "x$with_distro" = "xubuntu"])
# Allow disabling rpath
AC_ARG_ENABLE([rpath],
- [AC_HELP_STRING([--disable-rpath], [do not set rpath in executables])],
- [], [enable_rpath=yes])
+ [AC_HELP_STRING([--enable-rpath], [set rpath in executables [default=no]])],
+ [], [enable_rpath=no])
AM_CONDITIONAL([ENABLE_RPATH], [test "x$enable_rpath" = "xyes"])
# Documentation (manpages)
AC_ARG_ENABLE([doc],
- [AC_HELP_STRING([--enable-doc], [make mans (requires docbook2man or docbook2x-man to be installed) [default=auto]])],
+ [AC_HELP_STRING([--enable-doc], [make man pages [default=auto]])],
[], [enable_doc=auto])
if test "x$enable_doc" = "xyes" -o "x$enable_doc" = "xauto"; then
@@ -95,11 +95,13 @@ if test "x$enable_doc" = "xyes" -o "x$enable_doc" = "xauto"; then
if test -n "${db2xman}"; then
AC_MSG_RESULT([${db2xman}])
+ enable_doc="yes"
else
AC_MSG_RESULT([no])
if test "x$enable_doc" = "xyes"; then
- AC_MSG_ERROR([docbook2x-man required by man request, but not found])
+ AC_MSG_ERROR([docbook2x-man is required, but could not be found])
fi
+ enable_doc="no"
fi
AC_SUBST(db2xman)
@@ -117,22 +119,31 @@ AC_SUBST(docdtd)
# Documentation (API)
AC_ARG_ENABLE([api-docs],
[AC_HELP_STRING([--enable-api-docs],
- [make API documentation (requires doxygen to be installed) [default=auto]])],
+ [make API documentation [default=auto]])],
[], [enable_api_docs=auto])
if test "x$enable_api_docs" = "xyes" -o "x$enable_api_docs" = "xauto"; then
- AC_CHECK_PROGS([HAVE_DOXYGEN],[doxygen])
- AC_SUBST([HAVE_DOXYGEN])
+ AC_CHECK_PROGS([HAVE_DOXYGEN],[doxygen])
+ AC_SUBST([HAVE_DOXYGEN])
+
+ if test "x$HAVE_DOXYGEN" != "x"; then
+ enable_api_docs="yes"
+ else
+ if test "x$enable_api_docs" = "xyes"; then
+ AC_MSG_ERROR([doxygen is required, but could not be found])
+ fi
+ enable_api_docs="no"
+ fi
fi
-AM_CONDITIONAL([ENABLE_API_DOCS], [test ! -z "$HAVE_DOXYGEN"])
+AM_CONDITIONAL([ENABLE_API_DOCS], [test "x$HAVE_DOXYGEN" != "x"])
# Apparmor
AC_ARG_ENABLE([apparmor],
- [AC_HELP_STRING([--enable-apparmor], [enable apparmor])],
- [], [enable_apparmor=check])
+ [AC_HELP_STRING([--enable-apparmor], [enable apparmor support [default=auto]])],
+ [], [enable_apparmor=auto])
-if test "$enable_apparmor" = "check" ; then
+if test "$enable_apparmor" = "auto" ; then
AC_CHECK_LIB([apparmor],[aa_change_profile],[enable_apparmor=yes], [enable_apparmor=no])
fi
AM_CONDITIONAL([ENABLE_APPARMOR], [test "x$enable_apparmor" = "xyes"])
@@ -146,10 +157,10 @@ AM_COND_IF([ENABLE_APPARMOR],
# SELinux
AC_ARG_ENABLE([selinux],
- [AC_HELP_STRING([--enable-selinux], [enable SELinux support])],
- [], [enable_selinux=check])
+ [AC_HELP_STRING([--enable-selinux], [enable SELinux support [default=auto]])],
+ [], [enable_selinux=auto])
-if test "x$enable_selinux" = xcheck; then
+if test "x$enable_selinux" = xauto; then
AC_CHECK_LIB([selinux],[setexeccon_raw],[enable_selinux=yes],[enable_selinux=no])
fi
AM_CONDITIONAL([ENABLE_SELINUX], [test "x$enable_selinux" = "xyes"])
@@ -160,10 +171,10 @@ AM_COND_IF([ENABLE_SELINUX],
# Seccomp syscall filter
AC_ARG_ENABLE([seccomp],
- [AC_HELP_STRING([--enable-seccomp], [enable seccomp])],
- [], [enable_seccomp=check])
+ [AC_HELP_STRING([--enable-seccomp], [enable seccomp support [default=auto]])],
+ [], [enable_seccomp=auto])
-if test "$enable_seccomp" = "check" ; then
+if test "x$enable_seccomp" = "xauto" ; then
AC_CHECK_LIB([seccomp],[seccomp_init],[enable_seccomp=yes],[enable_seccomp=no])
fi
AM_CONDITIONAL([ENABLE_SECCOMP], [test "x$enable_seccomp" = "xyes"])
@@ -173,19 +184,41 @@ AM_COND_IF([ENABLE_SECCOMP],
AC_CHECK_LIB([seccomp], [seccomp_init],[],[AC_MSG_ERROR([You must install the seccomp development package in order to compile lxc])])
AC_SUBST([SECCOMP_LIBS], [-lseccomp])])
+# Linux capabilities
+AC_ARG_ENABLE([capabilities],
+ [AC_HELP_STRING([--enable-capabilities], [enable kernel capabilities support [default=auto]])],
+ [], [enable_capabilities=auto])
+
+if test "x$enable_capabilities" = "xauto"; then
+ AC_CHECK_LIB([cap],[cap_set_proc],[enable_capabilities=yes],[enable_capabilities=no])
+fi
+AM_CONDITIONAL([ENABLE_CAP], [test "x$enable_capabilities" = "xyes"])
+
+AM_COND_IF([ENABLE_CAP],
+ [AC_CHECK_LIB(cap,cap_set_proc,[],[AC_MSG_ERROR([You are missing libcap support.])])
+ AC_SUBST([CAP_LIBS], [-lcap])])
+
# HAVE_SCMP_FILTER_CTX=1 will tell us we have libseccomp api >= 1.0.0
-AC_CHECK_TYPES([scmp_filter_ctx], [], [], [#include <seccomp.h>])
+AC_CHECK_TYPES([scmp_filter_ctx], [], [], [[#include <seccomp.h>]])
# Configuration examples
AC_ARG_ENABLE([examples],
- [AC_HELP_STRING([--disable-examples], [do not install configuration examples])],
+ [AC_HELP_STRING([--enable-examples], [install examples [default=yes]])],
[], [enable_examples=yes])
AM_CONDITIONAL([ENABLE_EXAMPLES], [test "x$enable_examples" = "xyes"])
+# We need pkg-config
+PKG_PROG_PKG_CONFIG
+
# Python3 module and scripts
AC_ARG_ENABLE([python],
- [AC_HELP_STRING([--enable-python], [enable python binding])],
- [enable_python=yes], [enable_python=no])
+ [AC_HELP_STRING([--enable-python], [enable python binding [default=auto]])],
+ [], [enable_python=auto])
+
+if test "x$enable_python" = "xauto"; then
+ PKG_CHECK_MODULES([PYTHONDEV], [python3 >= 3.2],[enable_python=yes],[enable_python=no])
+fi
+
AM_CONDITIONAL([ENABLE_PYTHON], [test "x$enable_python" = "xyes"])
AM_COND_IF([ENABLE_PYTHON],
@@ -195,8 +228,8 @@ AM_COND_IF([ENABLE_PYTHON],
# Enable dumping stack traces
AC_ARG_ENABLE([mutex-debugging],
- [AC_HELP_STRING([--enable-mutex-debugging], [Makes mutexes to report error and provide stack trace])],
- [enable_mutex_debugging=yes], [enable_mutex_debugging=no])
+ [AC_HELP_STRING([--enable-mutex-debugging], [Makes mutexes to report error and provide stack trace [default=no]])],
+ [], [enable_mutex_debugging=no])
AM_CONDITIONAL([MUTEX_DEBUGGING], [test "x$enable_mutex_debugging" = "xyes"])
AM_COND_IF([MUTEX_DEBUGGING],
@@ -225,8 +258,8 @@ m4_ifdef([PKG_CHECK_VAR], [],
# Lua module and scripts
AC_ARG_ENABLE([lua],
- [AC_HELP_STRING([--enable-lua], [enable lua binding])],
- [], [enable_lua=check])
+ [AC_HELP_STRING([--enable-lua], [enable lua binding [default=auto]])],
+ [], [enable_lua=auto])
AC_ARG_WITH([lua-pc],
[AS_HELP_STRING(
@@ -239,7 +272,7 @@ if test "x$enable_lua" = "xyes" -a "x$with_lua_pc" != "xno"; then
PKG_CHECK_MODULES([LUA], [$with_lua_pc], [LUAPKGCONFIG=$with_lua_pc])
fi
-if test "x$enable_lua" = "xcheck" -a "x$with_lua_pc" != "xno"; then
+if test "x$enable_lua" = "xauto" -a "x$with_lua_pc" != "xno"; then
PKG_CHECK_MODULES([LUA], [$with_lua_pc],
[LUAPKGCONFIG=$with_lua_pc
enable_lua=yes],
@@ -274,8 +307,8 @@ AM_COND_IF([ENABLE_LUA],
# Optional test binaries
AC_ARG_ENABLE([tests],
- [AC_HELP_STRING([--enable-tests], [build test/example binaries])],
- [enable_tests=yes], [enable_tests=no])
+ [AC_HELP_STRING([--enable-tests], [build test/example binaries [default=no]])],
+ [], [enable_tests=no])
AM_CONDITIONAL([ENABLE_TESTS], [test "x$enable_tests" = "xyes"])
# LXC container path, where the containers are actually stored
@@ -325,12 +358,12 @@ AC_ARG_WITH([cgroup-pattern],
# Container log path. By default, use $lxcpath.
AC_MSG_CHECKING([Whether to place logfiles in container config path])
AC_ARG_ENABLE([configpath-log],
- [AC_HELP_STRING([--enable-configpath-log], [use logfiles in config path])],
- [use_configpath_logs=yes], [use_configpath_logs=no])
-AC_MSG_RESULT([$use_configpath_logs])
-AM_CONDITIONAL([USE_CONFIGPATH_LOGS], [test "$use_configpath_logs" = "yes"])
+ [AC_HELP_STRING([--enable-configpath-log], [use logfiles in config path [default=no]])],
+ [], [enable_configpath_log=no])
+AC_MSG_RESULT([$enable_configpath_log])
+AM_CONDITIONAL([USE_CONFIGPATH_LOGS], [test "$enable_configpath_log" = "yes"])
-if test "$use_configpath_logs" = "yes"; then
+if test "$enable_configpath_log" = "yes"; then
default_log_path="${with_config_path}"
else
default_log_path="${localstatedir}/log/lxc"
@@ -373,27 +406,6 @@ AC_CHECK_HEADERS([linux/unistd.h linux/netlink.h linux/genetlink.h],
AC_MSG_ERROR([Please install the Linux kernel headers.]),
[#include <sys/socket.h>])
-# Allow disabling libcap support
-AC_ARG_ENABLE([capabilities],
- [AC_HELP_STRING([--disable-capabilities], [disable kernel capabilities])],
- [], [enable_capabilities=yes])
-
-# Check for libcap support
-if test "x$enable_capabilities" = "xyes"; then
- AC_CHECK_LIB(cap,cap_set_proc,caplib=yes,caplib=no)
- AC_MSG_CHECKING([linux capabilities])
- if test "x$caplib" = "xyes" ; then
- CAP_LIBS="-lcap"
- AC_MSG_RESULT([$CAP_LIBS])
- else
- AC_MSG_RESULT([no])
- AC_MSG_ERROR([You are missing libcap support. If you really want to build without kernel capabilities, use --disable-capabilities])
- fi
-else
- CAP_LIBS=""
-fi
-AC_SUBST([CAP_LIBS])
-
# Check for alternate C libraries
AC_MSG_CHECKING(for bionic libc)
AC_COMPILE_IFELSE([AC_LANG_PROGRAM(
@@ -577,3 +589,35 @@ AC_CONFIG_FILES([
])
AC_CONFIG_COMMANDS([default],[[]],[[]])
AC_OUTPUT
+
+# Configuration overview
+cat << EOF
+
+----------------------------
+Environment:
+ - compiler: $CC
+ - distribution: $with_distro
+ - rpath: $enable_rpath
+
+Security features:
+ - Apparmor: $enable_apparmor
+ - Linux capabilities: $enable_capabilities
+ - seccomp: $enable_seccomp
+ - SELinux: $enable_selinux
+
+Bindings:
+ - lua: $enable_lua
+ - python3: $enable_python
+
+Documentation:
+ - examples: $enable_examples
+ - API documentation: $enable_api_docs
+ - user documentation: $enable_doc
+
+Debugging:
+ - tests: $enable_tests
+ - mutex debugging: $enable_mutex_debugging
+
+Paths:
+ - Logs in configpath: $enable_configpath_log
+EOF
--
1.8.5.1
More information about the lxc-devel
mailing list