Skip to content

Fix OpenUtau's data folders appearing in the wrong place on macOS#1460

Merged
stakira merged 1 commit into
openutau:masterfrom
Nyaacinth:master
May 21, 2025
Merged

Fix OpenUtau's data folders appearing in the wrong place on macOS#1460
stakira merged 1 commit into
openutau:masterfrom
Nyaacinth:master

Conversation

@Nyaacinth

Copy link
Copy Markdown
Contributor

Ref: #1459

By using Environment.SpecialFolder.UserProfile instead of Environment.SpecialFolder.Personal to get user's home directory, this will fix the issue that OpenUtau's folder appearing in /Users/<username>/Documents/Library but not /Users/<username>/Library.

Since I don't have a matrix to fully test this on a variety of macOS devices, you might want to test it more comprehensively before merging or pushing it to release.

@RedBlackAka

Copy link
Copy Markdown
Contributor

Could you potentially add a temporary function in the main app startup to move the OpenUtau folder and its contents if detected, so that existing users can update hassle-free?

On a note: this is similar to my Windows AppData PR and a good cleanup for following conventions

@Seele-Vollerei32

Copy link
Copy Markdown
Contributor

Due to this change on dotnet, all the UNIX like os are affected, so please change the same field of linux

@stakira stakira merged commit 4aa9623 into openutau:master May 21, 2025
@Seele-Vollerei32

Copy link
Copy Markdown
Contributor

However, there is a same problem under Linux and haven't been fix. Maybe I need to make another PR

Seele-Vollerei32 added a commit to Seele-Vollerei32/OpenUtau that referenced this pull request May 25, 2025
This PR shouldn't exist, however, due to openutau#1460  , the problem under Linux haven't fixed.
stakira pushed a commit that referenced this pull request May 30, 2025
…1555)

This PR shouldn't exist, however, due to #1460  , the problem under Linux haven't fixed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants