Stop `configure` and throw an error when `--with-isl` is passed without
an argument.

If someone by mistake passes `--with-isl` using the option only or
sepparated by a space instead of `=` it currently tries to build it.

Since this syntax is valid in several programs (i.e. those built using
getopt_long), throwing an error might prevent build-time issues.

ChangeLog:

        * configure.ac: Validate the value passed for `--with-isl` to
        detect 'yes' (no argument).

        * configure: Regenerate `configure` with `autoreconf-2.69`.

Signed-off-by: Agatha Isabelle Moreira <[email protected]>
---
 configure    | 3 +++
 configure.ac | 3 +++
 2 files changed, 6 insertions(+)

diff --git a/configure b/configure
index 02cd3bb7997..69f1b54f954 100755
--- a/configure
+++ b/configure
@@ -9588,6 +9588,9 @@ if test "${with_isl+set}" = set; then :
 fi
 
 
+if test "x$with_isl" = "xyes"; then
+  as_fn_error $? "--with-isl requires a PATH argument" "$LINENO" 5
+fi
 # Treat --without-isl as a request to disable
 # GRAPHITE support and skip all following checks.
 if test "x$with_isl" != "xno"; then
diff --git a/configure.ac b/configure.ac
index ed66e1ea2e1..7961b281400 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2018,6 +2018,9 @@ AC_ARG_WITH(isl,
     Equivalent to --with-isl-include=PATH/include
     plus --with-isl-lib=PATH/lib])])
 
+if test "x$with_isl" = "xyes"; then
+  AC_MSG_ERROR([--with-isl requires a PATH argument])
+fi
 # Treat --without-isl as a request to disable
 # GRAPHITE support and skip all following checks.
 if test "x$with_isl" != "xno"; then
-- 
2.54.0

Reply via email to