diff options
author | Martin Nagy <mnagy@redhat.com> | 2008-08-15 18:08:01 +0200 |
---|---|---|
committer | Martin Nagy <mnagy@redhat.com> | 2008-09-11 23:34:01 +0200 |
commit | 885103c32127d10250564e25c5895464fb366f9e (patch) | |
tree | 5db92cd0d4282b3e1aacbfc04c9d076a0d515bec /ipa-admintools/ipa-addgroup | |
parent | 57669ba43224eee0d90556aeea03d14873b4bd7f (diff) | |
download | freeipa-885103c32127d10250564e25c5895464fb366f9e.tar.gz freeipa-885103c32127d10250564e25c5895464fb366f9e.tar.xz freeipa-885103c32127d10250564e25c5895464fb366f9e.zip |
Rework config.py and change cli tools. Maintain order of IPA servers from command line, config and DNS. Parse options before detecting IPA configuration. Don't ignore rest of the options if one is missing in ipa.conf. Drop the --usage options, we will rely on --help. Fixes: 458869, 459070, 458980, 459234
Diffstat (limited to 'ipa-admintools/ipa-addgroup')
-rw-r--r-- | ipa-admintools/ipa-addgroup | 27 |
1 files changed, 12 insertions, 15 deletions
diff --git a/ipa-admintools/ipa-addgroup b/ipa-admintools/ipa-addgroup index d99d588b..09ae9a67 100644 --- a/ipa-admintools/ipa-addgroup +++ b/ipa-admintools/ipa-addgroup @@ -43,15 +43,12 @@ error was: """ % sys.exc_value sys.exit(1) -def usage(): - print "ipa-addgroup [-d|--description STRING] [-g|--gid GID] [--addattr attribute=value] [--setattr attribute=value] [-v|--verbose] group" - sys.exit(1) - def set_add_usage(which): print "%s option usage: --%s NAME=VALUE" % (which, which) def parse_options(): - parser = OptionParser() + usage = "%prog [options] [group]" + parser = OptionParser(usage=usage) parser.add_option("-d", "--description", dest="desc", help="A description of this group") parser.add_option("-g", "--gid", dest="gid", @@ -64,11 +61,14 @@ def parse_options(): parser.add_option("--setattr", dest="setattr", help="Set an attribute, dropping any existing values that may exist", action="append") - parser.add_option("--usage", action="store_true", - help="Program usage") - args = ipa.config.init_config(sys.argv) - options, args = parser.parse_args(args) + ipa.config.add_standard_options(parser) + options, args = parser.parse_args() + + if len(args) > 1: + parser.error("too many arguments") + + ipa.config.init_config(options) return options, args @@ -79,13 +79,10 @@ def main(): group=ipa.group.Group() options, args = parse_options() - if options.usage: - usage() - - if (len(args) != 2): + if len(args) != 1: cn = ipautil.user_input_name("Group name") else: - cn = args[1] + cn = args[0] try: ipaadminutil.check_name(cn) except ValueError, e: @@ -96,7 +93,7 @@ def main(): desc = ipautil.user_input("Description", allow_empty = False) else: desc = options.desc - if (not ipavalidate.String(desc, notEmpty=True)): + if not ipavalidate.String(desc, notEmpty=True): print "Please enter a value" return 1 |