tldr-c-client
tldr-c-client copied to clipboard
Error: Could Not Rename
Reproduction Rate
<How often could you reproduce this issue, how often do you tried?> always, a lot
Steps to Reproduce
Result
<What is the result of this?> e.g. Error: Could Not Rename: /tmp/tldrmkhmBO/tldr-master to /home/roger/.tldrc/tldr-master/
Expected Result
<What is the expected result?> File renamed and moved
Additional Information
<Which version? Which operating system? Any other 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
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.
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 [email protected] 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 .
Closing as a duplicate of the following:
- https://github.com/tldr-pages/tldr-c-client/issues/95
Progress on this has been made by a contributor here:
- https://github.com/tldr-pages/tldr-c-client/pull/100
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.