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

Bundled QtCompress #3802

Merged
merged 4 commits into from
Jul 15, 2024
Merged

Bundled QtCompress #3802

merged 4 commits into from
Jul 15, 2024

Conversation

alex-w
Copy link
Member

@alex-w alex-w commented Jul 8, 2024

Description

Restore support the building Stellarium with bundled qtcompress library in additional to Qt-provided library for manage ZIP archives. Using bundled qtcompress library provide binary packages of Stellarium without qt6-base-abi dependency in our PPA.

Fixes #3793 (discussion)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • This change requires a documentation update
  • Housekeeping

@alex-w alex-w added infrastructure Infrastructure related issues dependencies dependencies (such like ShowMySky, QXlsx, etc.) related issues labels Jul 8, 2024
@github-actions github-actions bot requested review from 10110111 and gzotti July 8, 2024 14:48
Copy link

github-actions bot commented Jul 8, 2024

Great PR! Please pay attention to the following items before merging:

Files matching src/**/*.cpp:

  • Are possibly unused includes removed?

This is an automatically generated QA checklist based on modified files.

@alex-w alex-w added this to the 24.3 milestone Jul 8, 2024
@10110111
Copy link
Contributor

10110111 commented Jul 8, 2024

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?

@alex-w
Copy link
Member Author

alex-w commented Jul 8, 2024

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.

@alex-w
Copy link
Member Author

alex-w commented Jul 15, 2024

Any news for review?

@10110111
Copy link
Contributor

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.

@alex-w
Copy link
Member Author

alex-w commented Jul 15, 2024

I hope in future the code for manage archives will be public - https://bugreports.qt.io/browse/QTBUG-3897

@10110111
Copy link
Contributor

I wouldn't bet on it. The request was added 15 years ago at the times of Qt 4.5, and nothing changed since.

@alex-w alex-w merged commit 86f2406 into master Jul 15, 2024
30 of 31 checks passed
@alex-w alex-w deleted the qtcompress branch July 15, 2024 09:50
@alex-w alex-w added the state: published The fix has been published for testing in weekly binary package label Aug 25, 2024
Copy link

Hello @alex-w!

Please check the fresh version (development snapshot) of Stellarium:
https://github.com/Stellarium/stellarium-data/releases/tag/weekly-snapshot

@alex-w alex-w removed the state: published The fix has been published for testing in weekly binary package label Sep 22, 2024
Copy link

Hello @alex-w!

Please check the latest stable version of Stellarium:
https://github.com/Stellarium/stellarium/releases/latest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies dependencies (such like ShowMySky, QXlsx, etc.) related issues infrastructure Infrastructure related issues
Development

Successfully merging this pull request may close these issues.

2 participants