Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

sync remote2remote: set target_uri on updated files too, bug 131 #133

Merged
merged 1 commit into from

2 participants

@mdomsch
Owner

fixes bug #131

We were not setting the target_uri field on files that had changed,
only on files that were to be newly uploaded. This was causing a
traceback when referencing the target_uri field when copying the
changed file.

@mdomsch mdomsch sync remote2remote: set target_uri on updated files too, bug 131
fixes bug s3tools#131

We were not setting the target_uri field on files that had changed,
only on files that were to be newly uploaded.  This was causing a
traceback when referencing the target_uri field when copying the
changed file.
2a4fafc
@mludvig mludvig merged commit 7749423 into from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 6, 2013
  1. @mdomsch

    sync remote2remote: set target_uri on updated files too, bug 131

    mdomsch authored mdomsch committed
    fixes bug s3tools#131
    
    We were not setting the target_uri field on files that had changed,
    only on files that were to be newly uploaded.  This was causing a
    traceback when referencing the target_uri field when copying the
    changed file.
This page is out of date. Refresh to see the latest.
Showing with 5 additions and 4 deletions.
  1. +5 −4 s3cmd
View
9 s3cmd
@@ -656,10 +656,11 @@ def cmd_sync_remote2remote(args):
print(u"Summary: %d source files to copy, %d files at destination to delete" % (src_count, dst_count))
- if src_count > 0:
- ### Populate 'remote_uri' only if we've got something to sync from src to dst
- for key in src_list:
- src_list[key]['target_uri'] = destination_base + key
+ ### Populate 'target_uri' only if we've got something to sync from src to dst
+ for key in src_list:
+ src_list[key]['target_uri'] = destination_base + key
+ for key in update_list:
+ update_list[key]['target_uri'] = destination_base + key
if cfg.dry_run:
for key in exclude_list:
Something went wrong with that request. Please try again.