Hi,
I have submitted a patch for review:
https://gerrit.libreoffice.org/1789
To pull it, you can do:
git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/89/1789/1
disable odk if not explicitly requested and doxygen not in PATH
if odk is not explicitly requested by --enable-odk
and --with-doxygen is "yes" , i.e not a specific location
and doxygen could not be found in the PATH
then enable_odk is set to "no" as a default value
rather than an implicit 'yes'
Change-Id: I4f04ecc8dbe959001f6dfdccf0c97b61f79faff6
---
M configure.ac
1 file changed, 27 insertions(+), 22 deletions(-)
diff --git a/configure.ac b/configure.ac
index 7f001a6..559f6a0 100644
--- a/configure.ac
+++ b/configure.ac
@@ -6944,6 +6944,33 @@
dnl ===================================================================
dnl Check for building ODK
dnl ===================================================================
+if test "$enable_odk" = no; then
+ unset DOXYGEN
+else
+ if test "$with_doxygen" = no; then
+ AC_MSG_CHECKING([for doxygen])
+ unset DOXYGEN
+ AC_MSG_RESULT([no])
+ else
+ if test "$with_doxygen" = yes; then
+ AC_PATH_PROG([DOXYGEN], [doxygen])
+ if test -z "$DOXYGEN"; then
+ AC_MSG_RESULT([no])
+ if test "$enable_odk" = "" ; then
+ enable_odk="no"
+ else
+ AC_MSG_ERROR([doxygen not found in \$PATH; specify its
pathname via --with-doxygen=..., or disable its use via --without-doxygen])
+ fi
+ fi
+ else
+ AC_MSG_CHECKING([for doxygen])
+ DOXYGEN=$with_doxygen
+ AC_MSG_RESULT([$DOXYGEN])
+ fi
+ fi
+fi
+AC_SUBST([DOXYGEN])
+
AC_MSG_CHECKING([whether to build the ODK])
if test "$enable_odk" = "" -o "$enable_odk" != "no"; then
AC_MSG_RESULT([yes])
@@ -11655,28 +11682,6 @@
fi
AC_SUBST(OOO_JUNIT_JAR)
-if test "$enable_odk" = no; then
- unset DOXYGEN
-else
- if test "$with_doxygen" = no; then
- AC_MSG_CHECKING([for doxygen])
- unset DOXYGEN
- AC_MSG_RESULT([no])
- else
- if test "$with_doxygen" = yes; then
- AC_PATH_PROG([DOXYGEN], [doxygen])
- if test -z "$DOXYGEN"; then
- AC_MSG_RESULT([no])
- AC_MSG_ERROR([doxygen not found in \$PATH; specify its
pathname via --with-doxygen=..., or disable its use via --without-doxygen])
- fi
- else
- AC_MSG_CHECKING([for doxygen])
- DOXYGEN=$with_doxygen
- AC_MSG_RESULT([$DOXYGEN])
- fi
- fi
-fi
-AC_SUBST([DOXYGEN])
AC_SUBST(SCPDEFS)
--
To view, visit https://gerrit.libreoffice.org/1789
To unsubscribe, visit https://gerrit.libreoffice.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4f04ecc8dbe959001f6dfdccf0c97b61f79faff6
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Norbert Thiebaud <nthieb...@gmail.com>
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice
