search

tldr-pages / tldr-c-client

C command-line client for tldr pages
MIT License
296 stars 49 forks source link

Error: Could Not Rename #24

Closed rmisko11 closed 1 year ago

rmisko11 commented 6 years ago

Reproduction Rate

<How often could you reproduce this issue, how often do you tried?> always, a lot

Steps to Reproduce

tldr -u ## Result e.g. Error: Could Not Rename: /tmp/tldrmkhmBO/tldr-master to /home/roger/.tldrc/tldr-master/ ## Expected Result File renamed and moved ## Additional Information tldr v1.3.0 (v1.3.0-12-g0e54281) Linux Kilo 4.10.1-041001-generic #201702260735 SMP Sun Feb 26 12:36:48 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux /tmp and /home are in separate partitions which maybe why rename reports the following error: Invalid cross-device link
iguessthislldo commented 6 years ago

This is what #21 was meant to solve, unfortunately, it has yet to be accepted. If you wanted this to work at the moment you could install my fork or, maybe easier, use @12345ieee's workaround in the PR comments which manually puts the tldr files in the correct place.

rmisko11 commented 6 years ago

Thanks for the feedback.

Sorry, I don't know git-hub very well, I just saw the 5 issues and didn't see #21.

Anyway, for my quick and dirty work around I just replaced the rename with the system command using mv.

On Fri, Sep 8, 2017 at 3:26 AM, Fred Hornsey notifications@github.com wrote:

This is what #21 https://github.com/tldr-pages/tldr-cpp-client/pull/21 was meant to solve, unfortunately, it has yet to be accepted. If you wanted this to work at the moment you could install my fork or, maybe easier, use @12345ieee https://github.com/12345ieee's workaround in the PR comments which manually puts the tldr files in the correct place.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tldr-pages/tldr-cpp-client/issues/24#issuecomment-328023487, or mute the thread https://github.com/notifications/unsubscribe-auth/AY3hc-yrzJnrnyaCq9qhqngBEzLu0fG2ks5sgOxCgaJpZM4PQZQS .

SethFalco commented 1 year ago

Closing as a duplicate of the following:

Progress on this has been made by a contributor here:

I recognize this one is older and had more thumbs up. I opted to close this one, however, as the more recent issue is better formatted.