make required options actually required.

Issue #3 new
Ben Hartshorne
repo owner created an issue

From rtyler in IRC:

if you use dest='log_file' instead of action='store' then OptionParser will print the help if the argument is not there (unless you specify a default kwarg)

Current behavior: maplebed@foo:~$ sudo /usr/sbin/ganglia-logtailer Traceback (most recent call last): File "/usr/sbin/ganglia-logtailer", line 283, in ? main() File "/usr/sbin/ganglia-logtailer", line 171, in main dirsafe_logfile = log_file.replace('/','-') AttributeError: 'NoneType' object has no attribute 'replace'

It should, instead, complain that required arguments are missing and invoke --help

Comments (0)

  1. Log in to comment