Update chrome/nacl.gypi to fix chromeos build

tools/ld_bfd/ld

 #!/usr/bin/python
 # Copyright (c) 2011 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Wrapper for invoking the BFD loader
 
-A simple script to invoke the bfd loader instead of gold, removing
-threading command line options that the bfd loader doesn't support.
-Because this script is invoked from gcc via the -B flag, it needs
-to be in a file named "ld".
+A simple script to invoke the bfd loader instead of gold.
+This script is in a filename "ld" so it can be invoked from gcc
+via the -B flag.
 """
-
+# TODO(bradchen): Delete this script when Gold supports linker scripts properly.
 import os
 import subprocess
 import sys
 
-# TODO(bradchen): Delete this script when Gold supports -Ttext properly.
-# It should be fixed by this patch:
-# http://sourceware.org/ml/binutils/2011-07/msg00206.html
+def PathTo(fname):
+  if fname[0] == os.pathsep:
+    return fname
+  for p in os.environ["PATH"].split(os.pathsep):
+    fpath = os.path.join(p, fname)
+    if os.path.exists(fpath):
+      return fpath
+  return fname
+
+def FindLDBFD():
+  cxx = os.getenv("CXX")
+  if not cxx:
+    cxx = "gcc"
+  popen = subprocess.Popen(cxx + " -print-prog-name=ld",
+                           shell=True,
+                           stdout=subprocess.PIPE,
+                           stdin=subprocess.PIPE)
+  (ld, error) = popen.communicate()
+  if popen.wait() != 0:
+    print "Could not find ld:" + error
+    return "ld"
+  ld = ld.strip()
+  ld_bfd = PathTo(ld + ".bfd")
+  if os.path.exists(ld_bfd) and os.access(ld_bfd, os.X_OK):

[Roland McGrath, 2011/09/07 19:47:49]

os.path.exists is redundant since PathTo already did it, and it's redundant with
os.access anyway.

[Brad Chen, 2011/09/07 20:18:53]

Done.

+    return ld_bfd
+  return ld
+
 def main():
-  LD_BFD = "/usr/bin/ld.bfd"
-  if not (os.path.exists(LD_BFD) and os.access(LD_BFD, os.X_OK)):
-    # Can't find the BFD loader, so invoke the unmodified argv
-    args = sys.argv
-    args[0] = "ld"
-    print "ld_bfd/ld: using ld"
-    sys.exit(subprocess.call(args))
-  # found the BFD loader, so use it
-  args = list()
-  args.append("/usr/bin/ld.bfd")
-  for arg in sys.argv[1:]:
-    if arg == "-Wl,--threads" or arg == "--threads":
-      continue
-    if arg == "-Wl,--thread-count=4" or arg == "--thread-count=4":
-      continue
-    if arg == "--icf=none":
-      continue
-    args.append(arg)
-  print("ld_bfd/ld: exec ", args)
+  args = [FindLDBFD()] + sys.argv[1:]
+  print("tools/ld_bfd/ld: exec ", args)

[Roland McGrath, 2011/09/07 19:47:49]

better: print("tools/ld_bfd/ld:exec " + ' '.join(args))

[Brad Chen, 2011/09/07 20:18:53]

Done.

   sys.exit(subprocess.call(args))
 
 if __name__ == "__main__":
   main()