Added arguments to NaCl scons to allow custom toolchain directories.

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 json
 import os
 import platform
 import re
@@ skipping 130 matching lines @@
     # the discussion of why this argument is needed.
     'memcheck_command',
     # If the replacement memcheck command only works for trusted code,
     # set memcheck_trusted_only to non-zero.
     'memcheck_trusted_only',
     # colon-separated list of linker flags, e.g. "-lfoo:-Wl,-u,bar".
     'nacl_linkflags',
     # colon-separated list of pnacl bcld flags, e.g. "-lfoo:-Wl,-u,bar".
     # Not using nacl_linkflags since that gets clobbered in some tests.
     'pnacl_bcldflags',
-    'naclsdk_mode',
-    'pnaclsdk_mode',
     'platform',
     # Run tests under this tool (e.g. valgrind, tsan, strace, etc).
     # If the tool has options, pass them after comma: 'tool,--opt1,--opt2'.
     # NB: no way to use tools the names or the args of
     # which contains a comma.
     'run_under',
     # More args for the tool.
     'run_under_extra_args',
     # Multiply timeout values by this number.
     'scale_timeout',
@@ skipping 16 matching lines @@
     # activates buildbot-specific presets
     'buildbot',
     # Where to install header files for public consumption.
     'includedir',
     # Where to install libraries for public consumption.
     'libdir',
     # Where to install trusted-code binaries for public (SDK) consumption.
     'bindir',
     # Where a Breakpad build output directory is for optional Breakpad testing.
     'breakpad_tools_dir',
-    # Allows overriding the toolchain to use. The default toolchain will be
-    # a combination of the other arguments. Example toolchains:
-    #  NaCl (newlib): nacl_PLATFORM_newlib
-    #  NaCl (glibc):  nacl_PLATFORM_glibc
-    #  pnacl:         pnacl_PLATFORM
-    'toolchain',
+    # Allows overriding of the nacl newlib toolchain directory.
+    'nacl_newlib_dir',
+    # Allows override of the nacl glibc toolchain directory.
+    'nacl_glibc_dir',
+    # Allows override of the pnacl newlib toolchain directory.
+    'pnacl_newlib_dir',
     # Allows overriding the version number in the toolchain's
     # FEATURE_VERSION file.  This is used for PNaCl ABI compatibility
     # testing.
     'toolchain_feature_version',
   ])
 
 
 # Overly general to provide compatibility with existing build bots, etc.
 # In the future it might be worth restricting the values that are accepted.
 _TRUE_STRINGS = set(['1', 'true', 'yes'])
@@ skipping 959 matching lines @@
   # Currently 32/64 share the same tool build target directory. When we have
   # separate toolchains for each the architectures will probably have to use
   # the Arch3264() variant.
   build_arch = pynacl.platform.GetArch(GetBuildPlatform())
 
   return '%s_%s' % (os_name, build_arch)
 
 pre_base_env.AddMethod(GetPlatformBuildTargetDir)
 
 
-def GetToolchainName(env, target_arch=None, is_pnacl=None, lib_name=None):
-  toolchain = ARGUMENTS.get('toolchain', None)
-  if toolchain is None:
+def GetToolchainDir(env, platform_build_dir=None, toolchain_name=None,
+                    target_arch=None, is_pnacl=None, lib_name=None):
+  if platform_build_dir is None:
+    platform_build_dir = env.GetPlatformBuildTargetDir()
+
+  if toolchain_name is None:
+    # Fill in default arguments based on environment.
     if is_pnacl is None:
       is_pnacl = env.Bit('bitcode')
-    if lib_name is None:
-      if is_pnacl or not env.Bit('nacl_glibc'):
-        lib_name = 'newlib'
-      else:
-        lib_name = 'glibc'
+      if lib_name is None:
+        if is_pnacl or not env.Bit('nacl_glibc'):
+          lib_name = 'newlib'
+        else:
+          lib_name = 'glibc'
 
-    build_arch = pynacl.platform.GetArch(GetBuildPlatform())
     if target_arch is None:
       target_arch = pynacl.platform.GetArch(GetTargetPlatform())
 
     if is_pnacl:
       target_env = 'pnacl'
     else:
       target_env = 'nacl_%s' % target_arch
 
-    toolchain = '%s_%s' % (target_env, lib_name)
+    # See if we have a custom toolchain directory set.
+    if is_pnacl:
+      toolchain_arg = 'pnacl_%s_dir' % lib_name
+    else:
+      toolchain_arg = 'nacl_%s_dir' % lib_name
 
-  return toolchain
+    custom_toolchain_dir = ARGUMENTS.get(toolchain_arg, None)
+    if custom_toolchain_dir:
+      return env.SConstructAbsPath(custom_toolchain_dir)
 
-pre_base_env.AddMethod(GetToolchainName)
+    # Get the standard toolchain name since no directory custom was found.
+    if is_pnacl:
+      target_env = 'pnacl'
+    else:
+      target_env = 'nacl_%s' % target_arch
+    toolchain_name = '%s_%s' % (target_env, lib_name)
 
-
-def GetToolchainDir(env, platform_build_dir=None, toolchain_name=None):
-  if platform_build_dir is None:
-    platform_build_dir = env.GetPlatformBuildTargetDir()
-  if toolchain_name is None:
-    toolchain_name = env.GetToolchainName()
-
-  toolchain_sub_dir = os.path.join(
-      'toolchain',
-      platform_build_dir,
-      toolchain_name
-  )
+  # Get the absolute path for the platform build directory and toolchain.
+  toolchain_sub_dir = os.path.join('toolchain',
+                                   platform_build_dir,
+                                   toolchain_name)
   return env.SConstructAbsPath(toolchain_sub_dir)
 
 pre_base_env.AddMethod(GetToolchainDir)
 
 
 def GetSelLdr(env):
   sel_ldr = ARGUMENTS.get('force_sel_ldr')
   if sel_ldr:
     return env.File(env.SConstructAbsPath(sel_ldr))
 
@@ skipping 1013 matching lines @@
 * build just libraries:         scons build_lib
 * install just headers:         scons install_headers includedir=...
 * install just libraries:       scons install_lib libdir=...
 * install headers and libraries:scons install includedir=... libdir=...
 
 * dump system info:   scons --mode=nacl,opt-linux dummy
 
 Options:
 --------
 
-naclsdk_mode=<mode>   where <mode>:
-
-                    'local': use locally installed sdk kit
-                    'download': use the download copy (default)
-                    'custom:<path>': use kit at <path>
-                    'manual': use settings from env vars NACL_SDK_xxx
-
-pnaclsdk_mode=<mode> where <mode:
-                    'default': use the default (typically the downloaded copy)
-                    'custom:<path>': use kit from <path>
-
 --prebuilt          Do not build things, just do install steps
 
 --verbose           Full command line logging before command execution
 
 pp=1                Use command line pretty printing (more concise output)
 
 sysinfo=1           Verbose system info printing
 
 naclsdk_validate=0  Suppress presence check of sdk
 
@@ skipping 1572 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:')