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

Fixed #403 #411

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fixed #403 #411

wants to merge 2 commits into from

Conversation

jokesper
Copy link

@jokesper jokesper commented Jul 28, 2023

Hoogle will now use the $XDG_DATA_DIR instead of ~/.hoogle as the default path but still use the old one as a fall back if it doesn't exist.
Fixes #403

@kephas
Copy link

kephas commented Feb 8, 2024

Could hoogle issue a warning so that this compatibility can be deprecated in some time?

@jokesper
Copy link
Author

jokesper commented Feb 8, 2024

I marked it as deprecated as that is what warning about a feature is.
Though whether that is in the owner's intent is not known to me.

Hoogle will now use the `$XDG_DATA_DIR` instead of `~/.hoogle` as
default path but still keep use the old one as a fallback.
@jokesper
Copy link
Author

This should now be in an acceptable state with typos and commit / whitespace / spelling problems fixed.
@ndmitchell Could you please take a look at this

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.

Properly support XDG Base Directory Specification
2 participants