Skip to content

Commit

Permalink
tklbam-backup cli: removed -s shorthand for --simulate
Browse files Browse the repository at this point in the history
rational:

- gnu_getopt provides implicit shorthands such as --sim, which is short enough.
- -s is ambiguous, may stand for --silent, etc.
- consistency with tklbam-restore interface
  • Loading branch information
lirazsiri committed Aug 28, 2013
1 parent 28cd3bb commit 65d8782
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions cmd_backup.py
Expand Up @@ -27,7 +27,7 @@
--resume Resume aborted backup session
--disable-resume Disable implicit --resume when rerunning an aborted backup
-s --simulate Do a dry run simulation of the backup.
--simulate Do a dry run simulation of the backup.
-q --quiet Be less verbose
--logfile=PATH Path of file to log to
Expand Down Expand Up @@ -138,7 +138,7 @@ def get_server_id():

def main():
try:
opts, args = getopt.gnu_getopt(sys.argv[1:], 'qsh',
opts, args = getopt.gnu_getopt(sys.argv[1:], 'qh',
['help',
'dump=',
'skip-files', 'skip-database', 'skip-packages',
Expand Down Expand Up @@ -178,7 +178,7 @@ def main():
opt_disable_resume = True
conf.verbose = False

elif opt in ('-s', '--simulate'):
elif opt == '--simulate':
conf.simulate = True

elif opt == '--resume':
Expand Down

0 comments on commit 65d8782

Please sign in to comment.