Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Correct order of 3rd party libraries in wx-config output

Browse Source 
A recent change fixing monolithic build broke the normal build when
using built-in libraries under Unix as -lwxzlib now came before -lwxpng
and so zlib symbols used by libpng were not found (in fact, they could
still be found in the system library which is usually available under
Unix, so the problem wasn't seen immediately, but it was still there).

See https://github.com/wxWidgets/wxWidgets/pull/621
This commit is contained in:
Vadim Zeitlin
2017-12-08 02:36:12 +01:00
parent db00ab9c02
commit b0c4cb7131
2 changed files with 51 additions and 45 deletions
Download Patch File Download Diff File Expand all files Collapse all files

44
configure vendored
View File

@@ -35902,28 +35902,6 @@ LDFLAGS="$LDFLAGS $PROFILE_FLAGS"
WXCONFIG_LIBS="$LIBS" WXCONFIG_LIBS="$LIBS"
if test "$wxUSE_REGEX" = "builtin" ; then
wxconfig_3rdparty="regex${lib_unicode_suffix} $wxconfig_3rdparty"
fi
if test "$wxUSE_STC" = "yes" ; then
wxconfig_3rdparty="scintilla $wxconfig_3rdparty"
fi
case "$wxUSE_EXPAT" in
builtin)
wxconfig_3rdparty="expat $wxconfig_3rdparty"
;;
sys)
WXCONFIG_LIBS="$EXPAT_LINK $WXCONFIG_LIBS"
;;
esac
case "$wxUSE_ZLIB" in
builtin)
wxconfig_3rdparty="zlib $wxconfig_3rdparty"
;;
sys)
WXCONFIG_LIBS="$ZLIB_LINK $WXCONFIG_LIBS"
;;
esac
if test "$wxUSE_GUI" = "yes"; then if test "$wxUSE_GUI" = "yes"; then
case "$wxUSE_LIBTIFF" in case "$wxUSE_LIBTIFF" in
builtin) builtin)
@@ -35950,6 +35928,28 @@ if test "$wxUSE_GUI" = "yes"; then
;; ;;
esac esac
fi fi
if test "$wxUSE_REGEX" = "builtin" ; then
wxconfig_3rdparty="regex${lib_unicode_suffix} $wxconfig_3rdparty"
fi
if test "$wxUSE_STC" = "yes" ; then
wxconfig_3rdparty="scintilla $wxconfig_3rdparty"
fi
case "$wxUSE_EXPAT" in
builtin)
wxconfig_3rdparty="expat $wxconfig_3rdparty"
;;
sys)
WXCONFIG_LIBS="$EXPAT_LINK $WXCONFIG_LIBS"
;;
esac
case "$wxUSE_ZLIB" in
builtin)
wxconfig_3rdparty="zlib $wxconfig_3rdparty"
;;
sys)
WXCONFIG_LIBS="$ZLIB_LINK $WXCONFIG_LIBS"
;;
esac
for i in $wxconfig_3rdparty ; do for i in $wxconfig_3rdparty ; do
WXCONFIG_LIBS="-lwx${i}${WX_LIB_FLAVOUR}-${WX_RELEASE}${HOST_SUFFIX} $WXCONFIG_LIBS" WXCONFIG_LIBS="-lwx${i}${WX_LIB_FLAVOUR}-${WX_RELEASE}${HOST_SUFFIX} $WXCONFIG_LIBS"

52
configure.in
View File

@@ -7827,29 +7827,13 @@ LDFLAGS="$LDFLAGS $PROFILE_FLAGS"
WXCONFIG_LIBS="$LIBS" WXCONFIG_LIBS="$LIBS"
dnl wx-config must output 3rd party libs in --libs in static build: dnl wx-config must output 3rd party libs in --libs in static build, so collect
if test "$wxUSE_REGEX" = "builtin" ; then dnl them in wxconfig_3rdparty variable
wxconfig_3rdparty="regex${lib_unicode_suffix} $wxconfig_3rdparty" dnl
fi dnl Notice that the libraries must be listed in correct order whenever there
if test "$wxUSE_STC" = "yes" ; then dnl are dependencies between them for linking to work when using static
wxconfig_3rdparty="scintilla $wxconfig_3rdparty" dnl libraries, e.g. jpeg, used by tiff, must come after it and zlib, used by
fi dnl png, must come after it as well.
case "$wxUSE_EXPAT" in
builtin)
wxconfig_3rdparty="expat $wxconfig_3rdparty"
;;
sys)
WXCONFIG_LIBS="$EXPAT_LINK $WXCONFIG_LIBS"
;;
esac
case "$wxUSE_ZLIB" in
builtin)
wxconfig_3rdparty="zlib $wxconfig_3rdparty"
;;
sys)
WXCONFIG_LIBS="$ZLIB_LINK $WXCONFIG_LIBS"
;;
esac
if test "$wxUSE_GUI" = "yes"; then if test "$wxUSE_GUI" = "yes"; then
case "$wxUSE_LIBTIFF" in case "$wxUSE_LIBTIFF" in
builtin) builtin)
@@ -7876,6 +7860,28 @@ if test "$wxUSE_GUI" = "yes"; then
;; ;;
esac esac
fi fi
if test "$wxUSE_REGEX" = "builtin" ; then
wxconfig_3rdparty="regex${lib_unicode_suffix} $wxconfig_3rdparty"
fi
if test "$wxUSE_STC" = "yes" ; then
wxconfig_3rdparty="scintilla $wxconfig_3rdparty"
fi
case "$wxUSE_EXPAT" in
builtin)
wxconfig_3rdparty="expat $wxconfig_3rdparty"
;;
sys)
WXCONFIG_LIBS="$EXPAT_LINK $WXCONFIG_LIBS"
;;
esac
case "$wxUSE_ZLIB" in
builtin)
wxconfig_3rdparty="zlib $wxconfig_3rdparty"
;;
sys)
WXCONFIG_LIBS="$ZLIB_LINK $WXCONFIG_LIBS"
;;
esac
for i in $wxconfig_3rdparty ; do for i in $wxconfig_3rdparty ; do
WXCONFIG_LIBS="-lwx${i}${WX_LIB_FLAVOUR}-${WX_RELEASE}${HOST_SUFFIX} $WXCONFIG_LIBS" WXCONFIG_LIBS="-lwx${i}${WX_LIB_FLAVOUR}-${WX_RELEASE}${HOST_SUFFIX} $WXCONFIG_LIBS"