Date: prev next · Thread: first prev next last
2013 Archives by date, by thread · List index


On Mon, Jan 21, 2013 at 12:33:05AM -0800, Libreoffice Gerrit user wrote:
    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'

Can we please stop this "auto-deect-everything-and-disable-stuff-silently"
nonsense?

Imagine the following scenario.

Default build without doxygen but *with* the SDK. (e.g. binary-only builds).
The Debian buildds won't install doxygen because it's only needed for
arch-indep stuff (and the arch-dep part uses -without-doxygen). So  the
binary-only build will see no doxygen -> builds no SDK -> bad.

With that change above, you will not get any SDK built thus either
breaking the build (case here) or get a empty SDK package.

There *IS* already -without-doxygen for people who want the SDK but
not doxygen.

(NB: Didn't test-build that yet so maybe --enable-odk --disable-doxygen
still works, but that's not against my point.)

Oh, and it wozld make more sense to get such changes reviewed by Linux
people or distro packjagers who heavily rely on configure switches in
their packaging...

Regards,

Rene

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.