Error when try to update local database about tldr-c-client HOT 13 CLOSED

No, we didn't take care of this yet. Here's more info. It will just stay there until you manually delete it. But it has nothing to do with this issue.

from tldr-c-client.

#21

from tldr-c-client.

Update: I figured out the issue. In Linux (and probably macOS too) /tmp has it's own filesystem called tmpfs. We now try to rename the tldr directory in /tmp/tldrXXXX/tldr-main to ~/.tldrc/tldr, which doesn't work, because the rename() command doesn't support that. It set's the errno code 18 - Invalid cross-device link. This issue even existed in the code more than 5 years ago. Honestly, I'm wondering how this worked at all...

I'm currently making a PR to replace the rename() function to a copy() and delete().

from tldr-c-client.

I started by checking I was still seeing the issue:

Geoffs-MacBook-Pro:~ $ tldr -u
Error: Could Not Rename: /tmp/tldrYmdBuU/tldr to /Users/xxxxx/.tldrc/tldr/
Geoffs-MacBook-Pro:~ $ brew install tldr --HEAD
Updating Homebrew...
==> Auto-updated Homebrew!Updated 3 taps (homebrew/cask-versions, homebrew/core and homebrew/cask).

Error: tldr 1.4.0 is already installed
To install HEAD, first run:
  brew unlink tldr
Geoffs-MacBook-Pro:~ $ brew unlink tldr
Unlinking /usr/local/Cellar/tldr/1.4.0... 5 symlinks removed.
Geoffs-MacBook-Pro:~ $ brew install tldr --HEAD
==> Cloning https://github.com/tldr-pages/tldr-c-client.git
Cloning into '/Users/xxxxx/Library/Caches/Homebrew/tldr--git'...
==> Checking out branch master
Already on 'master'
Your branch is up to date with 'origin/master'.
==> make PREFIX=/usr/local/Cellar/tldr/HEAD-2faf09e install
==> Caveats
Bash completion has been installed to:
  /usr/local/etc/bash_completion.d
==> Summary
🍺  /usr/local/Cellar/tldr/HEAD-2faf09e: 10 files, 82.1KB, built in 11 seconds
Removing: /Users/xxxxx/Library/Caches/Homebrew/tldr--1.4.0... (14.7KB)
Geoffs-MacBook-Pro:~ $ tldr -u
Successfully updated local database
Geoffs-MacBook-Pro:~ $ 

I needed to unlink, but after that it worked, which is great! Thank you very much.

Not critical but tldr -v returns tldr v1.3.0 (v1.4.0-2-g2faf09e). Oh and I checked in the .tldrc directory and found both tldr-mainand tldr-masterwhich I believe is as expected.

If that was not what you wanted, or I did something wrong, please just ask.

from tldr-c-client.

Thanks, that was exactly what I wanted to know. We will create a new release with the fix soon, so you can get the new version without the --HEAD flag.

from tldr-c-client.

I just released the fix. Thanks for your help @glawrence

from tldr-c-client.

This is almost definitely #59

Did the fix work from #21

from tldr-c-client.

I tried the fix at #21 (comment) which did not work for me. However I note that, as above tldr -v -u shows main.zip and the instructions on #21 I just linked to refer to master.zip

from tldr-c-client.

This is really strange, because I made #59 to fix exactly this issue and it worked for me. Now, I'm getting the same error.

from tldr-c-client.

Argh, how annoying @marchersimon ! I guess on the plus side, from our point of view, it is useful you can reproduce the issue.... Happy to try and help by testing stuff on macOS

from tldr-c-client.

The main reason this is failing is, because the archive we download is called main.zip, because it's coming from the main branch. The archive contains the directory tldr-main/, so when we extract main.zip, we get tldr-main, which the program doesn't expect. I'll keep you up to date.

from tldr-c-client.

Will it remove ~/.tldrc/tldr-master as part of the process? I am guessing that's not needed, although removing it might stop tldr from working, I guess

from tldr-c-client.

@glawrence the fix for macOS is merged now. Could you please verify that it's working by updating your client using the --HEAD opption?

from tldr-c-client.