Skip to content

updater: Retain legacy module path support - #13939

Merged
RytoEX merged 1 commit into
obsproject:masterfrom
Warchamp7:updater-v33-fixes
Sep 30, 2026
Merged

RytoEX merged 1 commit into
obsproject:masterfrom
Warchamp7:updater-v33-fixes

Conversation

@Warchamp7

Copy link
Copy Markdown
Member

Description

When checking if the browser plugin is installed, check the legacy plugin location as well as the new core location from v33 onwards.

When installing the virtual cam, check the manifest for the version we are updating to and use the appropriate location based on that.

Motivation and Context

The updater itself is distributed and updated for all versions of OBS for which updates are still available. As such, it needs to be aware of the legacy plugin location for versions before v33 and the new location for v33 onwards.

When checking if the browser plugin exists, check both locations.

When installing the virtual cam, we reference the manifest for the version we are updating to and use the appropriate location.

How Has This Been Tested?

It has not.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • Tweak (non-breaking change to improve existing functionality)

Checklist:

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

Comment thread frontend/updater/updater.cpp Resolved
@RytoEX RytoEX self-assigned this Sep 28, 2026

@RytoEX RytoEX left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes here are correct, though it does not address the game capture hook file paths as previously mentioned off-thread.

Comment thread frontend/updater/updater.cpp Resolved
@RytoEX
RytoEX requested review from PatTheMav and RytoEX September 29, 2026 18:46
Comment thread frontend/updater/updater.cpp Resolved
The updater itself is distributed and updated for all versions of OBS
for which updates are still available. As such, it needs to be aware
of the legacy plugin location for versions before v33 and the new
location for v33 onwards.

When checking if the browser plugin exists, check both locations.

When installing the virtual cam, we reference the manifest for the
version we are updating to and use the appropriate location.
@Warchamp7
Warchamp7 force-pushed the updater-v33-fixes branch 2 times, most recently from 602b3c5 to 003fce6 Compare September 29, 2026 20:19

@PatTheMav PatTheMav left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes make sense to me.

@RytoEX RytoEX left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks fine.

@RytoEX RytoEX added kind/bug Categorizes issue or PR as related to a bug. kind/enhancement Enhancements are not bugs or new features but can improve usability or performance. platform/windows Categorizes issue or PR as affecting Windows specifically release-note/fix Groups pull request under the "Fixes" section in the associated patch notes labels Sep 30, 2026
@RytoEX
RytoEX merged commit 2c4e446 into obsproject:master Sep 30, 2026
15 of 28 checks passed
@Warchamp7
Warchamp7 deleted the updater-v33-fixes branch October 8, 2026 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bug Categorizes issue or PR as related to a bug. kind/enhancement Enhancements are not bugs or new features but can improve usability or performance. platform/windows Categorizes issue or PR as affecting Windows specifically release-note/fix Groups pull request under the "Fixes" section in the associated patch notes

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

3 participants