[cfe-commits] r171607 - /cfe/trunk/tools/scan-build/set-xcode-analyzer

Ted Kremenek kremenek at apple.com
Fri Jan 4 23:31:54 PST 2013


Author: kremenek
Date: Sat Jan  5 01:31:54 2013
New Revision: 171607

URL: http://llvm.org/viewvc/llvm-project?rev=171607&view=rev
Log:
Fix set-xcode-analyzer to only modify the ExecPath for the analyzer when using --use-xcode-clang.

Turns out that the ExecPath for the ObjC migrator would also get set.

Fixes <rdar://problem/12961769>.

Modified:
    cfe/trunk/tools/scan-build/set-xcode-analyzer

Modified: cfe/trunk/tools/scan-build/set-xcode-analyzer
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/tools/scan-build/set-xcode-analyzer?rev=171607&r1=171606&r2=171607&view=diff
==============================================================================
--- cfe/trunk/tools/scan-build/set-xcode-analyzer (original)
+++ cfe/trunk/tools/scan-build/set-xcode-analyzer Sat Jan  5 01:31:54 2013
@@ -45,6 +45,8 @@
         m = re.search('^(\s*ExecPath\s*=\s*")', line)
         if m:
           line = "".join([m.group(0), pathToChecker, '";\n'])
+          # Do not modify further ExecPath's later in the xcspec.
+          foundAnalyzer = False
       t.write(line)
   t.close()
   print "(+) processing:", path





More information about the cfe-commits mailing list