-
Notifications
You must be signed in to change notification settings - Fork 513
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix Poco only building "Basic Edition" #2553
base: master
Are you sure you want to change the base?
Conversation
|
Ref: https://digitalkarabela.com/mingw-w64-how-to-fix-file-too-big-too-many-sections/ |
4b1e6ef
to
82eba2b
Compare
The build script only dowloaded the sources of Poco's "Basic Edition" which only contains a few of its components. This commit makes it pull the complete Poco library in order to make all of it available under MXE. The PageCompiler and Data/ODBC module had to be disabled though as they don't build successfully. Fixes mxe#2552
Okay it turned out that my comments broke the code. Turns out suppressing newlines with Anyways: It's working now 👍 |
I think that in theory the following PRs that are currently only part of the Thus it might be possible to use the changes from these PRs as a patch and then switch to using cmake instead of autoconf for building Poco. This should have the advantage that the resulting packages should then also generate the cmake config files that'll facilitate cmake integration ☝️ |
@mabrand just to make sure there is no misunderstanding: This PR is done. It seems to be working and I don't have any intention of changing it anymore. My last comment was only meant to be a reference in case someone in the future might want to pick it up. |
The build fails for me. It looks like it's picking up the host system's mysql.h.
|
Oh that's interesting. Do you have any idea how the include path can be set in such a way that it'll include MXE's headers if present and only after that consider system headers? 🤔 |
The build script only dowloaded the sources of Poco's "Basic Edition"
which only contains a few of its components.
This commit makes it pull the complete Poco library in order to make all
of it available under MXE.
The PageCompiler and Data/ODBC module had to be disabled though as they
don't build successfully.
Fixes #2552