apr-bugs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 46175] Full Mingw+MSys support
Date Thu, 31 Mar 2011 19:01:26 GMT
https://issues.apache.org/bugzilla/show_bug.cgi?id=46175

Carlo Bramini <carlo.bramix@libero.it> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #26817|0                           |1
        is obsolete|                            |

--- Comment #20 from Carlo Bramini <carlo.bramix@libero.it> 2011-03-31 15:01:21 EDT
---
Created an attachment (id=26821)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=26821)
Fix for this bug

This last patch should fix creation of shared libraries (DLL) under MinGW+MSys.
The corrections can be resumed in:

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.

That's all, all these topic have also a good description in previous posts, if
you think that something else should be adjusted in this patch, let me know!

Sincerely,

Carlo Bramini.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe@apr.apache.org
For additional commands, e-mail: bugs-help@apr.apache.org


Mime
View raw message