Emit brief command strings for pexe translation.

SConstruct

 #! -*- python -*-
 # Copyright (c) 2012 The Native Client Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import atexit
 import glob
 import os
 import platform
 import shutil
@@ skipping 1950 matching lines @@
   return ('process_output_single' in extra or
           'log_golden' in extra)
 
 # ----------------------------------------------------------
 DeclareBit('tests_use_irt', 'Non-browser tests also load the IRT image', False)
 
 # Translate the given pexe.
 def GetTranslatedNexe(env, pexe):
   pexe_name = pexe.abspath
   nexe_name = pexe_name[:pexe_name.index('.pexe')] + '.nexe'
-  trans_cmd = ('${TRANSLATE} ${TRANSLATEFLAGS} -Wl,-L${LIB_DIR} %s -o %s' %
-               (pexe_name, nexe_name))
-  return env.Command(nexe_name, pexe_name, trans_cmd)
+  trans_cmd = ('${TRANSLATECOM} ${SOURCES} -o ${TARGET}')

[robertm, 2012/03/15 22:58:47]

the "${SOURCES} -o ${TARGET}" part
could me moved into TRANSLATECOM 

we do this for almost all other xxxCOM
not sure why STRIPCOM is special.

+
+  return env.Command(target=nexe_name, source=[pexe_name],
+                     action=[Action(trans_cmd, '${TRANSLATECOMSTR}')])
 
 pre_base_env.AddMethod(GetTranslatedNexe)
 
 def CommandSelLdrTestNacl(env, name, nexe,
                           args = None,
                           log_verbosity=2,
                           sel_ldr_flags=None,
                           loader=None,
                           size='medium',
                           # True for *.nexe statically linked with glibc
@@ skipping 267 matching lines @@
 def StripExecutable(env, name, exe):
   """StripExecutable returns a node representing the stripped version of |exe|.
      The stripped version will be given the basename |name|.
      NOTE: for now this only works with the untrusted toolchain.
      STRIP does not appear to be a first-class citizen in SCons and
      STRIP has only been set to point at the untrusted toolchain.
   """
   return env.Command(
       target=name,
       source=[exe],
-      action=[Action('${STRIPCOM} ${SOURCES} -o ${TARGET}')])
+      action=[Action('${STRIPCOM} ${SOURCES} -o ${TARGET}', '${STRIPCOMSTR}')])
 
 pre_base_env.AddMethod(StripExecutable)
 
 
 # TODO(ncbray): pretty up the log output when running this builder.
 def DisabledCommand(target, source, env):
   pass
 
 pre_base_env['BUILDERS']['DisabledCommand'] = Builder(action=DisabledCommand)
 
@@ skipping 1607 matching lines @@
   nacl_env.ValidateSdk()
 
 if BROKEN_TEST_COUNT > 0:
   msg = "There are %d broken tests." % BROKEN_TEST_COUNT
   if GetOption('brief_comstr'):
     msg += " Add --verbose to the command line for more information."
   print msg
 
 # separate warnings from actual build output
 Banner('B U I L D - O U T P U T:')