Skip to content

Remove Qt::Widgets dependency #42

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

Merged
merged 1 commit into from
Jun 4, 2025

Conversation

sandboxcoder
Copy link

Motivation for change

Removing Qt6 dependency that I inadvertantly merged from obs-30.2. Adding comment we do not need to enable Qt::Widgets.

Now when I run otool -L on the obs-browser.plugin for mac it no longer depends on Qt framework:

~/projects/streamlabs/obs-studio/build_macos/UI/RelWithDebInfo/OBS.app/Contents/PlugIns/obs-browser.plugin/Contents/MacOS/obs-browser:
	/System/Library/Frameworks/AppKit.framework/Versions/C/AppKit (compatibility version 45.0.0, current version 2575.50.26)
	/System/Library/Frameworks/CoreFoundation.framework/Versions/A/CoreFoundation (compatibility version 150.0.0, current version 3423.0.0)
	@rpath/libobs.framework/Versions/A/libobs (compatibility version 1.0.0, current version 30.0.0)
	@rpath/obs-frontend-api.dylib (compatibility version 1.0.0, current version 30.0.0)
	/System/Library/Frameworks/Foundation.framework/Versions/C/Foundation (compatibility version 300.0.0, current version 3423.0.0)
	/usr/lib/libobjc.A.dylib (compatibility version 1.0.0, current version 228.0.0)
	/usr/lib/libc++.1.dylib (compatibility version 1.0.0, current version 1900.178.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1351.0.0)

* OSN isn't a "QApplication" currently. It's just a Mac NSApplication atm
so removing this dependency.
@sandboxcoder sandboxcoder merged commit 9bd1946 into streamlabs Jun 4, 2025
2 checks passed
@sandboxcoder sandboxcoder deleted the rno/remove-qt-widgets-dependency branch June 4, 2025 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants