Skip to content

fixes #8463 do not assume C for root - #8464

Merged
nbolton merged 1 commit into
masterfrom
fix_8463
Apr 10, 2025
Merged

fixes #8463 do not assume C for root#8464
nbolton merged 1 commit into
masterfrom
fix_8463

Conversation

@sithlord48

@sithlord48 sithlord48 commented Apr 8, 2025

Copy link
Copy Markdown
Member

When we moved the settings we assumed C we need to use QDir::rootPath in the case of windows not being installed on the "C" drive.

@sithlord48
sithlord48 requested a review from nbolton April 8, 2025 11:44
@sithlord48 sithlord48 added 🪟 windows This issue affects Windows 🩹 fix This PR fixes a reported bug labels Apr 8, 2025
Comment thread src/lib/common/Settings.h
@nbolton
nbolton self-requested a review April 9, 2025 12:54
@nbolton
nbolton merged commit 84ae597 into master Apr 10, 2025
@nbolton
nbolton deleted the fix_8463 branch April 10, 2025 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🩹 fix This PR fixes a reported bug 🪟 windows This issue affects Windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants