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

Unified Diff: site_scons/site_tools/naclsdk.py

Issue 9824001: Change built_elsewhere flag to work with pexe mode (Closed) Base URL: svn://svn.chromium.org/native_client/trunk/src/native_client
Patch Set: add AS, line len Created 8 years, 9 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
Index: site_scons/site_tools/naclsdk.py
diff --git a/site_scons/site_tools/naclsdk.py b/site_scons/site_tools/naclsdk.py
index 0ff30f136692e4096403657c8a7d36e2565048a3..185ac8e6dd4159a1f3ab01c8241820f815c710fa 100755
--- a/site_scons/site_tools/naclsdk.py
+++ b/site_scons/site_tools/naclsdk.py
@@ -325,6 +325,14 @@ def _SetEnvForPnacl(env, root):
TRANSLATE=pnacl_translate + arch_flag + pnacl_translate_flags,
)
+ if env.Bit('built_elsewhere'):
+ def FakeInstall(dest, source, env):
+ print 'Not installing', dest
+ env.Replace(CC='true', CXX='true', LINK='true', AR='true',
+ RANLIB='true', AS='true', LD='true', INSTALL=FakeInstall,
jvoung - send to chromium... 2012/03/21 23:15:04 Do you need to stub out STRIP as well?
(google.com) Derek Schuff 2012/03/22 00:02:02 Apparently not, since it works without it? couldn'
+ #TODO(dschuff) remove this when we can translate on arm hw
+ TRANSLATE='true')
+
def _SetEnvForSdkManually(env):
def GetEnvOrDummy(v):

Powered by Google App Engine
This is Rietveld 408576698