Summary: | PKG_CHECK_MODULES not defined when building LO on Windows 7 64-bit, rather than message about needing to add pkg-config | ||
---|---|---|---|
Product: | LibreOffice | Reporter: | Martin Gallwey <martin> |
Component: | Libreoffice | Assignee: | Not Assigned <libreoffice-bugs> |
Status: | CLOSED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | medium | CC: | caolanm |
Version: | unspecified | ||
Hardware: | x86-64 (AMD64) | ||
OS: | Windows (All) | ||
Whiteboard: | |||
i915 platform: | i915 features: | ||
Attachments: |
Proposed fix
Better patch |
Description
Martin Gallwey
2010-10-19 12:55:54 UTC
Sorry, running autogen.sh again with pkg-config installed did fix my problem, but ideally autogen.sh would show an error message complaining about pkg-config not being installed rather than the current somewhat arcane problem. Hmm... checking whether to enable maintainer-specific portions of Makefiles... yes checking for perl... /usr/bin/perl checking for pkg-config... /usr/bin/pkg-config checking pkg-config is at least version 0.9.0... yes checking for MINIMUM_REQS... yes and my configure.in has... PKG_PROG_PKG_CONFIG() if test ! -x "$PKG_CONFIG" -a "z`uname -s`" != "zDarwin"; then AC_MSG_ERROR([ *** You need the pkg-config to build LibreOffice. *** Get the latest version of pkg-config from *** <http://www.freedesktop.org/software/pkgconfig/>.]) fi so it should spit that out if pkg-config isn't available since 2010-10-08. Do you have that latest configure.in ? Created attachment 39782 [details] [review] Proposed fix Proposed fix attached. Created attachment 39783 [details] [review] Better patch Pushed - thank you :-) I notice the inner configure has a platform conditional for win32 that we might want to duplicate in the outer configure but ... this should be an improvement whatever. Thanks. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.