Summary

Fixes #7566 - DeleteFile and DeleteFiles now actually respect the --backup-dir flag instead of always passing nil.

Before: Both functions always deleted files regardless of --backup-dir:

func DeleteFile(ctx context.Context, dst fs.Object) (err error) {
    return DeleteFileWithBackupDir(ctx, dst, nil)
}

After: Both functions check --backup-dir config and move files to the backup directory when configured:

func DeleteFile(ctx context.Context, dst fs.Object) (err error) {
    ci := fs.GetConfig(ctx)
    var backupDir fs.Fs
    if ci.BackupDir != "" {
        backupDir, err = cache.Get(ctx, ci.BackupDir)
        ...
    }
    return DeleteFileWithBackupDir(ctx, dst, backupDir)
}

Changes

• fs/operations/operations.go: Updated DeleteFile and DeleteFiles to look up --backup-dir from config and pass it to the underlying DeleteFileWithBackupDir / DeleteFilesWithBackupDir functions

Test plan

• go build ./fs/operations/ succeeds