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

Kiwix Desktop does not ignore other libraries when local library.xml is available #460

Closed
kelson42 opened this issue Jun 9, 2020 · 10 comments · Fixed by #1208
Closed

Kiwix Desktop does not ignore other libraries when local library.xml is available #460

kelson42 opened this issue Jun 9, 2020 · 10 comments · Fixed by #1208
Assignees
Milestone

Comments

@kelson42
Copy link
Collaborator

kelson42 commented Jun 9, 2020

I have a local Kiwix Desktop for Linux on my USB stick, with a local empty library.

$ find .
.
./library.xml
./kiwix-desktop_x86_64_2.0.1-1.appimage
kelson@camber:/media/kelson/Kiwix/Kiwix$ cat library.xml 
<library version="20110515"> </library>

But if I started, I don't get an empty local library:
image

It seems to still read my other libraries.

@stale
Copy link

stale bot commented Sep 29, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be now be reviewed manually. Thank you for your contributions.

@stale stale bot added the stale label Sep 29, 2020
@kelson42 kelson42 moved this to To do in Portable Kiwix May 19, 2024
@kelson42 kelson42 added this to the 2.5.0 milestone May 19, 2024
@stale stale bot removed the stale label May 19, 2024
@kelson42 kelson42 moved this from TODO to DOING in Portable Kiwix Aug 16, 2024
@kelson42
Copy link
Collaborator Author

Will be fixed at the same time as #1166 IMHO

@sgourdas
Copy link
Collaborator

@kelson42 isn't this due to the monitor directory in settings? If you reset it, it should not transfer the zim files in the new library.xml.

All ZIM files in this directory will be automatically added to the library.

@kelson42
Copy link
Collaborator Author

kelson42 commented Sep 9, 2024

@sgourdas AFAIK there is current a logic to load library.xml from local and profile directory if it exists. Can you please secure that it has been removed properly/it's not the case anymore?

@kelson42 kelson42 moved this from DOING to REVIEW in Portable Kiwix Sep 9, 2024
@sgourdas
Copy link
Collaborator

sgourdas commented Sep 9, 2024

@kelson42 it is not clear to me. From my experience and knowledge the only thing that loads information from other sources is through the "monitor" function. Am I wrong on this?

@kelson42
Copy link
Collaborator Author

kelson42 commented Sep 9, 2024

@kelson42
Copy link
Collaborator Author

kelson42 commented Sep 9, 2024

@sgourdas I just verified and it works like in the documentation. To me this principle of mixing local and system configuration is made deprecated by the implementation of the .portable approach and should be therefore be removed.

@sgourdas
Copy link
Collaborator

@kelson42 the KIWIX_DATA_DIR option was removed with #1169.

I imagine you are referring to this and you are correct.

As Kiwix Desktop can download and update the library.xml for you. It may be easier to use it. If your default library correspond to the files you want to put on your USB drive. You can simply copy the library.xml and all ZIM files in the user directory top the same directory than the Kiwix Desktop executable.

I will be coming back with a short PR to stop library loading from the executable's path as priority.

@sgourdas
Copy link
Collaborator

sgourdas commented Sep 13, 2024

Should I also properly update the relevant section after merging?

Edit: Nevermind, this is part of #461

@kelson42
Copy link
Collaborator Author

@sgourdas Not necessary, this is a topic for #461

@kelson42 kelson42 moved this from DOING to REVIEW in Portable Kiwix Sep 13, 2024
@kelson42 kelson42 removed this from the 2.5.0 milestone Sep 14, 2024
@kelson42 kelson42 added this to the 2.4.0 milestone Sep 14, 2024
@kelson42 kelson42 moved this from REVIEW to DONE in Portable Kiwix Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: DONE
Development

Successfully merging a pull request may close this issue.

3 participants