phonon : build against "older" Qt5 versions
Needs RevisionPublic

Authored by rjvbb on Dec 23 2019, 10:13 AM.

Details

Reviewers
sitter
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Summary

Phonon 4.11.x doesn't build against Qt 5.9 because of a single env.var-related function call that must have been introduced in a later Qt version.

I don't see any mininmum Qt version specification, so here's a patch that makes the code build against at least Qt 5.9.8 .

Test Plan

Building against Qt 5.9.8

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped
Build Status
Buildable 20192
Build 20210: arc lint + arc unit
rjvbb created this revision.Dec 23 2019, 10:13 AM
Owners added a reviewer: Restricted Owners Package.Dec 23 2019, 10:13 AM
rjvbb requested review of this revision.Dec 23 2019, 10:13 AM
sitter requested changes to this revision.Jan 7 2020, 12:20 PM

I'd rather just bump the required version to 5.12.

This revision now requires changes to proceed.Jan 7 2020, 12:20 PM
rjvbb added a comment.Jan 7 2020, 1:32 PM

But why, there's nothing that requires this other than that single env. var. function that may not even be the most appropriate one (I find the documentation a bit confusing).

Don't forget that Phonon is not Linux-specific and that on other platforms newer Qt versions no longer support older OS versions that are still being used. That includes the Mac OS version I'm on, so no, I won't condone taking that path.

sitter added a comment.Jan 7 2020, 2:04 PM

KF5's Qt support strategy is sound and so phonon should follow that. We've moved away from keeping phonon backwards compatible to the big bang, it's not sustainable.

rjvbb added a comment.Jan 7 2020, 3:41 PM
KF5's Qt support strategy is sound

Except that that too only requires overly new Qt versions for newfangled form-over-function stuff in frameworks like Kirigami, the rest would build against Qt 5.9 until at least 5.60.0 with minimal tweaking and zero feature loss.

And no framework serves as crucial a feature as Phonon does.

We've moved away from keeping phonon backwards compatible to the big bang, it's not sustainable.

Not what I am suggesting, but that's not the same as willingly dropping backwards compatibility just because you can. When was the big bang anyway, Qt v2? O:-) Certainly not Qt 5.9, which is officially still supported.