tldr-c-client icon indicating copy to clipboard operation
tldr-c-client copied to clipboard

Error: Could Not Rename

Open rmisko11 opened this issue 8 years ago • 2 comments
trafficstars

Reproduction Rate

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

Steps to Reproduce

tldr -u

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

rmisko11 avatar Sep 07 '17 21:09 rmisko11

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.

iguessthislldo avatar Sep 08 '17 07:09 iguessthislldo

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 .

rmisko11 avatar Sep 08 '17 08:09 rmisko11

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.

SethFalco avatar Apr 23 '23 08:04 SethFalco