-
-
Notifications
You must be signed in to change notification settings - Fork 104
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
XDG Compliance #87
Comments
Thanks for opening a new issue. The team has been notified and will review it as soon as possible. |
Dear Sir, XDG compliance is surely a priority, hence with the next release the default directory will change its position. |
Dear Sir, XDG compliance has been implemented. Of course it will break some sort of compatibility with previous versions. A fix can be users moving $HOME/.kb into $HOME/.local/share/kb or into the XDG_DATA_HOME directory. At the moment this is of course only implemented within the git version but as soon as 0.1.7 will come out this will be the default. |
Thank you very much! I am excited to see this change in the new release :) |
@gnebbia I checked some of the changes, and I just wanted to be sure of something :P Could it be that the following code matches your description a bit more: BASE_PATH = Path(os.environ.get("XDG_DATA_HOME",Path(Path.home(),".local","share")),"kb") It looks like that with the current changes, if |
This solves issue #87 that is related to XDG compliance. The error was related to cases where the XDG_DATA_HOME environment variable was defined. In fact, if XDG_DATA_HOME was defined, then PATH_KB would be set to $XDG_DATA_HOME and PATH_KB_GIT would be set to $XDG_DATA_HOME/.git, rather than $XDG_DATA_HOME/kb and $XDG_DATA_HOME/kb/.git, respectively.
Yes you are right, my bad. This the latest commit should fix this for good. Thanks to your suggestion and testing. Really, thanks a lot! |
Expected Behavior
Puts
kb
data in"${XDG_DATA_HOME:-$HOME/.local/share}/kb"
It would be great to follow the XDG Base Directory Specification and place application
data files in
$XDG_DATA_HOME/kb
, if it exists, and fall back to$HOME/.local/share/kb
if that environment variable does not exist. It was mentioned offhand in this thread, but I would like to make an issue to address this :)Actual Behavior
kb
data put in~/.kb
.Steps to Reproduce the Problem
Execute a
kb
commandSpecifications
The text was updated successfully, but these errors were encountered: