[llvm] r288385 - Recommit r287403 (reverted in r287804): [lit] When setting SDKROOT on Darwin, use '--sdk macosx' to find the right SDK path.

Kuba Mracek via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 1 09:45:22 PST 2016


Author: kuba.brecka
Date: Thu Dec  1 11:45:22 2016
New Revision: 288385

URL: http://llvm.org/viewvc/llvm-project?rev=288385&view=rev
Log:
Recommit r287403 (reverted in r287804): [lit] When setting SDKROOT on Darwin, use '--sdk macosx' to find the right SDK path.

This shouls now be safe and not break any more bots.  It's strictly better to use '--sdk macosx', otherwise xcrun can return weird things for example when you have Command Line Tools or the SDK installed into '/'.


Modified:
    llvm/trunk/utils/lit/lit/util.py

Modified: llvm/trunk/utils/lit/lit/util.py
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/utils/lit/lit/util.py?rev=288385&r1=288384&r2=288385&view=diff
==============================================================================
--- llvm/trunk/utils/lit/lit/util.py (original)
+++ llvm/trunk/utils/lit/lit/util.py Thu Dec  1 11:45:22 2016
@@ -251,7 +251,7 @@ def usePlatformSdkOnDarwin(config, lit_c
     # default system root path.
     if 'darwin' in config.target_triple:
         try:
-            cmd = subprocess.Popen(['xcrun', '--show-sdk-path'],
+            cmd = subprocess.Popen(['xcrun', '--show-sdk-path', '--sdk', 'macosx'],
                                    stdout=subprocess.PIPE, stderr=subprocess.PIPE)
             out, err = cmd.communicate()
             out = out.strip()




More information about the llvm-commits mailing list