Bug 252064
Summary: | Fix mistakes in WebKitTargetConditionals.xcconfig | ||
---|---|---|---|
Product: | WebKit | Reporter: | David Kilzer (:ddkilzer) <ddkilzer> |
Component: | WebKit Misc. | Assignee: | David Kilzer (:ddkilzer) <ddkilzer> |
Status: | RESOLVED FIXED | ||
Severity: | Normal | CC: | webkit-bug-importer |
Priority: | P2 | Keywords: | InRadar |
Version: | WebKit Nightly Build | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
See Also: |
https://bugs.webkit.org/show_bug.cgi?id=245692 https://bugs.webkit.org/show_bug.cgi?id=252076 |
||
Bug Depends on: | |||
Bug Blocks: | 252066, 252970, 269799 |
David Kilzer (:ddkilzer)
Fix mistakes in WebKitTargetConditionals.xcconfig.
The following macros are defined incorrectly (they result in circular definitions):
```
WK_IOS_17 = $(WK_IOS_17_$(WK_IOS_BEFORE_17));
WK_IOS_17_ = $(WK_IOS_17); // BUG: Circular definition.
[...]
WK_TVOS_16 = $(WK_TVOS_16_$(WK_TVOS_BEFORE_16));
WK_TVOS_16_ = $(WK_TVOS_16); // BUG: Circular definition.
[...]
WK_TVOS_17 = $(WK_TVOS_17_$(WK_TVOS_BEFORE_17));
WK_TVOS_17_ = $(WK_TVOS_17); // BUG: Circular definition.
```
The net effect of this error was that Bug 245692 had no effect on iOS builds because `WK_IOS_17` would never be defined as `_IOS_SINCE_17`:
Enable libc++ assertions
https://bugs.webkit.org/show_bug.cgi?id=245692
rdar://100416021
https://commits.webkit.org/259122@main
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/105284669>
David Kilzer (:ddkilzer)
Pull request: https://github.com/WebKit/WebKit/pull/9936
EWS
Committed 260134@main (adf544afeb9f): <https://commits.webkit.org/260134@main>
Reviewed commits have been landed. Closing PR #9936 and removing active labels.