Fixes and improvements to the ltfs_ordered_copy tool#586
Conversation
|
I've increased the buffer size to 256MiB, since with a lower values, such as the ones defined in the |
|
Could you please also add move functionality with a flag. I don't recall such feature in the old versions. Would be nice for moving directories from tape to another storage. EDIT: Nevermind I made another pr for that. |
XV02
left a comment
There was a problem hiding this comment.
Thanks for your work, it looks great, just one comment
XV02
left a comment
There was a problem hiding this comment.
LGTM! Thanks for all your work and quick resolution of requests
vandelvan
left a comment
There was a problem hiding this comment.
Thank you for your contribution! LGTM
Summary of changes
This pull request includes following changes or fixes:
ltfs_ordered_copytool crashing due to comparingbyteswithstr.Description
The first fix was required for me to even use this tool under modern python and
pyxattr.I believe a better progress bar might be more useful when copying smaller amounts of larger files - as of right now, this utility only shows the number of the file that's being copied.
I also believe it might be worth documenting somewhere, that sometimes it's absolutely crucial to use this tool instead of the system-avaliable
cporrsync. Without it, my drive (SAS IBM LTO7-HH) kept thrashing and throwing IO errors. Also, since python2 reached EOL six years ago, I think it might be worth removing mentions of it from the README.Type of change
Checklist: