Skip to content

Commit

Permalink
Move AC_MSG_ERROR to a safer place
Browse files Browse the repository at this point in the history
The AC_MSG_ERROR macro shouldn't be in the value-if-not-found argument of
AC_PATH_PROG; because it is not a valid value for that variable. If we
get to that point, then we want our error message, not a syntax error.
  • Loading branch information
ptomato committed Mar 2, 2015
1 parent 4164f92 commit fafdddf
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions configure.ac
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,9 @@ AS_IF([pkg-config --exists gjs-1.0], [
AC_MSG_RESULT([pkg-config file not found, will try to guess])
])
AS_IF([test "x$gjs_pc_file" = xno],
[AC_PATH_PROG([GJS_PATH], [gjs],
[AC_MSG_ERROR([You need GJS to install Jasmine for GJS!])])])
[AC_PATH_PROG([GJS_PATH], [gjs], [notfound])])
AS_IF([test "x$GJS_PATH" = xnotfound],
[AC_MSG_ERROR([You need GJS to install Jasmine for GJS!])])
AC_SUBST([GJS_PATH])

# Output build system
Expand Down

0 comments on commit fafdddf

Please sign in to comment.