Merge lp:~pete-woods/thumbnailer/cmake-extras-compatibility into lp:thumbnailer

Proposed by Pete Woods
Status: Merged
Merged at revision: 155
Proposed branch: lp:~pete-woods/thumbnailer/cmake-extras-compatibility
Merge into: lp:thumbnailer
Diff against target: 85 lines (+4/-47)
4 files modified
CMakeLists.txt (+2/-3)
cmake/UseGSettings.cmake (+0/-42)
debian/control (+1/-1)
doc/libthumbnailer-qt/CMakeLists.txt (+1/-1)
To merge this branch: bzr merge lp:~pete-woods/thumbnailer/cmake-extras-compatibility
Reviewer Review Type Date Requested Status
Michi Henning (community) Approve
Review via email: mp+312711@code.launchpad.net

Commit message

Compatibility with unversioned cmake-extras modules

Description of the change

Compatibility with unversioned cmake-extras modules

To post a comment you must log in.
Revision history for this message
Michi Henning (michihenning) wrote :

Works find for both coverage, doxygen, and gsettings, thanks!

review: Approve
Revision history for this message
Michi Henning (michihenning) wrote :

Re-approving this after Jenkins problems were fixed.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt 2016-11-08 22:32:22 +0000
+++ CMakeLists.txt 2016-12-07 16:59:23 +0000
@@ -119,9 +119,8 @@
119include(CTest)119include(CTest)
120enable_testing()120enable_testing()
121121
122include(EnableCoverageReport)122find_package(CoverageReport)
123123find_package(GSettings)
124include(cmake/UseGSettings.cmake)
125124
126set(SHARE_PRIV_DIR ${CMAKE_INSTALL_LIBDIR}/thumbnailer)125set(SHARE_PRIV_DIR ${CMAKE_INSTALL_LIBDIR}/thumbnailer)
127set(SHARE_PRIV_ABS ${CMAKE_INSTALL_FULL_LIBDIR}/thumbnailer)126set(SHARE_PRIV_ABS ${CMAKE_INSTALL_FULL_LIBDIR}/thumbnailer)
128127
=== removed directory 'cmake'
=== removed file 'cmake/UseGSettings.cmake'
--- cmake/UseGSettings.cmake 2014-09-16 12:12:21 +0000
+++ cmake/UseGSettings.cmake 1970-01-01 00:00:00 +0000
@@ -1,42 +0,0 @@
1# GSettings.cmake, CMake macros written for Marlin, feel free to re-use them.
2
3option (GSETTINGS_LOCALINSTALL "Install GSettings Schemas locally instead of to the GLib prefix" ${LOCAL_INSTALL})
4
5option (GSETTINGS_COMPILE "Compile GSettings Schemas after installation" ${GSETTINGS_LOCALINSTALL})
6
7if(GSETTINGS_LOCALINSTALL)
8 message(STATUS "GSettings schemas will be installed locally.")
9endif()
10
11if(GSETTINGS_COMPILE)
12 message(STATUS "GSettings shemas will be compiled.")
13endif()
14
15macro(add_schema SCHEMA_NAME)
16
17 set(PKG_CONFIG_EXECUTABLE pkg-config)
18 # Have an option to not install the schema into where GLib is
19 if (GSETTINGS_LOCALINSTALL)
20 SET (GSETTINGS_DIR "${CMAKE_INSTALL_PREFIX}/share/glib-2.0/schemas/")
21 else (GSETTINGS_LOCALINSTALL)
22 execute_process (COMMAND ${PKG_CONFIG_EXECUTABLE} glib-2.0 --variable prefix OUTPUT_VARIABLE _glib_prefix OUTPUT_STRIP_TRAILING_WHITESPACE)
23 SET (GSETTINGS_DIR "${_glib_prefix}/share/glib-2.0/schemas/")
24 endif (GSETTINGS_LOCALINSTALL)
25
26 # Run the validator and error if it fails
27 execute_process (COMMAND ${PKG_CONFIG_EXECUTABLE} gio-2.0 --variable glib_compile_schemas OUTPUT_VARIABLE _glib_comple_schemas OUTPUT_STRIP_TRAILING_WHITESPACE)
28 execute_process (COMMAND ${_glib_comple_schemas} --dry-run --schema-file=${CMAKE_CURRENT_SOURCE_DIR}/${SCHEMA_NAME} ERROR_VARIABLE _schemas_invalid OUTPUT_STRIP_TRAILING_WHITESPACE)
29
30 if (_schemas_invalid)
31 message (SEND_ERROR "Schema validation error: ${_schemas_invalid}")
32 endif (_schemas_invalid)
33
34 # Actually install and recomple schemas
35 message (STATUS "GSettings schemas will be installed into ${GSETTINGS_DIR}")
36 install (FILES ${CMAKE_CURRENT_SOURCE_DIR}/${SCHEMA_NAME} DESTINATION ${GSETTINGS_DIR} OPTIONAL)
37
38 if (GSETTINGS_COMPILE)
39 install (CODE "message (STATUS \"Compiling GSettings schemas\")")
40 install (CODE "execute_process (COMMAND ${_glib_comple_schemas} ${GSETTINGS_DIR})")
41 endif ()
42endmacro()
430
=== modified file 'debian/control'
--- debian/control 2016-11-14 01:05:29 +0000
+++ debian/control 2016-12-07 16:59:23 +0000
@@ -4,7 +4,7 @@
4Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>4Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
5Standards-Version: 3.9.65Standards-Version: 3.9.6
6Build-Depends: cmake,6Build-Depends: cmake,
7 cmake-extras (>= 0.4),7 cmake-extras (>= 0.10),
8 debhelper (>= 9),8 debhelper (>= 9),
9 devscripts,9 devscripts,
10 google-mock,10 google-mock,
1111
=== modified file 'doc/libthumbnailer-qt/CMakeLists.txt'
--- doc/libthumbnailer-qt/CMakeLists.txt 2015-12-12 00:05:29 +0000
+++ doc/libthumbnailer-qt/CMakeLists.txt 2016-12-07 16:59:23 +0000
@@ -1,4 +1,4 @@
1include(UseDoxygen OPTIONAL)1find_package(DoxygenBuilder)
22
3file(GLOB libthumbnailer_headers "${PROJECT_SOURCE_DIR}/include/unity/thumbnailer/qt/*.h")3file(GLOB libthumbnailer_headers "${PROJECT_SOURCE_DIR}/include/unity/thumbnailer/qt/*.h")
44

Subscribers

People subscribed via source and target branches

to all changes: