Attachment 361098 Details for Bug 194124 – Patch

[patch] Patch

bug-194124-20190204140445.patch (text/plain), 6.39 KB, created by Keith Rollin on 2019-02-04 14:04:45 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Keith Rollin

Created: 2019-02-04 14:04:45 PST

Size: 6.39 KB

patch

obsolete

>Subversion Revision: 240936
>diff --git a/Source/JavaScriptCore/ChangeLog b/Source/JavaScriptCore/ChangeLog
>index 135d7dc257ce4e60bd296587de9ff24434820ad8..64241c1ac118a1603107e21d2928b7d3dc4357ad 100644
>--- a/Source/JavaScriptCore/ChangeLog
>+++ b/Source/JavaScriptCore/ChangeLog
>@@ -1,3 +1,23 @@
>+2019-01-31  Keith Rollin  <krollin@apple.com>
>+
>+        Enable the automatic checking and regenerations of .xcfilelists during builds
>+        https://bugs.webkit.org/show_bug.cgi?id=194124
>+        <rdar://problem/47721277>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Bug 193790 add a facility for checking -- during build time -- that
>+        any needed .xcfilelist files are up-to-date and for updating them if
>+        they are not. This facility was initially opt-in by setting
>+        WK_ENABLE_CHECK_XCFILELISTS until other pieces were in place and until
>+        the process seemed robust. It's now time to enable this facility and
>+        make it opt-out. If there is a need to disable this facility, set and
>+        export WK_DISABLE_CHECK_XCFILELISTS=1 in your environment before
>+        running `make` or `build-webkit`, or before running Xcode from the
>+        command line, or by setting it in LocalOverrides.xcconfig.
>+
>+        * Scripts/check-xcfilelists.sh:
>+
> 2019-01-31  Keith Rollin  <krollin@apple.com>
> 
>         Update .xcfilelist files
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 794ebcbc0b198d4eb6ed349523f839fd154e444e..b9c32ff3d053bfd4a047716c0cdde6aaa1f955f5 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,25 @@
>+2019-01-31  Keith Rollin  <krollin@apple.com>
>+
>+        Enable the automatic checking and regenerations of .xcfilelists during builds
>+        https://bugs.webkit.org/show_bug.cgi?id=194124
>+        <rdar://problem/47721277>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Bug 193790 add a facility for checking -- during build time -- that
>+        any needed .xcfilelist files are up-to-date and for updating them if
>+        they are not. This facility was initially opt-in by setting
>+        WK_ENABLE_CHECK_XCFILELISTS until other pieces were in place and until
>+        the process seemed robust. It's now time to enable this facility and
>+        make it opt-out. If there is a need to disable this facility, set and
>+        export WK_DISABLE_CHECK_XCFILELISTS=1 in your environment before
>+        running `make` or `build-webkit`, or before running Xcode from the
>+        command line, or by setting it in LocalOverrides.xcconfig.
>+
>+        No new tests since there should be no observable behavior difference.
>+
>+        * Scripts/check-xcfilelists.sh:
>+
> 2019-01-31  Keith Rollin  <krollin@apple.com>
> 
>         Update .xcfilelist files
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index a3029eab137dd81703bca41ea7461c3277813703..234b885064951c681cd01eae4da7a93e22da1173 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,23 @@
>+2019-01-31  Keith Rollin  <krollin@apple.com>
>+
>+        Enable the automatic checking and regenerations of .xcfilelists during builds
>+        https://bugs.webkit.org/show_bug.cgi?id=194124
>+        <rdar://problem/47721277>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Bug 193790 add a facility for checking -- during build time -- that
>+        any needed .xcfilelist files are up-to-date and for updating them if
>+        they are not. This facility was initially opt-in by setting
>+        WK_ENABLE_CHECK_XCFILELISTS until other pieces were in place and until
>+        the process seemed robust. It's now time to enable this facility and
>+        make it opt-out. If there is a need to disable this facility, set and
>+        export WK_DISABLE_CHECK_XCFILELISTS=1 in your environment before
>+        running `make` or `build-webkit`, or before running Xcode from the
>+        command line, or by setting it in LocalOverrides.xcconfig.
>+
>+        * Scripts/check-xcfilelists.sh:
>+
> 2019-01-31  Keith Rollin  <krollin@apple.com>
> 
>         Update .xcfilelist files
>diff --git a/Source/JavaScriptCore/Scripts/check-xcfilelists.sh b/Source/JavaScriptCore/Scripts/check-xcfilelists.sh
>index 953dae273549424a79188b38b211b557d513201f..efb4ab1763d7c424ba27052c01b6eba8d8c1fb71 100755
>--- a/Source/JavaScriptCore/Scripts/check-xcfilelists.sh
>+++ b/Source/JavaScriptCore/Scripts/check-xcfilelists.sh
>@@ -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; }
> 
>diff --git a/Source/WebCore/Scripts/check-xcfilelists.sh b/Source/WebCore/Scripts/check-xcfilelists.sh
>index c669a7de60d175e1a1c1ac3a62ab849c98db962a..585b950cdcf80a1ef12ae51748e300d34ba4b429 100755
>--- a/Source/WebCore/Scripts/check-xcfilelists.sh
>+++ b/Source/WebCore/Scripts/check-xcfilelists.sh
>@@ -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; }
> 
>diff --git a/Source/WebKit/Scripts/check-xcfilelists.sh b/Source/WebKit/Scripts/check-xcfilelists.sh
>index 8e836f12599600eadee39dc414ea4cc4dfa85659..1150a4f60b2652d8bf28782d6b099209c2e10f08 100755
>--- a/Source/WebKit/Scripts/check-xcfilelists.sh
>+++ b/Source/WebKit/Scripts/check-xcfilelists.sh
>@@ -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; }
>

Actions: View | Formatted Diff | Diff

Attachments on bug 194124: 360807 | 361098 | 361216