Drop remote ioslave, which is now in KIO 5.32
ClosedPublic

Authored by elvisangelaccio on Nov 28 2016, 4:06 PM.

Details

Summary

As discussed on plasma-devel [1]. See https://phabricator.kde.org/D4690

This bumps the minimum KF5 version to 5.32

[1]: https://mail.kde.org/pipermail/plasma-devel/2016-November/061750.html

BUG: 371825
FIXED-IN: 5.10

Test Plan

Still builds.

Diff Detail

Repository
R120 Plasma Workspace
Branch
drop-slaves
Lint
No Linters Available
Unit
No Unit Test Coverage
elvisangelaccio retitled this revision from to Drop kioslave folder, will be in KIO 5.29.
elvisangelaccio updated this object.
elvisangelaccio edited the test plan for this revision. (Show Details)
elvisangelaccio added a reviewer: Plasma.
Restricted Application added a project: Plasma. · View Herald TranscriptNov 28 2016, 4:06 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript

5.29 is in only a few days, that's way too close for a change this massive.
Especially one that requires a lot of distribution co-ordination as we're creating installation conflicts.

Whilst I support this change in general, I don't want to see it going into 5.29.

Other than that, this part of the change is all good.

Speaking as a distro person: this might cause issues. The reason is that *if* we update to KF 5.29 (or 30) with this and Plasma stays as a stable version, we will get interesting conflicts.

And of course, removing this in a stable version is not an option.

5.29 is in only a few days, that's way too close for a change this massive.
Especially one that requires a lot of distribution co-ordination as we're creating installation conflicts.

Whilst I support this change in general, I don't want to see it going into 5.29.

Other than that, this part of the change is all good.

Right, didn't realize that 5.29 was so close.

  • Bump minimum version, 5.29 is too close
elvisangelaccio retitled this revision from Drop kioslave folder, will be in KIO 5.29 to Drop kioslave folder, will be in KIO 5.30.Nov 28 2016, 4:33 PM
elvisangelaccio updated this object.
davidedmundson accepted this revision.Nov 28 2016, 4:45 PM
davidedmundson added a reviewer: davidedmundson.

But don't merge till we merge the frameworks code obviously.

This revision is now accepted and ready to land.Nov 28 2016, 4:45 PM
elvisangelaccio edited edge metadata.
bshah added a subscriber: bshah.Nov 29 2016, 10:37 AM
bshah added inline comments.
CMakeLists.txt
8

You can't do this, framework version is bumped by release script automatically, so when you push this there is no 5.30.0 version of KIO. Which will make it fail, you need to bump version once frameworks 5.30 is relased.

CMakeLists.txt
8

Yep, this will be pushed only after 5.30 gets released

Only drop remote:/

elvisangelaccio requested review of this revision.Feb 20 2017, 5:14 PM
elvisangelaccio retitled this revision from Drop kioslave folder, will be in KIO 5.30 to Drop remote ioslave, will be in KIO 5.32.
elvisangelaccio edited the summary of this revision. (Show Details)
elvisangelaccio edited edge metadata.
elvisangelaccio retitled this revision from Drop remote ioslave, will be in KIO 5.32 to Drop remote ioslave, which is now in KIO 5.32.Mar 12 2017, 10:18 AM
elvisangelaccio edited the summary of this revision. (Show Details)

I'm going to push this by tomorrow unless someone has objections.

This revision was automatically updated to reflect the committed changes.