Commit 4bf20bf6 authored by Kevin Lyda's avatar Kevin Lyda 💬
Browse files

Fix option parsing for real.

parent e76ed418
Pipeline #1032 passed with stage
in 3 minutes and 44 seconds
......@@ -38,29 +38,25 @@ def main(argv):
log = check.LogCounter()
(options, args) = get_whitelisted_users(argv)
(options, args) = parse_chkcrontab_options(argv)
for crontab in args[1:]:
print('Checking correctness of %s' % crontab)
return check.check_crontab(crontab, log, options['whitelisted_users'])
def get_whitelisted_users(argv):
"""Gets the list of whitelisted users, if any.
def parse_chkcrontab_options(argv):
"""Parse the options for chkcrontab.
argv: The argument string supplied by the caller.
The list of whitelisted users.
options: A dictionary of options and their values.
args: Remaining arguments.
parser = OptionParser()
parser.add_option('-w', '--whitelist', dest='whitelisted_users', action='append',
help='A user to ignore when warning of unrecognized users This argument may be passed multiple times.')
(options, args) = parser.parse_args(argv)
if options.whitelisted_users:
return options.whitelisted_users
return None
return parser.parse_args(argv)
if __name__ == '__main__':
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment