-
Notifications
You must be signed in to change notification settings - Fork 228
Remove WS_CARBON check #3117
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 WS_CARBON check #3117
Conversation
f1d3caa to
ec9fcec
Compare
Carbon backend is no longer used or supported in SWT, as all modern macOS environments use the Cocoa backend (WS_COCOA). This check is now obsolete and can be removed. (eclipse-platform#2964)
ec9fcec to
b060d75
Compare
|
Thanks @elsazac |
|
This is causing #3141 and should probably be reverted (one may add a line to log warning when using 'carbon' instead of 'cocoa') as it seems clear that the ecosystem, and even Platform, is not ready for dropping this legacy definition. |
eclipse.platform.ui/bundles/org.eclipse.ui/plugin.xml Lines 236 to 240 in 5922d0c
I was hit by this problem when working in PDE and confirmed it also breaks navigation across simple text editors, so Platform was very much broken. At the very least, Platform's plugin.xml files should be updated when this change is applied. |
|
@basilevs if you have the opportunity to submit PRs replacing those |
@mickaelistria I would like to work on this again, but I wanted to understand , when we replace carbon with cocoa and remove the code containing WS_CARBON, that change will now honour the bindings used in all the plugins across Eclipse. But, would this change affect any third-party plugins that might still rely on carbon? |
This PR removes the only remaining WS_CARBON check from the codebase.The Carbon backend is no longer used or supported in SWT, as all modern macOS environments use the Cocoa backend (WS_COCOA). This check is now obsolete and can be removed. (#2964)