WarmUpTill / SceneSwitcher

An automation tool for OBS Studio
https://obsproject.com/forum/resources/automatic-scene-switching.395/
GNU General Public License v2.0
948 stars 73 forks source link

Crash in plugin built from source #1160

Closed flakywanderer closed 2 months ago

flakywanderer commented 3 months ago

Describe the bug When the plugin is built from source, opening the plugin's settings page crashes OBS Studio

To Reproduce Steps to reproduce the behavior:

  1. Build and install obs-advanced-scene-switcher from source (in my case, via https://aur.archlinux.org/packages/obs-advanced-scene-switcher)
  2. Open the settings for advanced scene switcher in the "tools" menu

Expected behavior The settings window for the plugin opens up, and OBS Studio continues running

Logs https://gist.github.com/flakywanderer/917ea25f7656e99b7397e560d5a2553b#file-obs-log I cannot access the settings to enable debug options, as trying to open the settings causes the crash.

Version information

Additional context The plugin itself loads without error, and the associated dock widget loads and says that advanced scene switcher is active, but none of the macros I have set actually run.

WarmUpTill commented 2 months ago

You might be able to get more logs by manually modifying the scene collection json file.

The ‘advanced-scene-switcher’ section should contain an integer value with the name ‘logLevel’. If you set it to ‘2’ the logs will be more verbose.

But please make sie to back up the JSON file beforehand just in case something goes wrong.

Are you able to provide a back trace fromme the generated core file? This should narrow down the problem area quite a bit.

Unfortunately I won’t be able to look into the issue for now since I only have rather limited access to my development environment. So I apologize in advance if it will take a bit to resolve this issue.

WarmUpTill commented 2 months ago

I believe this crash was fixed with 223ce0d0a0a076d0e0b051caf627fc354f75581d. Can you confirm this on your end?

flakywanderer commented 2 months ago

Yes, it indeed does fix the crash.

WarmUpTill commented 2 months ago

Thank you for the confirmation! I will try to release a new version containing this fix soon-ish.

If you should require an official release containing this fix urgently, let me know, and I will push a release earlier.

WarmUpTill commented 2 months ago

An official release containing this fix was now published, so I am closing this issue :)