-
-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add -L / --dereference option #36
Add -L / --dereference option #36
Conversation
602c778
to
a241d32
Compare
@SUPERCILEX I now managed to make the tests pass on all platforms. What do you think about this feature/PR? |
Signed-off-by: Alex Saveau <[email protected]>
Signed-off-by: Alex Saveau <[email protected]>
Holy shit! I'm very happy to see such a massive real world improvement, that's awesome. Can you double check that my changes still work on your end? Then I'll merge. |
…ption # Conflicts: # fuc_engine/src/ops/copy.rs
Signed-off-by: Alex Saveau <[email protected]>
Indeed! Thank you so much for the project!
I'm out of work now, I hope to test it on Sunday. Also, your changes also tweak the code to my liking ;) |
Yay! I'll be waiting for your confirmation. 👍 |
@SUPERCILEX I confirm, it still works with our CI! |
Sweet! |
Hi,
I added an equivalent to
cp -L
=cp --dereference
option. I need this for my work, where I want to replace acp
command on NFS which takes forever, and it's using-L
. I tried to make it behave likecp -L
does: just treat symlinks as if they were really the files or directories they refer to. FWIW, in my work I was able to replace thecp -rL
withcpz -L
, and it succeeded, reducing the copying time from 49 minutes to 4 minutes!WDYT?