Skip to content

Remove Qt5 support#1467

Merged
kelson42 merged 8 commits intomainfrom
remove-qt5
Feb 18, 2026
Merged

Remove Qt5 support#1467
kelson42 merged 8 commits intomainfrom
remove-qt5

Conversation

@kelson42
Copy link
Collaborator

@kelson42 kelson42 commented Feb 15, 2026

Fixes #1300

  • Update README.md
  • Remove ubuntu-22.04/qt5 (standart version) compilation from CI

@kelson42 kelson42 changed the title Remove qt5 Remove Qt5 support Feb 15, 2026
@kelson42 kelson42 marked this pull request as ready for review February 15, 2026 15:22
Copy link
Collaborator

@veloman-yunkan veloman-yunkan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that the PR title is a little misleading (at least I hope so, since I want to be able to build kiwix-desktop on Ubuntu 22.04 jammy using Qt5).

@kelson42
Copy link
Collaborator Author

I think that the PR title is a little misleading (at least I hope so, since I want to be able to build kiwix-desktop on Ubuntu 22.04 jammy using Qt5).

I don't understand this comment. 2.6.0 will build only on Qt6 because there is no reason to keep supporting buggy old version of Qt.

@veloman-yunkan
Copy link
Collaborator

I meant that this PR doesn't enforce Qt6 and therefore building kiwix-desktop on Ubuntu 22.04 is still possible, and I like that because it enables me to keep developing also on my computer running jammy.

@kelson42
Copy link
Collaborator Author

kelson42 commented Feb 17, 2026

@veloman-yunkan That is indeed correct, although bit surprising to me. Should we make a formal check in the .pro file?

@adamlamar
Copy link
Collaborator

LGTM. However note that the Ubuntu 22.04 OS is still supported for another year or so.

@kelson42
Copy link
Collaborator Author

I think that the PR title is a little misleading (at least I hope so, since I want to be able to build kiwix-desktop on Ubuntu 22.04 jammy using Qt5).

qmake nows exits with error if it does not have Qt5

@kelson42 kelson42 force-pushed the remove-qt5 branch 6 times, most recently from 61d84f2 to 3b96211 Compare February 18, 2026 18:35
@kelson42 kelson42 force-pushed the remove-qt5 branch 2 times, most recently from bba7f1a to b2db1b0 Compare February 18, 2026 18:59
@kelson42
Copy link
Collaborator Author

@legoktm Unfortunately, I'm in the impossibility tomget the package compiling properly with Qt6. Do you have maybe an idea why?

@kelson42 kelson42 merged commit c795c4c into main Feb 18, 2026
8 of 9 checks passed
@kelson42 kelson42 deleted the remove-qt5 branch February 18, 2026 19:33
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.

Requires Qt6 (and remove support of Qt5)

3 participants