-
Notifications
You must be signed in to change notification settings - Fork 23
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
Add XDG_STATE_HOME #43
base: master
Are you sure you want to change the base?
Conversation
Thanks for the PR. I'm sorry for the late in reviewing. I've been away from computer for a time.
You told that the new spec is not released. So do you think in the meantime |
I think that's up to you to decide. See also the discussion in whitequark/rust-xdg#33 about this topic. I think I'll go and ask for a proper release, but don't expect anything to happen soon. |
The spec won't move in any direction anytime soon, so don't expect any releases there. Are you still interested in this? |
Maybe I'm missing something, but every variant of the spec I can find suggests this was released a little over a year ago now. Notably both the Freedesktop.org Specifications page for basedir version 0.8 and the xdg gitlab variant list $XDG_STATE_HOME. |
True. How could I miss that? Thanks for the notice |
Actually I was digging around a bit more, it looks like this was added and released in dirs 4.0.0. This is the repo where development is happening for dirs now which threw me off since I got to here from another issue blocked by this and didn't look at the readme |
Their site wasn't updated until well after the fact so the only way to know at the time was if you were following the upstream repo itself. |
ping... do this has any process? |
Not released yet, but it's already part of the spec:
$XDG_STATE_HOME
.I added support for it in this crate by copy-pasting and following existing patterns as much as possible. As it's not officially released yet, maybe it should be hidden behind a feature flag?
I don't know anything about Windows and Mac, so I simply answered that question with
None
. I think implementing it for these platforms can be done in a future work. (I wouldn't want to do it since I can't test it.)