D5766: Change default pkgconfig install path for FreeBSD
Kevin Funk
noreply at phabricator.kde.org
Mon May 8 15:25:17 UTC 2017
kfunk added inline comments.
INLINE COMMENTS
> ECMGeneratePkgConfigFile.cmake:172
> if(EGPF_INSTALL)
> - set(ECM_PKGCONFIG_INSTALL_DIR "${EGPF_LIB_INSTALL_DIR}/pkgconfig" CACHE PATH "The directory where pkgconfig will be installed to.")
> + if(NOT CMAKE_SYSTEM_NAME matches "FreeBSD")
> + set(ECM_PKGCONFIG_INSTALL_DIR "${EGPF_LIB_INSTALL_DIR}/pkgconfig" CACHE PATH "The directory where pkgconfig will be installed to.")
`matches` -> `MATCHES` -- I'm surprised that actually works as-is.
Also: Why not `if (CMAKE_SYSTEM_NAME MATCHES ...)` and switch the bodies inside the if & else branch to simplify?
REPOSITORY
R240 Extra CMake Modules
REVISION DETAIL
https://phabricator.kde.org/D5766
To: tcberner, #freebsd, apol
Cc: kfunk, #frameworks, #build_system
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20170508/e6e2eed2/attachment.html>
More information about the Kde-buildsystem
mailing list