Bug 811909
Summary: | Linking against mingw32 libbzip2 fails | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | David Nečas <yeti> | ||||
Component: | mingw32-bzip2 | Assignee: | Richard W.M. Jones <rjones> | ||||
Status: | CLOSED NOTABUG | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
Severity: | unspecified | Docs Contact: | |||||
Priority: | unspecified | ||||||
Version: | 16 | CC: | amorilia, fedora-mingw, h.peter.anvin, lfarkas, rjones | ||||
Target Milestone: | --- | ||||||
Target Release: | --- | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2012-04-12 19:17:16 UTC | Type: | Bug | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Attachments: |
|
Description
David Nečas
2012-04-12 09:59:50 UTC
Created attachment 577056 [details]
possible patch
A patch is attached that appears to fix the library for me by removing the WINAPI trickery in the header (apparently intended for native Win32). I do not understand the Win32 stuff enough to say this is the right fix though.
That's not the correct solution. It will make the functions use the cdecl calling convention instead of stdcall upstream uses and thus make the bzip2 DLL we ship binary-incompatible with the upstream one. Renaming the DLL's symbols to remove the default stdcall mangling (those @8 etc. suffixes) is also binary-incompatible. I see. So does the configure test (and possibly linking) of every cross-compiled program using bzip2 need to be changed and how? Or are there other options? This is a general and very serious braindamage in autoconf: compiling test programs without including the necessary header files does not work in general. It might have been expected to work that way 20+ years ago, but today it's just broken. So, there is no bug in mingw32-bzip2. I may still want a cdecl version of it but that not relevant to this bug report. Thanks for your time. mingw-bzip2-1.0.6-1.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/mingw-bzip2-1.0.6-1.fc17 mingw-bzip2-1.0.6-1.fc17, mingw-boost-1.48.0-8.fc17 has been pushed to the Fedora 17 stable repository. If problems still persist, please make note of it in this bug report. |