-
-
Notifications
You must be signed in to change notification settings - Fork 95
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
tldr --update_cache
does not delete local-only pages
#242
Comments
|
Hi, thank you for opening this issue! This repository is just for the pages, specification for tldr clients (programs to visualize tldr pages), scripts for updating pages... But not for tldr clients. So your proposal would be for enforcing this to all clients. Well, I see some pros and cons of not allowing local-only pages:
We have to discuss if the pros would outweigh the cons on doing this (or if there is another unseen pro or con, really).
PS: I don't really see how the comment above is related with the original issue |
Which tldr client do you use? I.e. how did you install it? |
I do not think that mooreye is proposing to force clients to disallow local-only pages (which would be next to impossible anyways). I think that this is not an issue with the client spec, because the client spec uses the term "cache" and updating a cache implicates deleting outdated entries IMO. If I understood correctly, this is an issue with the specific tldr client used by mooreye. |
I use Fedora's DNF package, which is now v3.2.0 Their upstream info points to https://github.com/tldr-pages/tldr-python-client |
Thanks! I transferred the issue into the python-client's repo. Also, I have spotted the cause of the issue: The Lines 380 to 407 in 68acf05
|
How to reproduce on Linux:
tldr -u
~/.cache/tldr/pages/linux/
dircp cp.md zzzzz.md
tldr zzzzz
now shows the copiedcp
pagetldr -u
againzzzzz.md
is still present.The text was updated successfully, but these errors were encountered: