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

Unified Diff: build/common.gypi

Issue 1497743004: Remove run_pie from android/tools now that ICS support is deprecated (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years 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
Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index 0974b49752e64eb3aaa4cd9c89bc79f6fbaeb110..e0e4dc9b2445babf2dd6dfe7dda35ad73875069a 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -4955,20 +4955,14 @@
],
'target_conditions': [
['_type=="executable"', {
- # Force android tools to export the "main" symbol so they can be
- # loaded on ICS using the run_pie wrapper. See crbug.com/373219.
- # TODO(primiano): remove -fvisibility and -rdynamic flags below
- # when ICS support will be dropped.
'cflags': [
'-fPIE',
- '-fvisibility=default',
],
'ldflags': [
'-Bdynamic',
'-Wl,--gc-sections',
'-Wl,-z,nocopyreloc',
'-pie',
- '-rdynamic',
# crtbegin_dynamic.o should be the last item in ldflags.
'<(android_ndk_lib)/crtbegin_dynamic.o',
],

Powered by Google App Engine
This is Rietveld 408576698