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

UI: Fix Linux base folder location #11651

Merged
merged 1 commit into from
Jan 8, 2025

Conversation

Penwy
Copy link
Contributor

@Penwy Penwy commented Dec 20, 2024

Description

This changes the BASE_FOLDER to be .. instead of ../.. on linux.

Motivation and Context

On linux, the binaries are directly in the bin folder, so one level above the base folder. As such setting BASE_FOLDER to ../.. caused the config folder to be created one level below where it should be.

How Has This Been Tested?

Launched OBS, checked that the config folder was created correctly.

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 added Bug Fix Non-breaking change which fixes an issue Linux Affects Linux labels Dec 21, 2024
@RytoEX RytoEX added this to the OBS Studio 31 milestone Jan 7, 2025
@RytoEX RytoEX changed the title UI: Fix linux base folder location UI: Fix Linux base folder location Jan 7, 2025
@RytoEX RytoEX changed the base branch from master to release/31.0 January 7, 2025 21:45
@RytoEX RytoEX self-assigned this Jan 8, 2025
@RytoEX
Copy link
Member

RytoEX commented Jan 8, 2025

Re-targeting this to release/31.0. The change has been ported to master in #11622.

@RytoEX RytoEX merged commit 61a5a92 into obsproject:release/31.0 Jan 8, 2025
14 checks passed
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 Linux Affects Linux
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants