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

Qt6 compatibillity #7834

Closed
wants to merge 2 commits into from
Closed

Qt6 compatibillity #7834

wants to merge 2 commits into from

Conversation

ivaradi
Copy link
Contributor

@ivaradi ivaradi commented Feb 10, 2025

This patch adds compatibility with older Qt versions (from Qt 6.2.4)

Copy link

Artifact containing the AppImage: nextcloud-appimage-pr-7834.zip

SHA256 checksum: 90d2cea6c06a3fc1998553d3f628a6d74590eab572e37626ff9de8a6924ae33b

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
3 Security Hotspots
72.9% Coverage on New Code (required ≥ 80%)
D Security Rating on New Code (required ≥ A)
D Maintainability Rating on New Code (required ≥ A)
222 New Code Smells (required ≤ 0)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@Rello
Copy link
Contributor

Rello commented Feb 21, 2025

Hello István,

For transparency, I’d like to share the key points from our direct email exchange here:

We want to ensure that the desktop client can benefit from new Qt features across all platforms. Maintaining long-term compatibility with older Qt versions would limit our ability to do so.
We truly appreciate and value contributions from our community, including your work. However, we are unable to provide native package support for older Linux distributions. That’s why we distribute via AppImage and are also considering Snap and Flatpak packages.

Thank you for your effort on this PR! However, adding these compatibility checks introduces significant overhead and additional dependencies. In fact, we are currently working in the opposite direction—streamlining the codebase to enhance stability across platforms.

We truly appreciate your contributions and hope to collaborate on future improvements aligned with our development direction.

Best regards

@Rello Rello closed this Feb 21, 2025
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