Avoid unnecessary recompilation on mac when enabling/disabling distcc. Always...

site_scons/site_tools/distcc.py

 #!/usr/bin/python2.4
 # Copyright 2009, Google Inc.
 # All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
@@ skipping 47 matching lines @@
         action='store_true',
         help='enable distcc support')
     SCons.Script.Help('  --distcc                    Enable distcc suport.\n')
 
   except (OptionConflictError, optparse.OptionConflictError):
     # The distcc tool can be specified for multiple platforms, but the
     # --distcc option can only be added once.  Ignore the error which
     # results from trying to add it a second time.
     pass
 
+  # Modify compilers we support
+  distcc_compilers = env.get('DISTCC_COMPILERS', ['cc', 'gcc', 'c++', 'g++'])
+
+  if sys.platform == 'darwin':
+    for compiler_var in ('CC', 'CXX'):
+      compiler = env.get(compiler_var)
+      if compiler in distcc_compilers:
+        # On Mac, distcc requires the full path to the compiler. Even
+        # if distcc is disabled, change the compiler name to the
+        # full-path so that the compiler name is the same whether
+        # distcc is enabled or not. This prevents unnecessary
+        # recompilation when distcc is turned on/off. If we can't find
+        # the compiler, leave the non-absolute pathname as a tip-off
+        # that there's a problem and avoid the string concatenation
+        # with the none type.
+        compiler_path = env.WhereIs(compiler)
+        if compiler_path is not None:
+          env[compiler_var] = compiler_path
+
   # If distcc isn't enabled, stop now
   if not env.GetOption('distcc'):
     return
 
   # Copy DISTCC_HOSTS and HOME environment variables from system environment
   for envvar in ('DISTCC_HOSTS', 'HOME'):
     value = env.get(envvar, os.environ.get(envvar))
     if not value:
       print 'Warning: %s not set in environment; disabling distcc.' % envvar
       return
     env['ENV'][envvar] = value
 
+  # For distcc pump mode, copy the following variable only if they exist.
+  for envvar in ('INCLUDE_SERVER_PID', 'INCLUDE_SERVER_DIR',
+                 'INCLUDE_SERVER_PORT'):
+    value = env.get(envvar, os.environ.get(envvar))
+    if value:
+      env['ENV'][envvar] = value
+
   # Set name of distcc tool
   env['DISTCC'] = 'distcc'
 
-  # Modify compilers we support
-  distcc_compilers = env.get('DISTCC_COMPILERS', ['cc', 'gcc', 'c++', 'g++'])
+  # Change distcc_compilers to absolute paths for darwin systems since
+  # we changed the compiler variables in the environment to full-paths
+  # earlier.
+  if sys.platform == 'darwin':
+    for i in range(len(distcc_compilers)):
+      full_path = env.WhereIs(distcc_compilers[i])
+      if full_path is not None:
+        distcc_compilers[i] = full_path
+
   for compiler_var in ('CC', 'CXX'):
     compiler = env.get(compiler_var)
     if compiler in distcc_compilers:
-      if sys.platform == 'darwin':
-        # On Mac, distcc requires the full path to the compiler
-        compiler = env.WhereIs(compiler)
-      env[compiler_var] = '$DISTCC ' + compiler
+      # Surround DISTCC command with "ignore" tags to avoid
+      # recompiling unnecessarily when distcc is turned on/off
+      env[compiler_var] = '$( $DISTCC $) ' + compiler