Merge lp:~codygarver/pantheon-friends/cmake-cleanup into lp:pantheon-friends

Proposed by Cody Garver
Status: Merged
Approved by: David Gomes
Approved revision: 3
Merged at revision: 2
Proposed branch: lp:~codygarver/pantheon-friends/cmake-cleanup
Merge into: lp:pantheon-friends
Diff against target: 103 lines (+44/-38)
1 file modified
CMakeLists.txt (+44/-38)
To merge this branch: bzr merge lp:~codygarver/pantheon-friends/cmake-cleanup
Reviewer Review Type Date Requested Status
David Gomes (community) Approve
Review via email: mp+185948@code.launchpad.net

Commit message

CMakeLists.txt: Clean up and strict dependency requirements

Description of the change

This needs testing, I don't have the require GTK so I can't confirm it works for sure.

To post a comment you must log in.
Revision history for this message
Rico Tzschichholz (ricotz) wrote :

Actually valac 0.21.1 is required for having gtk 3.9.x support.

3. By Cody Garver

Bump valac to 0.21.1 as advised by Rico

Revision history for this message
Cody Garver (codygarver) wrote :

Corrected, thanks.

On Tue, Sep 17, 2013 at 12:36 AM, Rico Tzschichholz <email address hidden>wrote:

> Actually valac 0.21.1 is required for having gtk 3.9.x support.
> --
>
> https://code.launchpad.net/~codygarver/pantheon-friends/cmake-cleanup/+merge/185948
> Your team Developers of Friends is requested to review the proposed merge
> of lp:~codygarver/pantheon-friends/cmake-cleanup into lp:pantheon-friends.
>

--
Cody Garver

Revision history for this message
David Gomes (davidgomes) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'CMakeLists.txt'
2--- CMakeLists.txt 2013-09-16 14:37:45 +0000
3+++ CMakeLists.txt 2013-09-17 06:03:41 +0000
4@@ -1,56 +1,62 @@
5-
6-set (NAME friends)
7-set (PACKAGES granite telepathy-glib)
8-set (FILES src/Friends.vala src/main_window.vala src/Widget/contact_box.vala src/Widget/notification_button.vala src/Widget/request_box.vala src/Lib/telepathy_manager.vala)
9-
10-
11-project (${NAME})
12-cmake_minimum_required (VERSION 2.8)
13+cmake_minimum_required (VERSION 2.6)
14 cmake_policy (VERSION 2.6)
15-
16-list (APPEND CMAKE_MODULE_PATH ${CMAKE_SOURCE_DIR}/cmake/vala)
17-
18-set (CMAKE_INSTALL_PREFIX /usr)
19-
20-enable_testing ()
21-
22+project (friends C)
23+
24+list (APPEND CMAKE_MODULE_PATH ${CMAKE_SOURCE_DIR}/cmake)
25+
26 set (DATADIR "${CMAKE_INSTALL_PREFIX}/share")
27-set (PKGDATADIR "${DATADIR}/friends")
28-set (GETTEXT_PACKAGE "${NAME}")
29+set (PKGDATADIR "${DATADIR}/${CMAKE_PROJECT_NAME}")
30+set (GETTEXT_PACKAGE "${CMAKE_PROJECT_NAME}")
31+set (EXEC_NAME "${CMAKE_PROJECT_NAME}")
32+set (RELEASE_NAME "Isis")
33 set (VERSION "0.1")
34-
35-list (APPEND CMAKE_MODULE_PATH ${CMAKE_SOURCE_DIR}/cmake)
36-
37+set (VERSION_INFO "isis")
38+set (CMAKE_C_FLAGS "-ggdb")
39+set (PREFIX ${CMAKE_INSTALL_PREFIX})
40+set (DOLLAR "$")
41+
42+enable_testing ()
43+
44 configure_file (${CMAKE_SOURCE_DIR}/src/Config.vala.cmake ${CMAKE_BINARY_DIR}/src/Config.vala)
45 add_definitions(-DGETTEXT_PACKAGE=\"${GETTEXT_PACKAGE}\")
46-
47-find_package(PkgConfig)
48-pkg_check_modules(DEPS REQUIRED ${PACKAGES})
49-
50+
51 add_definitions (${DEPS_CFLAGS})
52 link_libraries (${DEPS_LIBRARIES})
53 link_directories (${DEPS_LIBRARY_DIRS})
54-
55+
56 find_package (Vala REQUIRED)
57 include (ValaVersion)
58-ensure_vala_version (0.16.0 MINIMUM)
59-
60+ensure_vala_version (0.21.1 MINIMUM)
61+
62+set (FRIENDS_DEPS
63+ granite
64+ gtk+-3.0>=3.9.14
65+ telepathy-glib)
66+
67+find_package(PkgConfig)
68+pkg_check_modules(DEPS REQUIRED ${FRIENDS_DEPS})
69+
70 include (ValaPrecompile)
71-vala_precompile (VALA_C ${NAME}
72- ${FILES}
73+vala_precompile (VALA_C ${CMAKE_PROJECT_NAME}
74+ src/Friends.vala
75+ src/main_window.vala
76+ src/Lib/telepathy_manager.vala
77+ src/Widget/contact_box.vala
78+ src/Widget/notification_button.vala
79+ src/Widget/request_box.vala
80 PACKAGES
81- ${PACKAGES}
82+ ${FRIENDS_DEPS}
83 OPTIONS
84- ${VALAFLAGS}
85+ --thread
86 )
87-
88+
89 add_subdirectory (po)
90-
91+
92 include (GSettings)
93 add_schema (data/org.pantheon.friends.gschema.xml)
94-
95-add_executable (${NAME} ${VALA_C})
96-
97-install (TARGETS ${NAME} RUNTIME DESTINATION bin)
98-install (FILES ${CMAKE_CURRENT_SOURCE_DIR}/data/${NAME}.desktop DESTINATION share/applications)
99+
100+add_executable (${CMAKE_PROJECT_NAME} ${VALA_C})
101+
102+install (TARGETS ${CMAKE_PROJECT_NAME} RUNTIME DESTINATION bin)
103+install (FILES ${CMAKE_CURRENT_SOURCE_DIR}/data/${CMAKE_PROJECT_NAME}.desktop DESTINATION ${DATADIR}/applications)
104

Subscribers

People subscribed via source and target branches