Skip to content
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

qt: Fix build on Tiger #1154

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

qt: Fix build on Tiger #1154

wants to merge 2 commits into from

Conversation

sevan
Copy link
Contributor

@sevan sevan commented May 7, 2024

Need to build with Carbon support.
Ride the default with qt3support as it might be handy. Since it's a lengthy build, the previous option didn't work and failed on install stage. Add sqlite support so at least there is some database support by default.

draft since there is a bodge patch to src/corelib/io/qfilesystemengine.cpp, I need to enable building with OpenSSL 1.1.1, I haven't tested on anything beyond Tiger.

When it is ready to land, the PRs 529 575 577 should be reviewed to see if there's any additional bits that should be integrated. This PR basically introduces the changes proposed in PR 529 + fixes the issue building QtHelp & qmake on Tiger.

Skipping building with OpenSSL 1.1.1 as the patches I found were for 1.1.0 and more things need guarding off regarding to OpenSSL API usage. Since I don't have an QT application which I could use to test with & the lengthy build time I'm going to mark this ready for review until someone comes up with an application which needs OpenSSL support.

Tested on Tiger powerpc (G5/G4) with GCC 4.0.1, took 208 minutes on a 1.8Ghz iMac G5, 281 minutes a 1.33Ghz PowerBook G4.
Tested on Leopard powerpc (G5/i386) with GCC 4.2, took 204 minutes on a 1.8Ghz iMac G5, 87 minutes on a c2d MacBook Air.
Snow Leopard (x86_64) with GCC 4.2, took 84 minutes on a c2d MacBook Air.
El Capitan with clang, took 79 minutes on a c2d MacBook Air.

Need to build with Carbon support.
Ride the default with qt3support as it might be handy. Since it's
a lengthy build, the previous option didn't work and failed on install stage.
Add sqlite support so at least there is some database support by default.
@sevan sevan marked this pull request as ready for review May 12, 2024 00:32
@mistydemeo
Copy link
Owner

Wow, nice job getting this fixed! I'll get this built tomorrow most likely.

@mistydemeo
Copy link
Owner

Messed up and built it with postgresql opportunistically linked. Not enough time to finish before bed, so I'll do it again tomorrow. It built though!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants