[CMake] Private/unsupported build options should be marked as advanced
authormcatanzaro@igalia.com <mcatanzaro@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 14 Oct 2016 18:29:16 +0000 (18:29 +0000)
committermcatanzaro@igalia.com <mcatanzaro@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 14 Oct 2016 18:29:16 +0000 (18:29 +0000)
https://bugs.webkit.org/show_bug.cgi?id=163451

Reviewed by Carlos Garcia Campos.

When checking to decide whether to mark an option as advanced, the conditional checks
whether _WEBKIT_AVAILABLE_OPTIONS_IS_PUBLIC_${_name} is defined. It is always defined. We
need to check its value instead.

* Source/cmake/WebKitFeatures.cmake:

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

ChangeLog
Source/cmake/WebKitFeatures.cmake

index 4915583..733a5fe 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,16 @@
+2016-10-14  Michael Catanzaro  <mcatanzaro@igalia.com>
+
+        [CMake] Private/unsupported build options should be marked as advanced
+        https://bugs.webkit.org/show_bug.cgi?id=163451
+
+        Reviewed by Carlos Garcia Campos.
+
+        When checking to decide whether to mark an option as advanced, the conditional checks
+        whether _WEBKIT_AVAILABLE_OPTIONS_IS_PUBLIC_${_name} is defined. It is always defined. We
+        need to check its value instead.
+
+        * Source/cmake/WebKitFeatures.cmake:
+
 2016-10-13  Brian Burg  <bburg@apple.com>
 
         [CMake] Split the definition of CODE_GENERATOR_PREPROCESSOR so it can be used in a custom command
index 7b7ef5e..3bb4c13 100644 (file)
@@ -268,7 +268,7 @@ macro(WEBKIT_OPTION_END)
         endif ()
 
         option(${_name} "${_WEBKIT_AVAILABLE_OPTIONS_DESCRIPTION_${_name}}" ${_WEBKIT_AVAILABLE_OPTIONS_INITIAL_VALUE_${_name}})
-        if (NOT _WEBKIT_AVAILABLE_OPTIONS_IS_PUBLIC_${_name})
+        if (NOT ${_WEBKIT_AVAILABLE_OPTIONS_IS_PUBLIC_${_name}})
             mark_as_advanced(FORCE ${_name})
         endif ()
     endforeach ()