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

Remove KIWIX_DATA_DIR #1085

Open
kelson42 opened this issue Apr 12, 2024 · 2 comments
Open

Remove KIWIX_DATA_DIR #1085

kelson42 opened this issue Apr 12, 2024 · 2 comments
Assignees
Labels
Milestone

Comments

@kelson42
Copy link
Collaborator

To me this ENV variable KIWIX_DATA_DIR:

  • Is not clear which feature it serves (seems useless to me)
  • I'm again to instrument Kiwix Desktop with "hidden" variable environnements.
  • Is not documented (the best doc I have is this comment from @veloman-yunkan )
  • As a bad naming because only part of the data are stored in it

For all this reasons, I ask if we should just not remove it?

This is a follow-up of kiwix/kiwix-desktop#1081 and kiwix/kiwix-desktop#694

@mgautierfr
Copy link
Member

Is not clear which feature it serves (seems useless to me)

It is a way to configure libkiwix and overide the resolution order of getDataDirectory

I'm again to instrument Kiwix Desktop with "hidden" variable environnements.

It is a libkiwix feature, which is used by kiwix-desktop

Is not documented (the best doc I have is kiwix/kiwix-desktop#694 (comment) from @veloman-yunkan )

Documented here : https://libkiwix.readthedocs.io/en/latest/api/function_tools_8h_1ab559be1bc8eca66c26901660949870c0.html

As a bad naming because only part of the data are stored in it

I would say that it would be to kiwix-desktop to always use the directory returned by kiwix::getDataDirectory unless specific config changed that (and fix kiwix-desktop if it is not the case (whatever we decide about KIWIX_DATA_DIR))

@kelson42 kelson42 transferred this issue from kiwix/kiwix-desktop May 19, 2024
@kelson42 kelson42 added this to the 13.2.0 milestone May 19, 2024
@kelson42
Copy link
Collaborator Author

kelson42 commented May 19, 2024

@mgautierfr Thank you for the clarification, I have moved the issue. The documentation link is also very helpful to understand. I have created an issue to request a proper documentation of all local paths to save/load things in the Wiki kiwix/kiwix-desktop#1114

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: TODO
Development

No branches or pull requests

3 participants