Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5725)

Unified Diff: chrome/chrome_exe.gypi

Issue 9414022: Linux: Remove references to '-nopie' now that it's fixed in cros and doesn't exist on Pangolin. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/common.gypi ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/chrome_exe.gypi
diff --git a/chrome/chrome_exe.gypi b/chrome/chrome_exe.gypi
index daa3630b10e46efa06abf51d5215029463873191..4a1daa6031a59eb28a5bd946c117217997e5be4c 100644
--- a/chrome/chrome_exe.gypi
+++ b/chrome/chrome_exe.gypi
@@ -97,18 +97,6 @@
],
},
],
- # TODO(rkc): Remove once crosbug.com/15266 is fixed.
- ['profiling==1', {
- 'ldflags': ['-nopie'],
- }, {
- # Building with -pie needs investigating on ARM.
- # For now, at least use it on Linux Intel.
- 'conditions': [
- ['(target_arch=="x64" or target_arch=="ia32") and linux_disable_pie!=1', {
- 'ldflags': ['-pie'],
- }],
- ],
- }],
['use_system_xdg_utils==0', {
'copies': [
{
« no previous file with comments | « build/common.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698