-
-
Notifications
You must be signed in to change notification settings - Fork 850
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
Bundled QtCompress #3802
Bundled QtCompress #3802
Conversation
Great PR! Please pay attention to the following items before merging: Files matching
This is an automatically generated QA checklist based on modified files. |
What are the cases when one would want to use the Qt-provided version? Are the maintainers of, say, Linux distributions supposed to always use the Qt-provided version? |
I think for source-based Linux distros, Qt-provided version will be better solution, but maintainers should have known best way for their distros. |
Any news for review? |
I'm not completely convinced that we want to use the Qt-provided code in this case, since it's simply unclean to rely on implementation details. OTOH, this flag is already an improvement, since we can now disable this reliance. If all the issues with mixing different licenses are clear (which I'm not competent enough to check), I don't have any additional objections. |
I hope in future the code for manage archives will be public - https://bugreports.qt.io/browse/QTBUG-3897 |
I wouldn't bet on it. The request was added 15 years ago at the times of Qt 4.5, and nothing changed since. |
Hello @alex-w! Please check the fresh version (development snapshot) of Stellarium: |
Hello @alex-w! Please check the latest stable version of Stellarium: |
Description
Restore support the building Stellarium with bundled
qtcompress
library in additional to Qt-provided library for manage ZIP archives. Using bundledqtcompress
library provide binary packages of Stellarium withoutqt6-base-abi
dependency in our PPA.Fixes #3793 (discussion)
Type of change