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

Linux: move the user home Archipelago dir to $XDG_STATE_HOME #4347

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ruby0b
Copy link

@ruby0b ruby0b commented Dec 9, 2024

Closes #4337.

This affects builds with non-writable installation directories. Instead of saving data in ~/Archipelago we now use $XDG_STATE_HOME/Archipelago (defaulting to ~/.local/state/Archipelago).
If ~/Archipelago still exists we move it to the new location and link ~/Archipelago to it.

Motivation: This follows the XDG Base Directory Specification to at least some degree and doesn't clutter the user's home directory.

How was this tested?

I ran the AppImage once while still having a ~/Archipelago dir and it moved it correctly and put a working symlink at ~/Archipelago.

This affects builds with non-writable installation directories.
Instead of saving data in ~/Archipelago we now use $XDG_STATE_HOME/Archipelago
(defaulting to ~/.local/state/Archipelago).
If ~/Archipelago still exists we move it to the new location and link ~/Archipelago to it.

Motivation: This follows the [XDG Base Directory Specification](https://specifications.freedesktop.org/basedir-spec/latest/)
to at least some degree and doesn't clutter the user's home directory.
@github-actions github-actions bot added affects: core Issues/PRs that touch core and may need additional validation. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet. labels Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects: core Issues/PRs that touch core and may need additional validation. waiting-on: peer-review Issue/PR has not been reviewed by enough people yet.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Linux: Follow the XDG Base Directory Specification
1 participant