Change built_elsewhere flag to work with pexe mode

site_scons/site_tools/naclsdk.py

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..5bbefaa35e3720b0dbde83bc2cf33da8d12ff536 100755
--- a/site_scons/site_tools/naclsdk.py
+++ b/site_scons/site_tools/naclsdk.py
@@ -325,6 +325,15 @@ 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',

[robertm, 2012/03/22 14:09:37]

if it is not too much work it be nice to make the faked out
command more verbose, e.g.

 'Skip CC', dest
'Skip CXX', dest

etc

[(google.com) Derek Schuff, 2012/03/22 16:10:25]

How would that work? Will it work if I just pass a python function here instead
of a shell command?

On 2012/03/22 14:09:37, robertm wrote:

[robertm, 2012/03/22 16:18:43]

not sure. 
another option could be CC='echo CC #' where the purpose of the '#' is to ignore
the command line.
if the simple stuff does not work don't bother

On 2012/03/22 16:10:25, Derek Schuff wrote:

+                RANLIB='true', AS='true', LD='true',
+                STRIP='true', INSTALL=FakeInstall,
+                #TODO(dschuff) remove this when we can translate on arm hw
+                TRANSLATE='true')
+
 
 def _SetEnvForSdkManually(env):
   def GetEnvOrDummy(v):
@@ -359,6 +368,8 @@ def PNaClForceNative(env):
              LINK=cc_flags) # Already has -arch
   env['LD'] = '${NATIVELD}' + arch_flag
   env['SHLINK'] = '${LINK}'
+  if env.Bit('built_elsewhere'):
+    env.Replace(CC='true', CXX='true', ASPP='true', LINK='true', LD='true')

[jvoung - send to chromium..., 2012/03/22 00:17:45]

Should this be done earlier?  Mainly concerned with how this interacts with the
pnacl_generate_pexe check above (with an early return).

[robertm, 2012/03/22 14:09:37]

if this ends up staying around you could unify this with the similar statement
above.

On 2012/03/22 00:17:45, jvoung wrote:

[(google.com) Derek Schuff, 2012/03/22 16:10:25]

The change above is for the nacl environment and applies in both pexe and
non-pexe mode. This change only applies in non-pexe mode since this function
should not be called otherwise (hence the check above). This change is just
necessary because the change to the way built_elsewhere is implemented affects
both pexe and non-pexe mode.

On 2012/03/22 00:17:45, jvoung wrote:

 
 # Get an environment for a different frontend when in
 # PNaCl mode.