Review Request 119798: Generating PkgConfig files from ECM
Alex Merry
alex.merry at kde.org
Tue Sep 2 18:57:05 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119798/#review65735
-----------------------------------------------------------
OK, just some documentation cleanups, then I think we're good to go :-)
modules/ECMGeneratePkgConfigFile.cmake
<https://git.reviewboard.kde.org/r/119798/#comment45944>
Full stop at end. Note that this is the name that will be passed to pkg-config to find this module.
modules/ECMGeneratePkgConfigFile.cmake
<https://git.reviewboard.kde.org/r/119798/#comment45945>
Emphasise that this is *not* the CMake target name if the OUTPUT_NAME target property has been changed from the default.
modules/ECMGeneratePkgConfigFile.cmake
<https://git.reviewboard.kde.org/r/119798/#comment45942>
It doesn't receive a variable name, it should *be* a variable name, which will be set to the location of the generated file.
modules/ECMGeneratePkgConfigFile.cmake
<https://git.reviewboard.kde.org/r/119798/#comment45946>
s/are being/will be/
modules/ECMGeneratePkgConfigFile.cmake
<https://git.reviewboard.kde.org/r/119798/#comment45950>
Default values?
modules/ECMGeneratePkgConfigFile.cmake
<https://git.reviewboard.kde.org/r/119798/#comment45943>
Full stop at end.
s/is being/will be/
modules/ECMGeneratePkgConfigFile.cmake
<https://git.reviewboard.kde.org/r/119798/#comment45947>
"DEFINES is a list of preprocessor defines that it is recommended users of the library pass to the compiler when using it."
modules/ECMGeneratePkgConfigFile.cmake
<https://git.reviewboard.kde.org/r/119798/#comment45948>
"INSTALL will cause the module to be installed to the ``pkgconfig`` subdirectory of LIB_INSTALL_DIR, unless the ECM_PKGCONFIG_INSTALL_DIR cache variable is set to something different. Note that the first call to ecm_generate_pkgconfig_file with the INSTALL argument will cause ECM_PKGCONFIG_INSTALL_DIR to be set, and this will be used in any subsequent calls."
modules/ECMGeneratePkgConfigFile.cmake
<https://git.reviewboard.kde.org/r/119798/#comment45949>
"use the project() (CMake 3.0) or"
- Alex Merry
On Sept. 1, 2014, 1:53 p.m., Aleix Pol Gonzalez wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119798/
> -----------------------------------------------------------
>
> (Updated Sept. 1, 2014, 1:53 p.m.)
>
>
> Review request for Build System, KDE Frameworks and Harald Sitter.
>
>
> Repository: extra-cmake-modules
>
>
> Description
> -------
>
> So we decided we wanted those .pc files, so I created a small script that generates one, I haven't used pc in the past, so feedback is welcome.
>
>
> Diffs
> -----
>
> modules/ECMGeneratePkgConfigFile.cmake PRE-CREATION
> tests/CMakeLists.txt 65de038
> tests/ECMGeneratePkgConfigFile/CMakeLists.txt PRE-CREATION
> tests/ECMGeneratePkgConfigFile/KF5CoreAddons.pc PRE-CREATION
> tests/ECMGeneratePkgConfigFile/run_test.cmake.config PRE-CREATION
>
> Diff: https://git.reviewboard.kde.org/r/119798/diff/
>
>
> Testing
> -------
>
> I added it in KCoreAddons, this is the patch:
> diff --git src/lib/CMakeLists.txt src/lib/CMakeLists.txt
> index 26eb5a1..3a07d1c 100644
> --- src/lib/CMakeLists.txt
> +++ src/lib/CMakeLists.txt
> @@ -188,4 +188,6 @@ install(FILES
>
> include(ECMGeneratePriFile)
> ecm_generate_pri_file(BASE_NAME KCoreAddons LIB_NAME KF5CoreAddons DEPS "core" FILENAME_VAR PRI_FILENAME INCLUDE_INSTALL_DIR ${KF5_INCLUDE_INSTALL_DIR}/KCoreAddons)
> +ecm_generate_pkgconfig_file(BASE_NAME KCoreAddons LIB_NAME KF5CoreAddons DEPS Qt5Core INCLUDE_INSTALL_DIR ${KF5_INCLUDE_INSTALL_DIR}/KCoreAddons INSTALL)
> install(FILES ${PRI_FILENAME} DESTINATION ${ECM_MKSPECS_INSTALL_DIR})
>
> This is the result, on my system:
>
> Name: KF5CoreAddons
> Version: 5.1.0
> Libs: -L/home/kde-devel/kde5/lib64 -l/home/kde-devel/kde5/lib64
> Cflags: -I/home/kde-devel/kde5/include/KF5/KCoreAddons
> Requires: Qt5Core
>
>
> Thanks,
>
> Aleix Pol Gonzalez
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-buildsystem/attachments/20140902/2d0fab8b/attachment-0001.html>
More information about the Kde-buildsystem
mailing list