Skip to content
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

Set the right Ozone platform on Linux #448

Merged

Conversation

tytan652
Copy link
Contributor

@tytan652 tytan652 commented Aug 1, 2024

Description

Fixes obsproject/obs-studio#11019

Forcing CEF to use the right Ozone platform allows to avoid having a dependency on XWayland (a X11 socket) under Wayland.

Motivation and Context

Chromium still does not default to the matching platform, but in our use of CEF we need CEF to use the same platform at Qt/OBS

How Has This Been Tested?

Ran the Flatpak without any X11 permission and added a browser source, no crash occurs.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code has been run through clang-format.
  • I have read the contributing document.
  • My code is not on the master branch.
  • The code has been tested.
  • All commit messages are properly formatted and commits squashed where appropriate.
  • I have included updates to all appropriate documentation.

@WizardCM WizardCM self-requested a review August 1, 2024 13:21
Forcing CEF to use the right Ozone platform allows to avoid having a
dependency on XWayland (a X11 socket) under Wayland.
@tytan652 tytan652 force-pushed the force_ozone_to_the_right_platform branch from a8eac87 to e7e1743 Compare August 1, 2024 14:42
@WizardCM WizardCM added the Bug Fix Non-breaking change which fixes an issue label Aug 4, 2024
@WizardCM WizardCM merged commit 8e2b31f into obsproject:master Aug 25, 2024
1 check passed
@tytan652 tytan652 deleted the force_ozone_to_the_right_platform branch August 25, 2024 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Fix Non-breaking change which fixes an issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

(Flatpak) OBS crash with browser source and no X11 DISPLAY (only Wayland)
2 participants