Fix DeleteFile backup-dir handling by MD-Mushfiqur123 · Pull Request #9518 · rclone/rclone · GitHub
Skip to content

Fix DeleteFile backup-dir handling#9518

Open
MD-Mushfiqur123 wants to merge 1 commit into
rclone:masterfrom
MD-Mushfiqur123:fix/deletefile-backupdir
Open

Fix DeleteFile backup-dir handling#9518
MD-Mushfiqur123 wants to merge 1 commit into
rclone:masterfrom
MD-Mushfiqur123:fix/deletefile-backupdir

Conversation

@MD-Mushfiqur123

Copy link
Copy Markdown

Fixes #7566

  • Corrects DeleteFile comment to accurately state it does not use --backup-dir
  • Adds doc note that BackupDir is expensive, call outside loops
  • Adds GetBackupDir convenience wrapper for callers
  • Updates callers that should use DeleteFileWithBackupDir: cmd/deletefile, cmd/ncdu, operations/rc (deletefile rpc)

Fix the misleading comment on DeleteFile that claimed it respects --backup-dir when it doesn't. Update callers that should use DeleteFileWithBackupDir instead.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

operations.DeleteFile does not use --backup-dir, despite comment

2 participants