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

Bash on Ubuntu on Windows - TLDR update command fails #33

Closed
j86989 opened this issue Feb 27, 2018 · 2 comments
Closed

Bash on Ubuntu on Windows - TLDR update command fails #33

j86989 opened this issue Feb 27, 2018 · 2 comments

Comments

@j86989
Copy link

j86989 commented Feb 27, 2018

Reproduction Rate

I could reproduce the issue each time I tried to update TLDR local database.

Steps to Reproduce

  1. Install Windows Subsystem for Linux.
  2. Install Ubuntu on Windows.
  3. Run Bash on Ubuntu on Windows.
  4. Install linuxbrew. There is installation instruction on http://linuxbrew.sh.
  5. Install TLDR C++ client: brew update and brew install tldr.
  6. Try to update TLDR local database: tldr --update.

Result

$ tldr --update
Error: Could Not Rename: /tmp/tldrElmG3W/tldr-master to /mnt/d/joachim/.tldrc/tldr-master/

Expected Result

Update of TLDR local database.

Additional Information

Windows Subsystem for Linux, Bash on Ubuntu on Windows
TLDR C++ has been installed with linuxbrew

I have tried to update TLDR as superuser:

$ sudo tldr --update
[sudo] password for joachim:

There is no error message when I update TLDR as superuser but I'm still unable to print any TLDR page. Each attempt ends with This page doesn't exist yet! error message.

@danimo
Copy link

danimo commented May 9, 2018

The rename() function does not work across file system boundaries. If your $HOME is mounted seperately OR /tmp is a tmpfs, you will see above error. This means tldr will only work successfully in trivial installations, and practically none of the modern Linux distros.

A fix would be to back up with a recursive copy-then-remove if rename() fails, or just use that in all cases.

@SethFalco
Copy link
Member

Closing as a duplicate of the following:

Progress on this has been made by a contributor here:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants