backuplocal => backup_local for API standardization reasons

pull/993/merge
Michael DeHaan 12 years ago
parent b97e2a6f5a
commit aa704a6111

@ -276,7 +276,7 @@ class AnsibleModule(object):
infile.close()
return digest.hexdigest()
def backuplocal(self, fn):
def backup_local(self, fn):
'''make a date-marked backup of the specified file, return True or False on success or failure'''
# backups named basename-YYYY-MM-DD@HH:MM~
ext = time.strftime("%Y-%m-%d@%H:%M~", time.localtime(time.time()))

@ -75,7 +75,7 @@ def main():
if pathmd5 != destmd5:
if backup and destmd5 is not None:
module.backuplocal(dest)
module.backup_local(dest)
shutil.copy(path, dest)
changed = True

@ -63,7 +63,7 @@ def main():
try:
if backup:
if os.path.exists(dest):
backup_file = module.backuplocal(dest)
backup_file = module.backup_local(dest)
shutil.copyfile(src, dest)
except shutil.Error:
module.fail_json(msg="failed to copy: %s and %s are the same" % (src, dest))

@ -70,7 +70,7 @@ def present(module, name, regexp, line, insertafter, backup):
if changed:
if backup:
module.backuplocal(name)
module.backup_local(name)
f = open(name, 'wb')
f.writelines(lines)
f.close()
@ -93,7 +93,7 @@ def absent(module, name, regexp, backup):
changed = len(found) > 0
if changed:
if backup:
module.backuplocal(name)
module.backup_local(name)
f = open(name, 'wb')
f.writelines(lines)
f.close()

Loading…
Cancel
Save