Ignore:
Timestamp:
Oct 11, 2018, 2:22:40 PM (7 years ago)
Author:
Keith Rollin
Message:

CURRENT_ARCH should not be used in Run Script phase.
https://bugs.webkit.org/show_bug.cgi?id=190407
<rdar://problem/45133556>

Reviewed by Alexey Proskuryakov.

CURRENT_ARCH is used in a number of Xcode Run Script phases. However,
CURRENT_ARCH is not well-defined during this phase (and may even have
the value "undefined") since this phase is run just once per build
rather than once per supported architecture. Migrate away from
CURRENT_ARCH in favor of ARCHS, either by iterating over ARCHS and
performing an operation for each value, or by picking the first entry
in ARCHS and using that as a representative value.

Source/JavaScriptCore:

LLIntDesiredOffsets.h into a directory with a name based on ARCHS
rather than CURRENT_ARCH.

Source/WebCore:

No new tests -- no functional changes.

  • DerivedSources.make: When forming TARGET_TRIPLE_FLAGS, grab the

first entry in ARCHS rather than use CURRENT_ARCH.

Source/WebKit:

  • DerivedSources.make: When forming TARGET_TRIPLE_FLAGS, grab the

first entry in ARCHS rather than use CURRENT_ARCH.

Source/WebKitLegacy:

  • WebKitLegacy.xcodeproj/project.pbxproj: When generating

WebKitLegacy.*.exp, generate both 32- and 64-bit versions for
macosx platforms.

Source/WebKitLegacy/mac:

  • Configurations/WebKitLegacy.xcconfig: When generating

WebKitLegacy.*.exp, generate both 32- and 64-bit versions for
macosx platforms.

  • MigrateHeaders.make:
  • migrate-headers.sh: When generating ReexportedWebCoreSymbols_*.exp,

generate one for each architecture in ARCHS.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/Source/JavaScriptCore/ChangeLog

    r237042 r237047  
     12018-10-11  Keith Rollin  <[email protected]>
     2
     3        CURRENT_ARCH should not be used in Run Script phase.
     4        https://bugs.webkit.org/show_bug.cgi?id=190407
     5        <rdar://problem/45133556>
     6
     7        Reviewed by Alexey Proskuryakov.
     8
     9        CURRENT_ARCH is used in a number of Xcode Run Script phases. However,
     10        CURRENT_ARCH is not well-defined during this phase (and may even have
     11        the value "undefined") since this phase is run just once per build
     12        rather than once per supported architecture. Migrate away from
     13        CURRENT_ARCH in favor of ARCHS, either by iterating over ARCHS and
     14        performing an operation for each value, or by picking the first entry
     15        in ARCHS and using that as a representative value.
     16
     17        * JavaScriptCore.xcodeproj/project.pbxproj: Store
     18        LLIntDesiredOffsets.h into a directory with a name based on ARCHS
     19        rather than CURRENT_ARCH.
     20
    1212018-10-10  Mark Lam  <[email protected]>
    222
Note: See TracChangeset for help on using the changeset viewer.