Merge lp:~elementary-apps/pantheon-terminal/fix-cmake-cppflags into lp:~elementary-apps/pantheon-terminal/trunk

Proposed by Sergey "Shnatsel" Davidoff
Status: Superseded
Proposed branch: lp:~elementary-apps/pantheon-terminal/fix-cmake-cppflags
Merge into: lp:~elementary-apps/pantheon-terminal/trunk
Diff against target: 22 lines (+2/-1)
2 files modified
CMakeLists.txt (+1/-0)
INSTALL (+1/-1)
To merge this branch: bzr merge lp:~elementary-apps/pantheon-terminal/fix-cmake-cppflags
Reviewer Review Type Date Requested Status
David Gomes (community) Needs Fixing
Review via email: mp+221631@code.launchpad.net

This proposal has been superseded by a proposal from 2014-06-03.

Commit message

Honor CPPFLAGS env variable in CMake to fix bug #1325329.

Description of the change

Honor CPPFLAGS env variable in CMake to fix bug #1325329

https://mondaybynoon.com/wp-content/uploads/xVyoSl.jpg

To post a comment you must log in.
Revision history for this message
David Gomes (davidgomes) :
review: Approve
Revision history for this message
David Gomes (davidgomes) wrote :

13:55:39 <ricotz> Munchor, hmm, http://www.cmake.org/pipermail/cmake/2009-April/028853.html

And put that newline in INSTALL back.

review: Needs Fixing
Revision history for this message
Rico Tzschichholz (ricotz) wrote :
576. By Launchpad Translations on behalf of elementary-apps

Launchpad automatic translations update.

Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

David: I've *added* the newline instead of removing it ;)

Rico: So what should I do? Apply this patch? Hack around in packaging? Or something else?

577. By Sergey "Shnatsel" Davidoff

Ditch the custom TerminalWidget.process_argv() method in favor of Glib.Shell.parse_argv() and drop a bash hardcoding in the process since it's no longer required. This fixes bug 1324577

578. By Launchpad Translations on behalf of elementary-apps

Launchpad automatic translations update.

579. By Launchpad Translations on behalf of elementary-apps

Launchpad automatic translations update.

580. By Launchpad Translations on behalf of elementary-apps

Launchpad automatic translations update.

581. By Phillip Sz

fix CMakeLists.txt URL and update to ubuntu 14.04

582. By Sergey "Shnatsel" Davidoff

Implement process competion notifications as per https://blueprints.launchpad.net/pantheon-terminal/+spec/process-completion-notifications

583. By Tom Beckmann

fix regression in previous commit that would assign no shell to be started if a custom one is set

584. By Launchpad Translations on behalf of elementary-apps

Launchpad automatic translations update.

585. By Launchpad Translations on behalf of elementary-apps

Launchpad automatic translations update.

586. By Sergey "Shnatsel" Davidoff

CMake: add contents of CPPFLAGS to CMake's flags, fixes hardening options not being passed (bug #1325329)

Unmerged revisions

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 2014-05-29 10:10:46 +0000
3+++ CMakeLists.txt 2014-06-01 08:57:22 +0000
4@@ -47,6 +47,7 @@
5 gdk-3.0)
6
7 add_definitions (${DEPS_CFLAGS})
8+add_definitions (${CPPFLAGS})
9
10 link_libraries (${DEPS_LIBRARIES})
11 link_directories (${DEPS_LIBRARY_DIRS})
12
13=== modified file 'INSTALL'
14--- INSTALL 2013-09-30 20:08:22 +0000
15+++ INSTALL 2014-06-01 08:57:22 +0000
16@@ -8,4 +8,4 @@
17 cd build/
18 make install #Might need sudo
19
20-To launch the Terminal locally (for development purposes), use "./pantheon-terminal" from the build/ directory.
21\ No newline at end of file
22+To launch the Terminal locally (for development purposes), use "./pantheon-terminal" from the build/ directory.

Subscribers

People subscribed via source and target branches