diff options
| author | David Kupka <dkupka@redhat.com> | 2016-06-09 07:58:32 +0200 |
|---|---|---|
| committer | Martin Basti <mbasti@redhat.com> | 2016-06-13 18:04:40 +0200 |
| commit | 9f48c396497bd3e07045838d23afbb0d051dc136 (patch) | |
| tree | d2a4914aeb84d8d5c62e1a5d3cf6bcba4d6a7bc1 /ipapython/install | |
| parent | 54318d1a2c5133fc3a735872b7edc3cfacb032f9 (diff) | |
| download | freeipa-9f48c396497bd3e07045838d23afbb0d051dc136.tar.gz freeipa-9f48c396497bd3e07045838d23afbb0d051dc136.tar.xz freeipa-9f48c396497bd3e07045838d23afbb0d051dc136.zip | |
installer: index() raises ValueError
Expecting IndexError instead of ValueError led to traceback instead of correctly
reporting the error situation.
https://fedorahosted.org/freeipa/ticket/5945
Reviewed-By: Martin Babinsky <mbabinsk@redhat.com>
Diffstat (limited to 'ipapython/install')
| -rw-r--r-- | ipapython/install/cli.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ipapython/install/cli.py b/ipapython/install/cli.py index aed0bc9fe..9cd9ec768 100644 --- a/ipapython/install/cli.py +++ b/ipapython/install/cli.py @@ -299,9 +299,9 @@ class ConfigureTool(admintool.AdminTool): knob_cls = knob_classes[e.name] try: if self.positional_arguments is None: - raise IndexError + raise ValueError index = self.positional_arguments.index(e.name) - except IndexError: + except ValueError: cli_name = knob_cls.cli_name or e.name.replace('_', '-') desc = "option --{0}".format(cli_name) else: |