Summary: | REGRESSION(r263960): Re-land, because r263959 somehow fixed the build issue caused by r263953 | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Yusuke Suzuki <ysuzuki> | ||||
Component: | New Bugs | Assignee: | Yusuke Suzuki <ysuzuki> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, darin, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 213979, 233304 | ||||||
Attachments: |
|
Description
Yusuke Suzuki
2020-07-05 17:58:16 PDT
Created attachment 403568 [details] REVERT of r263960 Any committer can land this patch automatically by marking it commit-queue+. The commit-queue will build and test the patch before landing to ensure that the revert will be successful. This process takes approximately 15 minutes. If you would like to land the revert faster, you can use the following command: webkit-patch land-attachment ATTACHMENT_ID where ATTACHMENT_ID is the ID of this attachment. Committed r263961: <https://trac.webkit.org/changeset/263961> All reviewed patches have been landed. Closing bug and clearing flags on attachment 403568 [details]. OK, some notes for the record on what I figured out after the fact. The reason that r263953 broke Apple's internal builds is that there was a bug in the DerivedSources.make files where FEATURE_DEFINE_FLAGS would incorrect be defined as "-D" if FEATURE_DEFINES was empty. The reason that FEATURE_DEFINES was empty on the internal builds after r263953 is that the only feature left there was ENABLE_EXPERIMENTAL_FEATURES. That was enabled for builds on individual engineer’s computers, and the open source builds, but not for Apple internal production builds. The reason that r263959 made the problem go away is that it deleted FEATURE_DEFINE_FLAGS entirely, taking the bug with it. |