Bump QtQuick.Controls dependency to 2.12 (from Qt 5.12).
AbandonedPublic

Authored by ouwerkerk on Sep 24 2019, 2:17 PM.

Details

Reviewers
mart
ngraham
Group Reviewers
Plasma
Summary

It is a bad idea to depend on non-existent versions: we should more clearly mark our intended QQC dependency version (ie. Qt 5.12 at a minimum) by refering to an actual release.
While technically correct, non-existent dependency versions such as 2.5 and 2.7 are confusing and causing people to mistakenly propose downstream patches to fix imagined 'typos', cf.: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=940966

Diff Detail

Repository
R169 Kirigami
Branch
bump-qcc2-to-qt-5.12
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 16969
Build 16987: arc lint + arc unit
ouwerkerk created this revision.Sep 24 2019, 2:17 PM
Restricted Application added a project: Kirigami. · View Herald TranscriptSep 24 2019, 2:17 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ouwerkerk requested review of this revision.Sep 24 2019, 2:17 PM
mart added a comment.Sep 24 2019, 2:36 PM

we have to depend from 2 qt versions ago as per KDE frameworks policy, so max version that can be used is Qt 5.11

If Kirigami is supposed to depend on Qt 5.11, then *why* were these old imports there in the first place? They cannot work with Qt 5.11, because then the max QQC version is 2.4

Maybe I am misunderstanding something here? Are you saying the old Kirigami code effectively broke its own policy on depending on (newer) Qt versions?

And if so, sure I can 'fix' this by downgrading to QQC 2.4 instead but in that case: is this 'safe'? I.e. does this introduce bugs/regressions?

ngraham requested changes to this revision.Sep 24 2019, 3:04 PM
ngraham added a subscriber: ngraham.

If Kirigami is supposed to depend on Qt 5.11, then *why* were these old imports there in the first place? They cannot work with Qt 5.11, because then the max QQC version is 2.4

Maybe I am misunderstanding something here? Are you saying the old Kirigami code effectively broke its own policy on depending on (newer) Qt versions?

Yes, D22974 broke this policy. The correct fix is D24147.

This revision now requires changes to proceed.Sep 24 2019, 3:04 PM
ouwerkerk abandoned this revision.Jan 2 2020, 3:02 PM

Cleanup.