Improve the error message about missing PLY Python package

Previously, only a message about missing Python was printed, which was
misleading to many users.  The new message clearly states that Python
AND PLY is required and prints basic instructions how to install PLY
package.

(cherry picked from commit 55b48700da)
This commit is contained in:
Ondřej Surý 2019-05-10 09:29:00 +07:00 committed by Evan Hunt
parent c3162ac196
commit 31fbfe56fd
2 changed files with 10 additions and 2 deletions

2
configure vendored
View file

@ -12310,7 +12310,7 @@ fi
$as_echo_n "checking for Python support... " >&6; }
{ $as_echo "$as_me:${as_lineno-$LINENO}: result: no" >&5
$as_echo "no" >&6; }
as_fn_error $? "Python required for dnssec-keymgr" "$LINENO" 5
as_fn_error $? "Python >= 2.7 or >= 3.2 and the PLY package are required for dnssec-keymgr and other Python-based tools. PLY may be available from your OS package manager as python-ply or python3-ply; it can also be installed via pip. To build without Python/PLY, use --disable-python." "$LINENO" 5
fi
fi

View file

@ -252,7 +252,15 @@ AS_IF([test "$with_python" = "no"],
AS_IF([test "X$PYTHON" = "X"],
[AC_MSG_CHECKING([for Python support])
AC_MSG_RESULT([no])
AC_MSG_ERROR([Python required for dnssec-keymgr])])])
AC_MSG_ERROR([m4_normalize(
[Python >= 2.7 or >= 3.2 and the PLY package
are required for dnssec-keymgr and other
Python-based tools. PLY may be
available from your OS package manager
as python-ply or python3-ply; it can also
be installed via pip. To build without
Python/PLY, use --disable-python.]
)])])])
PYTHON_TOOLS=''
CHECKDS=''