Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
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 JeremyGrosser:master
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.