[anaconda][rhel7-branch/master][PATCH] Don't crash if escrow certificate is requested without network access (#1085265)
Chris Lumens
clumens at redhat.com
Fri Sep 26 15:39:34 UTC 2014
> There's nothing wrong with the value, it's the context, so KickstartError
> is the most appropriate to raise.
"""
It also exports several exception classes:
KickstartError - A generic exception class.
"""
So, KicsktartError is really intended for things that don't fit anywhere
else. I see one use in pykickstart/commands/repo.py that really ought to
be something else, and then there's a couple in pykickstart/parser.py
that at least need to be audited.
But in general, it's meant as a top-level exception class for everything
to derive from. It's also more meant to be caught than raised. Think
about "except Exception" instead of "raise Exception".
- Chris
More information about the anaconda-patches
mailing list