Make pacakge usefull for cross compilation
Needs ReviewPublic

Authored by knauss on Jan 24 2018, 1:14 PM.

Details

Reviewers
skelly
Group Reviewers
Frameworks
Summary

kauth-gen-policy appears to be a tool for turning a text file in
qsettings format (similar to .ini) into an xml file. Both are textual,
so it seems like putting kauth-gen-policy into a Multi-Arch: foreign
package (like kconfig_compiler_kf5 in #887452).

In a cross compilation environment, kauth-gen-policy is called from the
host system and not from the target system, so a arch depdend path
(/usr/lib/<triplet>) do not make sense in that case and a arch indepdend
should be used. As we do not want to break other usecases we used the
non indrudive way possible, to change this to an option, so we can
modify the path with a camke option.

The bigger picture is to fix libexec path in extra-cmake-modules to a
arch-independed path, as recommended in the FHS 3.0.

887452 https://bugs.debian.org/887452

Bug-Debian: https://bugs.debian.org/887521

Diff Detail

Repository
R283 KAuth
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
knauss created this revision.Jan 24 2018, 1:14 PM
Restricted Application added a project: Frameworks. · View Herald TranscriptJan 24 2018, 1:14 PM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
knauss requested review of this revision.Jan 24 2018, 1:14 PM

@skelly: IMO it makes sense to discuss also the needed change in ECM, to solve the issue popperly.

Just a side note - me personally is not very deep into the cross compilation topic. That's why I can understand, if you have many questions and do not get it, why this is needed :D

Please feel free to ask any questions, I will try to anwser them, of forward them, if needed.
Also feel welcome to join #debian-qt-kde on irc.debian.org to discuss this more direct.

krop added a subscriber: krop.Jan 24 2018, 2:10 PM
krop added inline comments.
CMakeLists.txt
45

the 3rd parameter is a bool. This change won't do what you expect.

(https://cmake.org/cmake/help/v3.0/command/option.html)

knauss added inline comments.Apr 21 2018, 5:39 PM
CMakeLists.txt
45

well it compiles like I expected it... but yeah I see this is a bug not a feature :)