-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove Apple Win Support from WTF #9933
Remove Apple Win Support from WTF #9933
Conversation
EWS run on previous version of this PR (hash c61dec6) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We will also be able to remove the OS(WINDOWS)
/PLATFORM(WIN)
bits from cf/FileSystemCF.cpp
and cf/URLCF.cpp
, but I believe WinCairo is still building some of this in support of the WebKitLegacy code (which will go away soon). So I think we should hold off on removing those until Don tell us they are ready.
c61dec6
to
6d3f382
Compare
EWS run on current version of this PR (hash 6d3f382) |
Just waiting for the Win Cairo CI to wrap up before landing. |
https://bugs.webkit.org/show_bug.cgi?id=252061 Reviewed by Don Olmstead and Brent Fulgham. Remove Apple Win Support from WTF as it is no longer in use. This should not touch the WIN CAIRO build at all. * Source/WTF/WTF.xcodeproj/project.pbxproj: * Source/WTF/wtf/CMakeLists.txt: * Source/WTF/wtf/PlatformEnable.h: * Source/WTF/wtf/PlatformEnableWinApple.h: Removed. Canonical link: https://commits.webkit.org/260137@main
6d3f382
to
4ad56ab
Compare
Committed 260137@main (4ad56ab): https://commits.webkit.org/260137@main Reviewed commits have been landed. Closing PR #9933 and removing active labels. |
4ad56ab
6d3f382
π mac-AS-debugπ§ͺ api-gtkπ π§ͺ jscπ jsc-armv7π§ͺ mac-AS-debug-wk2π§ͺ jsc-armv7-tests