Title: [241103] trunk
Revision
241103
Author
[email protected]
Date
2019-02-06 14:54:54 -0800 (Wed, 06 Feb 2019)

Log Message

Really enable the automatic checking and regenerations of .xcfilelists during builds
https://bugs.webkit.org/show_bug.cgi?id=194357
<rdar://problem/47861231>

Reviewed by Chris Dumez.

Bug 194124 was supposed to enable the automatic checking and
regenerating of .xcfilelist files during the build. While related
changes were included in that patch, the change to actually enable the
operation somehow was omitted. This patch actually enables the
operation. The check-xcfilelist.sh scripts now check
WK_DISABLE_CHECK_XCFILELISTS, and if it's "1", opts-out the developer
from the checking.

Source/_javascript_Core:

* Scripts/check-xcfilelists.sh:

Source/WebCore:

No new tests since there should be no observable behavior difference.

* Scripts/check-xcfilelists.sh:

Source/WebKit:

* Scripts/check-xcfilelists.sh:

Tools:

* DumpRenderTree/Scripts/check-xcfilelists.sh:
* WebKitTestRunner/Scripts/check-xcfilelists.sh:

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (241102 => 241103)


--- trunk/Source/_javascript_Core/ChangeLog	2019-02-06 22:18:36 UTC (rev 241102)
+++ trunk/Source/_javascript_Core/ChangeLog	2019-02-06 22:54:54 UTC (rev 241103)
@@ -1,3 +1,21 @@
+2019-02-06  Keith Rollin  <[email protected]>
+
+        Really enable the automatic checking and regenerations of .xcfilelists during builds
+        https://bugs.webkit.org/show_bug.cgi?id=194357
+        <rdar://problem/47861231>
+
+        Reviewed by Chris Dumez.
+
+        Bug 194124 was supposed to enable the automatic checking and
+        regenerating of .xcfilelist files during the build. While related
+        changes were included in that patch, the change to actually enable the
+        operation somehow was omitted. This patch actually enables the
+        operation. The check-xcfilelist.sh scripts now check
+        WK_DISABLE_CHECK_XCFILELISTS, and if it's "1", opts-out the developer
+        from the checking.
+
+        * Scripts/check-xcfilelists.sh:
+
 2019-02-06  Yusuke Suzuki  <[email protected]>
 
         [JSC] Unify indirectEvalExecutableSpace and directEvalExecutableSpace

Modified: trunk/Source/_javascript_Core/Scripts/check-xcfilelists.sh (241102 => 241103)


--- trunk/Source/_javascript_Core/Scripts/check-xcfilelists.sh	2019-02-06 22:18:36 UTC (rev 241102)
+++ trunk/Source/_javascript_Core/Scripts/check-xcfilelists.sh	2019-02-06 22:54:54 UTC (rev 241103)
@@ -1,6 +1,6 @@
 #!/bin/sh
 
-[ "${WK_ENABLE_CHECK_XCFILELISTS}" == "" ] && { echo "### Not running because WK_ENABLE_CHECK_XCFILELISTS is not set"; exit 0; }
+[ "${WK_DISABLE_CHECK_XCFILELISTS}" == "1" ] && { echo "### Not running because WK_DISABLE_CHECK_XCFILELISTS is set"; exit 0; }
 [ "${DEPLOYMENT_LOCATION}" == "YES" ] && { echo "### Not running because DEPLOYMENT_LOCATION is YES"; exit 0; }
 [ "${USE_INTERNAL_SDK}" == "YES" ] || { echo "### Not running because USE_INTERNAL_SDK is not YES"; exit 0; }
 

Modified: trunk/Source/WebCore/ChangeLog (241102 => 241103)


--- trunk/Source/WebCore/ChangeLog	2019-02-06 22:18:36 UTC (rev 241102)
+++ trunk/Source/WebCore/ChangeLog	2019-02-06 22:54:54 UTC (rev 241103)
@@ -1,3 +1,23 @@
+2019-02-06  Keith Rollin  <[email protected]>
+
+        Really enable the automatic checking and regenerations of .xcfilelists during builds
+        https://bugs.webkit.org/show_bug.cgi?id=194357
+        <rdar://problem/47861231>
+
+        Reviewed by Chris Dumez.
+
+        Bug 194124 was supposed to enable the automatic checking and
+        regenerating of .xcfilelist files during the build. While related
+        changes were included in that patch, the change to actually enable the
+        operation somehow was omitted. This patch actually enables the
+        operation. The check-xcfilelist.sh scripts now check
+        WK_DISABLE_CHECK_XCFILELISTS, and if it's "1", opts-out the developer
+        from the checking.
+
+        No new tests since there should be no observable behavior difference.
+
+        * Scripts/check-xcfilelists.sh:
+
 2019-02-06  John Wilander  <[email protected]>
 
         Forward Ad Click Attribution data from HTMLAnchorElement::handleClick() to WebKit::NavigationActionData

Modified: trunk/Source/WebCore/Scripts/check-xcfilelists.sh (241102 => 241103)


--- trunk/Source/WebCore/Scripts/check-xcfilelists.sh	2019-02-06 22:18:36 UTC (rev 241102)
+++ trunk/Source/WebCore/Scripts/check-xcfilelists.sh	2019-02-06 22:54:54 UTC (rev 241103)
@@ -1,6 +1,6 @@
 #!/bin/sh
 
-[ "${WK_ENABLE_CHECK_XCFILELISTS}" == "" ] && { echo "### Not running because WK_ENABLE_CHECK_XCFILELISTS is not set"; exit 0; }
+[ "${WK_DISABLE_CHECK_XCFILELISTS}" == "1" ] && { echo "### Not running because WK_DISABLE_CHECK_XCFILELISTS is set"; exit 0; }
 [ "${DEPLOYMENT_LOCATION}" == "YES" ] && { echo "### Not running because DEPLOYMENT_LOCATION is YES"; exit 0; }
 [ "${USE_INTERNAL_SDK}" == "YES" ] || { echo "### Not running because USE_INTERNAL_SDK is not YES"; exit 0; }
 

Modified: trunk/Source/WebKit/ChangeLog (241102 => 241103)


--- trunk/Source/WebKit/ChangeLog	2019-02-06 22:18:36 UTC (rev 241102)
+++ trunk/Source/WebKit/ChangeLog	2019-02-06 22:54:54 UTC (rev 241103)
@@ -1,3 +1,21 @@
+2019-02-06  Keith Rollin  <[email protected]>
+
+        Really enable the automatic checking and regenerations of .xcfilelists during builds
+        https://bugs.webkit.org/show_bug.cgi?id=194357
+        <rdar://problem/47861231>
+
+        Reviewed by Chris Dumez.
+
+        Bug 194124 was supposed to enable the automatic checking and
+        regenerating of .xcfilelist files during the build. While related
+        changes were included in that patch, the change to actually enable the
+        operation somehow was omitted. This patch actually enables the
+        operation. The check-xcfilelist.sh scripts now check
+        WK_DISABLE_CHECK_XCFILELISTS, and if it's "1", opts-out the developer
+        from the checking.
+
+        * Scripts/check-xcfilelists.sh:
+
 2019-02-06  John Wilander  <[email protected]>
 
         Forward Ad Click Attribution data from HTMLAnchorElement::handleClick() to WebKit::NavigationActionData

Modified: trunk/Source/WebKit/Scripts/check-xcfilelists.sh (241102 => 241103)


--- trunk/Source/WebKit/Scripts/check-xcfilelists.sh	2019-02-06 22:18:36 UTC (rev 241102)
+++ trunk/Source/WebKit/Scripts/check-xcfilelists.sh	2019-02-06 22:54:54 UTC (rev 241103)
@@ -1,6 +1,6 @@
 #!/bin/sh
 
-[ "${WK_ENABLE_CHECK_XCFILELISTS}" == "" ] && { echo "### Not running because WK_ENABLE_CHECK_XCFILELISTS is not set"; exit 0; }
+[ "${WK_DISABLE_CHECK_XCFILELISTS}" == "1" ] && { echo "### Not running because WK_DISABLE_CHECK_XCFILELISTS is set"; exit 0; }
 [ "${DEPLOYMENT_LOCATION}" == "YES" ] && { echo "### Not running because DEPLOYMENT_LOCATION is YES"; exit 0; }
 [ "${USE_INTERNAL_SDK}" == "YES" ] || { echo "### Not running because USE_INTERNAL_SDK is not YES"; exit 0; }
 

Modified: trunk/Tools/ChangeLog (241102 => 241103)


--- trunk/Tools/ChangeLog	2019-02-06 22:18:36 UTC (rev 241102)
+++ trunk/Tools/ChangeLog	2019-02-06 22:54:54 UTC (rev 241103)
@@ -1,3 +1,22 @@
+2019-02-06  Keith Rollin  <[email protected]>
+
+        Really enable the automatic checking and regenerations of .xcfilelists during builds
+        https://bugs.webkit.org/show_bug.cgi?id=194357
+        <rdar://problem/47861231>
+
+        Reviewed by Chris Dumez.
+
+        Bug 194124 was supposed to enable the automatic checking and
+        regenerating of .xcfilelist files during the build. While related
+        changes were included in that patch, the change to actually enable the
+        operation somehow was omitted. This patch actually enables the
+        operation. The check-xcfilelist.sh scripts now check
+        WK_DISABLE_CHECK_XCFILELISTS, and if it's "1", opts-out the developer
+        from the checking.
+
+        * DumpRenderTree/Scripts/check-xcfilelists.sh:
+        * WebKitTestRunner/Scripts/check-xcfilelists.sh:
+
 2019-02-06  David Kilzer  <[email protected]>
 
         WTR::InjectedBundlePage::willSendRequestForFrame() leaks a WKDataRef

Modified: trunk/Tools/DumpRenderTree/Scripts/check-xcfilelists.sh (241102 => 241103)


--- trunk/Tools/DumpRenderTree/Scripts/check-xcfilelists.sh	2019-02-06 22:18:36 UTC (rev 241102)
+++ trunk/Tools/DumpRenderTree/Scripts/check-xcfilelists.sh	2019-02-06 22:54:54 UTC (rev 241103)
@@ -1,6 +1,6 @@
 #!/bin/sh
 
-[ "${WK_ENABLE_CHECK_XCFILELISTS}" == "" ] && { echo "### Not running because WK_ENABLE_CHECK_XCFILELISTS is not set"; exit 0; }
+[ "${WK_DISABLE_CHECK_XCFILELISTS}" == "1" ] && { echo "### Not running because WK_DISABLE_CHECK_XCFILELISTS is set"; exit 0; }
 [ "${DEPLOYMENT_LOCATION}" == "YES" ] && { echo "### Not running because DEPLOYMENT_LOCATION is YES"; exit 0; }
 [ "${USE_INTERNAL_SDK}" == "YES" ] || { echo "### Not running because USE_INTERNAL_SDK is not YES"; exit 0; }
 

Modified: trunk/Tools/WebKitTestRunner/Scripts/check-xcfilelists.sh (241102 => 241103)


--- trunk/Tools/WebKitTestRunner/Scripts/check-xcfilelists.sh	2019-02-06 22:18:36 UTC (rev 241102)
+++ trunk/Tools/WebKitTestRunner/Scripts/check-xcfilelists.sh	2019-02-06 22:54:54 UTC (rev 241103)
@@ -1,6 +1,6 @@
 #!/bin/sh
 
-[ "${WK_ENABLE_CHECK_XCFILELISTS}" == "" ] && { echo "### Not running because WK_ENABLE_CHECK_XCFILELISTS is not set"; exit 0; }
+[ "${WK_DISABLE_CHECK_XCFILELISTS}" == "1" ] && { echo "### Not running because WK_DISABLE_CHECK_XCFILELISTS is set"; exit 0; }
 [ "${DEPLOYMENT_LOCATION}" == "YES" ] && { echo "### Not running because DEPLOYMENT_LOCATION is YES"; exit 0; }
 [ "${USE_INTERNAL_SDK}" == "YES" ] || { echo "### Not running because USE_INTERNAL_SDK is not YES"; exit 0; }
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to