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

Implement XDG Base Directory Support for Linux #16

Closed
wants to merge 4 commits into from

Conversation

SuperNascher
Copy link

This Pull-Request adds the XDG Base Directory Specification for Linux.

This resolves #13

@itdelatrisu
Copy link
Owner

I'll get to this soon, sorry for the delay. I'm thinking that I'll have a switch in the config file to enable/disable this, and have this off by default (to be consistent with past behavior).

Is there any reason you used System.getProperty("file.separator") instead of just / (which works in all operating systems I've seen) or File.separator (the Java standard)?

@SuperNascher
Copy link
Author

There was no spezific reason for using System.getProperty("file.separator")
, but I prefer File.separator
over the /.

itdelatrisu added a commit that referenced this pull request Feb 13, 2015
Implemented XDG base directory support for Unix-like operating systems.  This is disabled by default.  To use these directories, uncomment the line in pom.xml that sets the "XDG" system property when packaging opsu!.

Signed-off-by: Jeffrey Han <itdelatrisu@gmail.com>
@itdelatrisu
Copy link
Owner

Added in ed2d305, with some modifications since I wanted this off by default. Due to how JarSplice works, you'll have to uncomment a line in pom.xml when packaging opsu! (in Maven) to use these directories. Thanks again!

@SuperNascher
Copy link
Author

Thank you, for adding this Pull-Request 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Save beatmaps/config etc. XDG-conform
3 participants