Skip to content

frontend: Fix plugin manager Flatpak data path - #14103

Merged
RytoEX merged 1 commit into
obsproject:masterfrom
FiniteSingularity:bugfix/flatpak-data-path
Oct 27, 2026
Merged

RytoEX merged 1 commit into
obsproject:masterfrom
FiniteSingularity:bugfix/flatpak-data-path

Conversation

@FiniteSingularity

Copy link
Copy Markdown
Contributor

Update the Plugin Manager Flatpak loader data path to match the Flatpak third party plugin merge-dir.

Description

Currently, the plugin manager attempts to load plugin data for Flatpak distributed plugins from an incorrect location. The OBS Flatpak manifest states that plugins should store their data at:

/app/plugins/share/obs/obs-modules/plugins/<PLUGIN_NAME>

but the plugin manager was attempting to load data from:

/app/plugins/share/obs-modules/plugins/<PLUGIN_NAME>

This PR updates the loading path to include the missing obs/ subdirectory.

Motivation and Context

Without this change, it is not possible for Flatpak plugins to be loaded using the new plugin format.

How Has This Been Tested?

I did a quick test on a local Flatpak build to make sure this works properly, and it does. I was not able to test the actual live Flatpak OBS currently on Flathub.

Types of changes

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

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.

Update the Plugin Manager Flatpak loader data path to match the Flatpak
third party plugin merge-dir.
@RytoEX RytoEX added kind/bug Categorizes issue or PR as related to a bug. release-note/fix Groups pull request under the "Fixes" section in the associated patch notes labels Oct 27, 2026
@RytoEX RytoEX added this to the OBS Studio 33.0 milestone Oct 27, 2026
@tytan652 tytan652 added Seeking Testers Build artifacts on CI platform/linux Categorizes issue or PR as affecting Linux specifically labels Oct 27, 2026

@tytan652 tytan652 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM and tested locally.

@RytoEX
RytoEX merged commit 66b1abd into obsproject:master Oct 27, 2026
30 checks passed
@github-project-automation github-project-automation Bot moved this from Ready For Review to Merged in Current Release Tracker (33.0) Oct 27, 2026
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. platform/linux Categorizes issue or PR as affecting Linux specifically release-note/fix Groups pull request under the "Fixes" section in the associated patch notes Seeking Testers Build artifacts on CI

Projects

Status: Merged

Development

Successfully merging this pull request may close these issues.

4 participants