diff options
author | iphydf <iphydf@users.noreply.github.com> | 2016-12-11 20:38:26 +0000 |
---|---|---|
committer | iphydf <iphydf@users.noreply.github.com> | 2016-12-13 00:52:58 +0000 |
commit | 2dc2ac52b93e3a922337294c1e32f7bed794788f (patch) | |
tree | 860d532d00babbdf102b4ee706533d7e1daedf6f /configure.ac | |
parent | 029c4fb83f66de62428593d86a6493400a8f4b27 (diff) |
Fix `--enable-logging` flag in autotools configure script.
We also never really tested this, because we run make distcheck, which
does another configure with default flags instead of the ones we passed.
Fixes #317.
Diffstat (limited to 'configure.ac')
-rw-r--r-- | configure.ac | 27 |
1 files changed, 6 insertions, 21 deletions
diff --git a/configure.ac b/configure.ac index e36900a3..246b6c76 100644 --- a/configure.ac +++ b/configure.ac | |||
@@ -34,7 +34,6 @@ BUILD_TESTING="yes" | |||
34 | 34 | ||
35 | TOX_LOGGER="no" | 35 | TOX_LOGGER="no" |
36 | TOX_DEBUG="no" | 36 | TOX_DEBUG="no" |
37 | LOGGING_OUTNAM="libtoxcore.log" | ||
38 | 37 | ||
39 | NCURSES_FOUND="no" | 38 | NCURSES_FOUND="no" |
40 | LIBCONFIG_FOUND="no" | 39 | LIBCONFIG_FOUND="no" |
@@ -88,9 +87,7 @@ AC_ARG_ENABLE([logging], | |||
88 | if test "x$enableval" = "xyes"; then | 87 | if test "x$enableval" = "xyes"; then |
89 | TOX_LOGGER="yes" | 88 | TOX_LOGGER="yes" |
90 | 89 | ||
91 | AC_DEFINE([TOX_LOGGER], [], [If logging enabled]) | 90 | AC_DEFINE([MIN_LOGGER_LEVEL], [LOG_DEBUG], [LOG_LEVEL value]) |
92 | AC_DEFINE([LOGGER_LEVEL], [LOG_DEBUG], [LOG_LEVEL value]) | ||
93 | AC_DEFINE_UNQUOTED([LOGGER_OUTPUT_FILE], ["$LOGGING_OUTNAM"], [Output of logger]) | ||
94 | fi | 91 | fi |
95 | ] | 92 | ] |
96 | ) | 93 | ) |
@@ -114,19 +111,19 @@ AC_ARG_WITH(log-level, | |||
114 | AC_MSG_WARN([Logging disabled!]) | 111 | AC_MSG_WARN([Logging disabled!]) |
115 | else | 112 | else |
116 | if test "x$withval" = "xTRACE"; then | 113 | if test "x$withval" = "xTRACE"; then |
117 | AC_DEFINE([LOGGER_LEVEL], [LOG_TRACE], [LOG_LEVEL value]) | 114 | AC_DEFINE([MIN_LOGGER_LEVEL], [LOG_TRACE], [LOG_LEVEL value]) |
118 | 115 | ||
119 | elif test "x$withval" = "xDEBUG"; then | 116 | elif test "x$withval" = "xDEBUG"; then |
120 | AC_DEFINE([LOGGER_LEVEL], [LOG_DEBUG], [LOG_LEVEL value]) | 117 | AC_DEFINE([MIN_LOGGER_LEVEL], [LOG_DEBUG], [LOG_LEVEL value]) |
121 | 118 | ||
122 | elif test "x$withval" = "xINFO"; then | 119 | elif test "x$withval" = "xINFO"; then |
123 | AC_DEFINE([LOGGER_LEVEL], [LOG_INFO], [LOG_LEVEL value]) | 120 | AC_DEFINE([MIN_LOGGER_LEVEL], [LOG_INFO], [LOG_LEVEL value]) |
124 | 121 | ||
125 | elif test "x$withval" = "xWARNING"; then | 122 | elif test "x$withval" = "xWARNING"; then |
126 | AC_DEFINE([LOGGER_LEVEL], [LOG_WARNING], [LOG_LEVEL value]) | 123 | AC_DEFINE([MIN_LOGGER_LEVEL], [LOG_WARNING], [LOG_LEVEL value]) |
127 | 124 | ||
128 | elif test "x$withval" = "xERROR"; then | 125 | elif test "x$withval" = "xERROR"; then |
129 | AC_DEFINE([LOGGER_LEVEL], [LOG_ERROR], [LOG_LEVEL value]) | 126 | AC_DEFINE([MIN_LOGGER_LEVEL], [LOG_ERROR], [LOG_LEVEL value]) |
130 | else | 127 | else |
131 | AC_MSG_WARN([Invalid logger level: $withval. Using default 'DEBUG']) | 128 | AC_MSG_WARN([Invalid logger level: $withval. Using default 'DEBUG']) |
132 | fi | 129 | fi |
@@ -134,18 +131,6 @@ AC_ARG_WITH(log-level, | |||
134 | ] | 131 | ] |
135 | ) | 132 | ) |
136 | 133 | ||
137 | AC_ARG_WITH(log-path, | ||
138 | AC_HELP_STRING([--with-log-path=DIR], | ||
139 | [Path of logger output]), | ||
140 | [ | ||
141 | if test "x$TOX_LOGGER" = "xno"; then | ||
142 | AC_MSG_WARN([Logging disabled!]) | ||
143 | else | ||
144 | AC_DEFINE_UNQUOTED([LOGGER_OUTPUT_FILE], ["$withval""/""$LOGGING_OUTNAM"], [Output of logger]) | ||
145 | fi | ||
146 | ] | ||
147 | ) | ||
148 | |||
149 | PKG_PROG_PKG_CONFIG | 134 | PKG_PROG_PKG_CONFIG |
150 | 135 | ||
151 | AC_ARG_ENABLE([av], | 136 | AC_ARG_ENABLE([av], |