Skip to content

Commit

Permalink
Move AC_PROG_CC to the outside of the if block
Browse files Browse the repository at this point in the history
configure: error: conditional "AMDEP" was never defined.

Automake doesn't allow using AC_PROG_CC conditionally. This commit fixes
build when WebKitGTK+-based helper is disabled. It also makes C compiler
a hard requirement to build the project because I cannot find an easy
way to make AC_PROG_CC failure non-fatal.
  • Loading branch information
lantw44 committed Aug 25, 2017
1 parent 9f3c5ea commit 621d5ae
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions configure.ac
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# -*- Autoconf -*-
# Process this file with autoconf to produce a configure script.

AC_INIT([ceiba-dl], [0.1], [https://github.com/lantw44/ceiba-dl/issues])
AC_INIT([ceiba-dl], [0.1.1], [https://github.com/lantw44/ceiba-dl/issues])
AC_CONFIG_SRCDIR([ceiba_dl/__init__.py])
AC_CONFIG_HEADERS([config.h])

Expand All @@ -15,6 +15,8 @@ AC_ARG_ENABLE([helper-webkitgtk],
[enable_helper_webkitgtk="yes"])
AM_CONDITIONAL([ENABLE_HELPER_WEBKITGTK], [test "$enable_helper_webkitgtk" = "yes"])

AC_PROG_CC
AC_PROG_CC_STDC
AM_PATH_PYTHON([3])
case "$PYTHON" in
/*)
Expand All @@ -31,8 +33,6 @@ ceiba_dl_pythondir='${pythondir}/ceiba_dl'
AC_SUBST([ceiba_dl_pythondir], [${ceiba_dl_pythondir}])

if test "$enable_helper_webkitgtk" = "yes"; then
AC_PROG_CC
AC_PROG_CC_STDC
AX_IS_RELEASE([git-directory])
AX_COMPILER_FLAGS()
case "$WARN_CFLAGS" in
Expand Down

0 comments on commit 621d5ae

Please sign in to comment.