Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

remove duplicate backup_files() funcs #12

Merged
merged 1 commit into from

2 participants

@amorton

There were duplicate definitions for backup_files().

I removed the first, and added the log param to the second.

@JeremyGrosser JeremyGrosser merged commit e60d837 into from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Sep 27, 2012
  1. @amorton
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 9 deletions.
  1. +1 −9 tablesnap
View
10 tablesnap
@@ -335,15 +335,7 @@ def backup_file(handler, filename, filedir):
if filename.find('-tmp') != -1:
return
-def backup_files(handler, paths, log=default_log):
- for path in paths:
- log.info('Backing up %s' % path)
- for filename in os.listdir(path):
- if filename.find('-tmp') != -1:
- continue
- handler.add_file(fullpath)
-
-def backup_files(handler, paths, recurse):
+def backup_files(handler, paths, recurse, log=default_log):
for path in paths:
log.info('Backing up %s' % path)
if recurse:
Something went wrong with that request. Please try again.