Add kwayland-integration to KWin dependencies
ClosedPublic

Authored by meven on Sep 19 2019, 8:06 AM.

Diff Detail

Repository
R499 Metadata for build scripts
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
meven requested review of this revision.Sep 19 2019, 8:06 AM
meven created this revision.
meven added a comment.Sep 19 2019, 8:07 AM

Apparently my previous changes made in D23931 were overriden, I had to redo them.

bcooksley requested changes to this revision.Sep 19 2019, 8:33 AM

The changes were probably rolled back when I synced stable and master to ensure they matched.

Note that the KWayland Integration changes are likely unnecessary - all Frameworks are provided by default to projects outside of frameworks/*

Assuming the same logic holds, and as long as the respective maintainers of KWayland Integration and KWin are still fine with this, it can go in again.

This revision now requires changes to proceed.Sep 19 2019, 8:33 AM
meven added a comment.Sep 20 2019, 7:56 AM

The changes were probably rolled back when I synced stable and master to ensure they matched.

Note that the KWayland Integration changes are likely unnecessary - all Frameworks are provided by default to projects outside of frameworks/*

As far i can it does not hold when building with kde-srcbuild : kwayland-integration was not built when building plasma-desktop or Kwin for instance.

Assuming the same logic holds, and as long as the respective maintainers of KWayland Integration and KWin are still fine with this, it can go in again.

David approved the previous diff, this one juste copies the change to master and stable. https://phabricator.kde.org/D24071#

Sorry for not being clear - the lines i'm referring to are lines 467 to 469 in your diff - you don't need to mark KWayland Integration as depending on particular Frameworks, because that is done by a global rule.

meven updated this revision to Diff 66797.Sep 25 2019, 7:01 AM

Remove kwayland-integration frameworks dependencies section

bcooksley accepted this revision.Sep 25 2019, 9:32 AM

Thanks for the update - this is all good to go in now.

This revision is now accepted and ready to land.Sep 25 2019, 9:32 AM
This revision was automatically updated to reflect the committed changes.