[CMake] build-webkit should be able to enable address sanitizer

https://bugs.webkit.org/show_bug.cgi?id=187222

Reviewed by Daniel Bates.

If 'set-webkit-configuration --asan' has been run, then build-webkit should pass
-DENABLE_ADDRESS_SANITIZER=ON to CMake, so that it actually does something.

This also works for build-jsc.

* Scripts/webkitdirs.pm:
(generateBuildSystemFromCMakeProject):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233402 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/Tools/ChangeLog b/Tools/ChangeLog
index b07b6b4..c961a76 100644 (file)
--- a/Tools/ChangeLog
+++ b/Tools/ChangeLog
@@ -1,3 +1,18 @@
+2018-06-30  Michael Catanzaro  <mcatanzaro@igalia.com>
+
+        [CMake] build-webkit should be able to enable address sanitizer
+        https://bugs.webkit.org/show_bug.cgi?id=187222
+
+        Reviewed by Daniel Bates.
+
+        If 'set-webkit-configuration --asan' has been run, then build-webkit should pass
+        -DENABLE_ADDRESS_SANITIZER=ON to CMake, so that it actually does something.
+
+        This also works for build-jsc.
+
+        * Scripts/webkitdirs.pm:
+        (generateBuildSystemFromCMakeProject):
+

 2018-06-30  Thibault Saunier  <tsaunier@igalia.com>
 
         Make check_flatpak() not verbose when only trying to run command inside flatpak
 2018-06-30  Thibault Saunier  <tsaunier@igalia.com>
 
         Make check_flatpak() not verbose when only trying to run command inside flatpak

diff --git a/Tools/Scripts/webkitdirs.pm b/Tools/Scripts/webkitdirs.pm
index 8dadf43..88bf6d0 100755 (executable)
--- a/Tools/Scripts/webkitdirs.pm
+++ b/Tools/Scripts/webkitdirs.pm
@@ -2252,6 +2252,8 @@ sub generateBuildSystemFromCMakeProject
         push @args, "-DCMAKE_BUILD_TYPE=Debug";
     }
 
         push @args, "-DCMAKE_BUILD_TYPE=Debug";
     }
 

+    push @args, "-DENABLE_ADDRESS_SANITIZER=ON" if asanIsEnabled();
+

     if ($willUseNinja) {
         push @args, "-G";
         if (canUseEclipseNinjaGenerator()) {
     if ($willUseNinja) {
         push @args, "-G";
         if (canUseEclipseNinjaGenerator()) {