Summary: | [CMake] Expose HAVE macros globally | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Don Olmstead <don.olmstead> | ||||||
Component: | Tools / Tests | Assignee: | Don Olmstead <don.olmstead> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | achristensen, annulen, bfulgham, commit-queue, lforschler, mcatanzaro, pvollan, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Nightly Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Don Olmstead
2018-02-21 11:58:38 PST
Created attachment 334398 [details]
Patch
Created attachment 334400 [details]
Patch
I am planning on using these changes in https://bugs.webkit.org/show_bug.cgi?id=182891 and figured I'd get this reviewed on its own while I wait for Per Arne to check that WIP patch to make sure it works for Apple's internal windows build. Comment on attachment 334400 [details] Patch Clearing flags on attachment: 334400 Committed r228900: <https://trac.webkit.org/changeset/228900> All reviewed patches have been landed. Closing bug. Comment on attachment 334400 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=334400&action=review > Source/cmake/OptionsCommon.cmake:119 > +WEBKIT_CHECK_HAVE_INCLUDE(HAVE_FEATURES_H features.h) I think WEBKIT_CHECK_INCLUDE would be a better name |