create the whole plugin structure in the build directory
ClosedPublic

Authored by mart on Sep 13 2017, 7:52 AM.

Details

Summary

when building copy all qml files in build/org/kde/kirigami.2
as well generating the binary plugin there.
this is neccessary to be able to load and test the whole QML
import without installing it system wide

Test Plan

after make, the hierarchy under build is complete with the same files
that end up being installed

Diff Detail

Repository
R169 Kirigami
Branch
cmakehack
Lint
No Linters Available
Unit
No Unit Test Coverage
mart created this revision.Sep 13 2017, 7:52 AM
Restricted Application added a project: Kirigami. · View Herald TranscriptSep 13 2017, 7:52 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart updated this revision to Diff 19473.Sep 13 2017, 8:51 AM
  • alternative approach
apol added a subscriber: apol.Sep 13 2017, 10:10 AM

Also you maybe can move this code into the tests subdirectory, so it only needs to be run in BUILD_TESTING.

I am not sure if this is going to work but I agree that if it has any opportunity then it's probably along these lines...

src/CMakeLists.txt
33

Why do you need to GLOB? can't you just copy the whole directory over with all of the styles/controls inside?

mart added a comment.Sep 18 2017, 9:14 AM
In D7794#145371, @apol wrote:

Also you maybe can move this code into the tests subdirectory, so it only needs to be run in BUILD_TESTING.

I am not sure if this is going to work but I agree that if it has any opportunity then it's probably along these lines...

i tought the direction now was to make things usable also when not installed, which would make it needed not only for tests

mart added inline comments.Sep 18 2017, 9:36 AM
src/CMakeLists.txt
33

if instead of the GLOB i put in the copy command directly the directory name, it seems to not actually copy anything

mart updated this revision to Diff 19636.Sep 18 2017, 11:31 AM
  • get rid of GLOB
mart updated this revision to Diff 19637.Sep 18 2017, 11:37 AM
  • remove dead code
apol accepted this revision.Sep 18 2017, 11:38 AM

Looks good to me.

This revision is now accepted and ready to land.Sep 18 2017, 11:38 AM
mart updated this revision to Diff 19638.Sep 18 2017, 11:39 AM
  • lowercase
apol accepted this revision.Sep 18 2017, 11:40 AM
This revision was automatically updated to reflect the committed changes.