Commit daf3bd27 authored by Ryan Tandy's avatar Ryan Tandy Committed by Quanah Gibson-Mount
Browse files

ITS#9307 --enable-<option>=mod requires --enable-modules

Provide a more specific diagnostic when configuring with
--enable-<option>=mod but not --enable-modules. Make this an error for
both backends and overlays, and remove the automatic fallback to static.
parent d4c7126e
Pipeline #660 passed with stage
in 28 minutes and 9 seconds
......@@ -447,6 +447,14 @@ else
dnl then require at least one built-in backend
if test $ol_enable_modules = no; then
for i in backends overlays $Backends $Overlays; do
eval "ol_tmp=\$ol_enable_$i"
if test -n "$ol_tmp" && test "$ol_tmp" = mod ; then
AC_MSG_ERROR([--enable-$i=mod requires --enable-modules])
fi
done
ol_any_backend=no
for i in $Backends; do
eval "ol_tmp=\$ol_enable_$i"
......@@ -754,22 +762,6 @@ if test $ol_enable_modules != no ; then
fi
ol_link_modules=yes
WITH_MODULES_ENABLED=yes
else
for i in $Backends; do
eval "ol_tmp=\$ol_enable_$i"
if test $ol_tmp = mod ; then
AC_MSG_WARN([building static $i backend])
eval "ol_enable_$i=yes"
fi
done
for i in $Overlays; do
eval "ol_tmp=\$ol_enable_$i"
if test $ol_tmp = mod ; then
AC_MSG_WARN([building static $i overlay])
eval "ol_enable_$i=yes"
fi
done
fi
dnl ----------------------------------------------------------------
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment