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

Support XDG Base Directory specification on Linux #160

Open
wants to merge 3 commits into
base: feature
Choose a base branch
from

Conversation

witchymary
Copy link

@witchymary witchymary commented Sep 15, 2024

I simply fixed merge conflicts and compilation issues from wangqr#132. Closes #122. Here's the original description by @WhiredPlanck:

This PR will fix Issue Aegisub#226. Inspired by or adapted from aria2/aria2@8bc1d37 and Aegisub#148.

My basic idea just is:

* If `$HOME/.aegisub` exists, then the program will still read/write config/cache/data from there;

* If `$HOME/.aegisub` doesn't exist, then the program will read/write config/cache/data from corresponding XDG Base Directory.

I think this idea may need more optimization, but I just pull a request first so we could discuss the detail.

@witchymary
Copy link
Author

witchymary commented Sep 15, 2024

Note, this currently breaks DependencyControl due to https://github.com/TypesettingTools/DependencyControl/blob/master/modules/DependencyControl/Record.moon#L44. A fix would be to simply undo this small excerpt, but that kinda defies the point of the PR.

-	auto path_log = config::path->Decode("?user/log/");
+	auto path_log = config::path->Decode("?state/log/");

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.

Add XDG Base Directory specification support
3 participants