[GTK] Build is broken without option --enable-unstable-features
authorzandobersek@gmail.com <zandobersek@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 18 Sep 2012 17:43:06 +0000 (17:43 +0000)
committerzandobersek@gmail.com <zandobersek@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 18 Sep 2012 17:43:06 +0000 (17:43 +0000)
https://bugs.webkit.org/show_bug.cgi?id=96996

Reviewed by Martin Robinson.

When searching for the bare feature define in feature_defines_unstable or
feature_defines_overrides, search for the bare define followed by = character.
This avoids incorrectly matching the ENABLE_VIDEO define to the ENABLE_VIDEO_TRACK
overriding define (and works as well for other similarly named feature defines).

No new tests - no new functionality.

* GNUmakefile.am:

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

Source/WebCore/ChangeLog
Source/WebCore/GNUmakefile.am

index 594fc2d..8b4202a 100644 (file)
@@ -1,3 +1,19 @@
+2012-09-18  Zan Dobersek  <zandobersek@gmail.com>
+
+        [GTK] Build is broken without option --enable-unstable-features
+        https://bugs.webkit.org/show_bug.cgi?id=96996
+
+        Reviewed by Martin Robinson.
+
+        When searching for the bare feature define in feature_defines_unstable or
+        feature_defines_overrides, search for the bare define followed by = character.
+        This avoids incorrectly matching the ENABLE_VIDEO define to the ENABLE_VIDEO_TRACK
+        overriding define (and works as well for other similarly named feature defines).
+
+        No new tests - no new functionality.
+
+        * GNUmakefile.am:
+
 2012-09-18  Andrey Kosyakov  <caseq@chromium.org>
 
         Web Inspector: [Extensions API] postpone requests to add extensions until extension server is initialized
index 61f2d9e..e1c6462 100644 (file)
@@ -365,9 +365,9 @@ feature_defines += $(strip $(foreach bare_define, \
                        $(if $(findstring =$(fvalue),$(define)),$(patsubst %=$(fvalue),%,$(define))) \
                ) \
        ), \
-       $(if $(findstring $(bare_define), $(feature_defines_unstable)), \
+       $(if $(findstring $(bare_define)=, $(feature_defines_unstable)), \
                $(filter $(bare_define)=%,$(feature_defines_unstable)), \
-               $(if $(findstring $(bare_define), $(feature_defines_overrides)), \
+               $(if $(findstring $(bare_define)=, $(feature_defines_overrides)), \
                        $(filter $(bare_define)=%,$(feature_defines_overrides)), \
                        $(filter $(bare_define)=%,$(feature_defines_defaults)) \
                ) \