You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by tr...@apache.org on 2011/04/02 15:58:15 UTC

svn propchange: r1088023 - svn:log

Author: trawick
Revision: 1088023
Modified property: svn:log

Modified: svn:log at Sat Apr  2 13:58:15 2011
------------------------------------------------------------------------------
--- svn:log (original)
+++ svn:log Sat Apr  2 13:58:15 2011
@@ -1,5 +1,31 @@
 MinGW/MSYS:  Support shared builds of APR, other general improvements
 to support of this toolchain.
 
+test/Makefile.in:
+* $(LT_LDFLAGS) must be placed after the driver, otherwise it will act exactly
+like $(LTFLAGS). This is required, otherwise libtool will be never able to
+recognize the "-no-undefined" flag.
+* if mod_test.la does not specify $(LOCAL_LIBS), compilation will fail (this is
+required by both static and shared builds).
+
+include/apr.h.in:
+* It is required to declare APR_DECLARE, APR_DECLARE_NONSTD and
+APR_DECLARE_DATA as they are in apr.hw, otherwise the generate DLL won't export
+any symbol. APR_MODULE_DECLARE_DATA is already included, but previous ones are
+not.
+
+configure.in:
+* $(LT_LDFLAGS) must be placed after the driver, as described for
+test/Makefile.in.
+* Added "-no-undefined" flag to $(LT_LDFLAGS) if the target platform is
+Windows.
+* Declare APR_DECLARE_EXPORT or APR_DECLARE_STATIC, required by the macro added
+in include/apr.h.in.
+* Removed the "strange" libraries detection (btw, probably it was also breaking
+platforms without __stdcall calling convention, like Windows CE/Mobile/Phone,
+since function names were decorated), linker gave error without sense if adding
+"-lkernel32", probably because it creates a conflict of library precedence and
+dependency.
+
 PR: 46175
 Submitted by: Carlo Bramini <carlo.bramix libero.it>