Create install targets for scan-build and scan-view

[jroelofs]

Update www too.

[jroelofs]

www/analyzer/installation.html

103 ↗

(On Diff #39439)

@zaks.anna Do you know if there was a particular reason for this? It's kind of a shame to have to use them from the src dir.

[zaks.anna]

www/analyzer/installation.html

103 ↗

(On Diff #39439)

I do not know. Ted might know.

[jroelofs]

Forgot to svn add after rebasing the patch on top of upstream.

[beanz]

I would prefer if installing these were optional and could be toggled on/off.

Other than that this all looks fine to me.

[bcraig]

For what it's worth, our internal branch installs these by default, and I was unpleasantly surprised when I found that upstream didn't.

[jroelofs]

In D14403#284054, @beanz wrote:

I would prefer if installing these were optional and could be toggled on/off.

I'll add flags for that.

[jroelofs]

Add flags to turn the new targets off.

[jroelofs]

r252474

tools/CMakeLists.txt

 add_clang_subdirectory(clang-format)
 add_clang_subdirectory(clang-format-vs)
 add_clang_subdirectory(clang-fuzzer)
+add_clang_subdirectory(scan-build)
+add_clang_subdirectory(scan-view)
 
 add_clang_subdirectory(c-index-test)
 add_clang_subdirectory(libclang)

tools/Makefile

 PARALLEL_DIRS := clang-format driver diagtool
 
 ifeq ($(ENABLE_CLANG_STATIC_ANALYZER), 1)
-  PARALLEL_DIRS += clang-check
+  PARALLEL_DIRS += clang-check scan-build scan-view
 endif
 
 ifeq ($(ENABLE_CLANG_ARCMT), 1)

tools/scan-build/scan-build

 
 # Determine the location of ccc-analyzer.
 my $AbsRealBin = Cwd::realpath($RealBin);
-my $Cmd = "$AbsRealBin/libexec/ccc-analyzer";
-my $CmdCXX = "$AbsRealBin/libexec/c++-analyzer";
+my $Cmd = "$AbsRealBin/../libexec/ccc-analyzer";
+my $CmdCXX = "$AbsRealBin/../libexec/c++-analyzer";
 
 # Portability: use less strict but portable check -e (file exists) instead of
 # non-portable -x (file is executable). On some windows ports -x just checks

tools/scan-view/Reporter.py

     @staticmethod
     def isAvailable():
         # FIXME: Find this .scpt better
-        path = os.path.join(os.path.dirname(__file__),'Resources/GetRadarVersion.scpt')
+        path = os.path.join(os.path.dirname(__file__),'../share/scan-view/GetRadarVersion.scpt')
         try:
           p = subprocess.Popen(['osascript',path], 
           stdout=subprocess.PIPE, stderr=subprocess.PIPE)

         if not componentVersion.strip():
             componentVersion = 'X'
 
-        script = os.path.join(os.path.dirname(__file__),'Resources/FileRadar.scpt')
+        script = os.path.join(os.path.dirname(__file__),'../share/scan-view/FileRadar.scpt')
         args = ['osascript', script, component, componentVersion, classification, personID, report.title,
                 report.description, diagnosis, config] + map(os.path.abspath, report.files)
 #        print >>sys.stderr, args

tools/scan-view/ScanView.py

 ###
 # Other simple parameters
 
-kResources = posixpath.join(posixpath.dirname(__file__), 'Resources')
+kResources = posixpath.join(posixpath.dirname(__file__), '../share/scan-view')
 kConfigPath = os.path.expanduser('~/.scanview.cfg')
 
 ###