Merge lp:~dktrkranz/granite/non-linux-build into lp:~elementary-pantheon/granite/granite

Proposed by Luca Falavigna
Status: Merged
Approved by: Victor Martinez
Approved revision: 467
Merged at revision: 466
Proposed branch: lp:~dktrkranz/granite/non-linux-build
Merge into: lp:~elementary-pantheon/granite/granite
Diff against target: 54 lines (+11/-1)
2 files modified
lib/Application.vala (+4/-0)
lib/CMakeLists.txt (+7/-1)
To merge this branch: bzr merge lp:~dktrkranz/granite/non-linux-build
Reviewer Review Type Date Requested Status
Victor Martinez (community) Approve
Devid Antonio Filoni (community) Needs Fixing
Review via email: mp+135765@code.launchpad.net

Description of the change

This new patch fixes a mistake I did in my previous patch

To post a comment you must log in.
Revision history for this message
Victor Martinez (victored) wrote :

I cannot test this on a non-linux system ATM, but the CMake-side implementation is reliable. Thanks for the quick fix!

review: Approve
Revision history for this message
Devid Antonio Filoni (d.filoni) wrote :

There is a typo on this patch:
 ${OS_TYPE)

review: Needs Fixing
Revision history for this message
Victor Martinez (victored) wrote :

Good catch Devid. I didn't see that ')' on line 21 (diff)

review: Needs Fixing
466. By Luca Falavigna

Fix typo

Revision history for this message
Luca Falavigna (dktrkranz) wrote :

Thanks, fixed.

467. By Luca Falavigna

Merge with trunk

Revision history for this message
Victor Martinez (victored) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/Application.vala'
--- lib/Application.vala 2012-11-23 01:19:02 +0000
+++ lib/Application.vala 2012-11-23 08:52:30 +0000
@@ -69,7 +69,9 @@
6969
70 Granite.init ();70 Granite.init ();
71 // set program name71 // set program name
72#if LINUX
72 prctl (15, exec_name, 0, 0, 0);73 prctl (15, exec_name, 0, 0, 0);
74#endif
73 Environment.set_prgname (exec_name);75 Environment.set_prgname (exec_name);
7476
75 Logger.initialize (program_name);77 Logger.initialize (program_name);
@@ -86,8 +88,10 @@
8688
87 }89 }
8890
91#if LINUX
89 [CCode (cheader_filename = "sys/prctl.h", cname = "prctl")]92 [CCode (cheader_filename = "sys/prctl.h", cname = "prctl")]
90 protected extern static int prctl (int option, string arg2, ulong arg3, ulong arg4, ulong arg5);93 protected extern static int prctl (int option, string arg2, ulong arg3, ulong arg4, ulong arg5);
94#endif
9195
92 /**96 /**
93 * This method runs the application97 * This method runs the application
9498
=== modified file 'lib/CMakeLists.txt'
--- lib/CMakeLists.txt 2012-11-23 01:19:02 +0000
+++ lib/CMakeLists.txt 2012-11-23 08:52:30 +0000
@@ -44,6 +44,12 @@
44 ${DEPS_LIBRARY_DIRS}44 ${DEPS_LIBRARY_DIRS}
45)45)
46link_directories(${LIB_PATHS})46link_directories(${LIB_PATHS})
47
48# Check whether OS type is Linux
49if (CMAKE_SYSTEM_NAME MATCHES "Linux")
50 set (OS_TYPE --define=LINUX)
51endif ()
52
47#VALA stuff53#VALA stuff
4854
49include_directories(${CMAKE_CURRENT_SOURCE_DIR}/Widgets/)55include_directories(${CMAKE_CURRENT_SOURCE_DIR}/Widgets/)
@@ -96,6 +102,7 @@
96 gee-1.0102 gee-1.0
97OPTIONS103OPTIONS
98 --thread104 --thread
105 ${OS_TYPE}
99GENERATE_VAPI106GENERATE_VAPI
100 granite107 granite
101GENERATE_HEADER108GENERATE_HEADER
@@ -148,4 +155,3 @@
148add_custom_command (TARGET valadocs COMMAND rm ${CMAKE_CURRENT_BINARY_DIR}/images -R -f)155add_custom_command (TARGET valadocs COMMAND rm ${CMAKE_CURRENT_BINARY_DIR}/images -R -f)
149add_custom_command (TARGET valadocs COMMAND cp ${CMAKE_SOURCE_DIR}/doc/images/ ${CMAKE_CURRENT_BINARY_DIR}/images -R)156add_custom_command (TARGET valadocs COMMAND cp ${CMAKE_SOURCE_DIR}/doc/images/ ${CMAKE_CURRENT_BINARY_DIR}/images -R)
150add_custom_command (TARGET valadocs COMMAND VALADOC_HEADER=${CMAKE_CURRENT_SOURCE_DIR}/docs/header.html valadoc -o docs_vala ${CMAKE_CURRENT_SOURCE_DIR}/*/*.vala ${CMAKE_CURRENT_SOURCE_DIR}/*.vapi ${CMAKE_CURRENT_SOURCE_DIR}/*.vala --pkg=glib-2.0 --pkg=gio-2.0 --pkg=posix --pkg=gtk+-3.0 --pkg=gee-1.0 --pkg=gio-unix-2.0 --package-name=${PKGNAME} --package-version=${GRANITE_VERSION} --force --doclet=html)157add_custom_command (TARGET valadocs COMMAND VALADOC_HEADER=${CMAKE_CURRENT_SOURCE_DIR}/docs/header.html valadoc -o docs_vala ${CMAKE_CURRENT_SOURCE_DIR}/*/*.vala ${CMAKE_CURRENT_SOURCE_DIR}/*.vapi ${CMAKE_CURRENT_SOURCE_DIR}/*.vala --pkg=glib-2.0 --pkg=gio-2.0 --pkg=posix --pkg=gtk+-3.0 --pkg=gee-1.0 --pkg=gio-unix-2.0 --package-name=${PKGNAME} --package-version=${GRANITE_VERSION} --force --doclet=html)
151

Subscribers

People subscribed via source and target branches