Review Request 115684: Generate local forwarding headers under a local subdir, to fix clash on Mac OS X.
David Faure
faure at kde.org
Sat Feb 15 19:22:35 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115684/
-----------------------------------------------------------
(Updated Feb. 15, 2014, 7:22 p.m.)
Status
------
This change has been discarded.
Review request for Build System, Extra Cmake Modules, KDE Frameworks, and Harald Fernengel.
Repository: extra-cmake-modules
Description
-------
Generate local forwarding headers under a local subdir, to fix clash on Mac OS X.
This is intended to replace RR 115541.
With case-insensitive filesystems, creating KParts and kparts subdirs
in the same parent was obviously a bad idea, especially since we then
make a copy of "KParts" and don't expect the contents of "kparts" to tag along.
Solved by making that KParts (installed) and local/kparts (not installed).
Downside: the modules that use this PREFIX feature need a change like this:
-target_include_directories(KF5Parts PUBLIC "$<BUILD_INTERFACE:${KParts_BINARY_DIR}>")
+target_include_directories(KF5Parts PUBLIC "$<BUILD_INTERFACE:${KParts_BINARY_DIR};${CMAKE_CURRENT_BINARY_DIR}/local>")
Easily scripted though:
perl -pi -e 's/>/\;\${CMAKE_CURRENT_BINARY_DIR}\/local>/ if (/target_include_directories/ && /PUBLIC/)' `grep -rwl PREFIX .`
Diffs
-----
modules/ECMGenerateHeaders.cmake e98a22e91151d23d7c798ff22a33097ec2a59d10
Diff: https://git.reviewboard.kde.org/r/115684/diff/
Testing
-------
Applied it, ran the perl script, and a full build-from-scratch worked.
Not tested on a Mac, though :)
Thanks,
David Faure
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20140215/9a684ef8/attachment.html>
More information about the Kde-buildsystem
mailing list