-
Notifications
You must be signed in to change notification settings - Fork 29
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
Unable to drop files from a remote #8
Comments
The failure with "purge" happens only with rclone 1.33. I think the behaviour of "purge" in the new version of rclone is correct because now the command refuse to remove files, it accepts only directories. With the old versions of rclone, you can use "purge" on a file but it remove also the directory containing the file. |
Thanks for the report. I believe version 0.4 should address issues with rclone 1.33. If you run into any further problems, please feel free to re-open this issue or file a new issue. |
@montag451 I just read more closely what you said. Are you saying that "rclone purge" on older versions would delete a parent directory even if other files existed in that directory? If so, please let me know, because that would mean older git-annex-remote-rclone versions may be unsafe to use. |
Yes. For example, say that you have a directory "foo" on a remote with 2 files inside it (say "bar" and "baz"). If you type the following command: rclone purge remote:foo/bar with previous version of rclone, it will delete "bar", "baz" and the directory "foo". So I think previous version of git-annex-remote-rclone are unsafe since dropping one file could in fact drop several files. |
That's extremely troubling. Thank you for the report. |
You are welcome! |
Added a warning in README.md in 3c2b84b |
When I try to drop files from a remote, it doesn't work. Looking at the code, I see that you use "rclone purge" to remove file from a remote, I think you should use "rclone delete" because "rclone purge" only works on "directory".
The text was updated successfully, but these errors were encountered: