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

Respect the XDG Base Directory Specification #227

Open
dsifford opened this issue Jul 24, 2018 · 3 comments
Open

Respect the XDG Base Directory Specification #227

dsifford opened this issue Jul 24, 2018 · 3 comments

Comments

@dsifford
Copy link

Hi there,

Long time torus user just getting merged over into manifold. Great service you all have provided.

One little gripe that I've had with torus and that I now have with manifold is that you guys and gals are not respecting the XDG Base Directory specification.

Specifically, ~/.manifoldrc and ~/.manifold/ are being created in the home directory irrespective of the presence of both $MANIFOLD_DIR and $XDG_CONFIG_HOME.

Because you guys are still relatively new, I think it would behoove you all to consider supporting the XDG specification from the get-go, being that it'll almost certainly be more difficult to add in later on once the service progresses further.

Information about the spec can be found here (official spec) and here (easier read).

Thanks in advance!

@enmand
Copy link
Member

enmand commented Jul 25, 2018

Hi!

Thanks for the suggestion. I've created a ticket internally to check this out.

As a bit of a workaround, you can use the HOME environment variable to specify a directory to use to search the for the .manifoldrc file.

@louisbranch
Copy link
Contributor

@dsifford can you take a look on the referenced PR?

This is a partial solution that doesn't address the ~/.manifold/bin part. AFAIK there is no xdg dir for binaries, the closest would be data, but it would make more sense to be ~/.local/bin instead of share.

@dsifford
Copy link
Author

Thanks for working on this @LuizBranco!

I'll add my thoughts to the PR thread.

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

No branches or pull requests

3 participants