[PATCH] Use KSOptionParser so we can catch bad command options

James Laska jlaska at redhat.com
Thu Jan 29 19:45:40 UTC 2009


On Thu, 2009-01-29 at 14:15 -0500, Chris Lumens wrote:
>  This looks okay to me.  Are there other places that would benefit
> from using this idiom?

I think it's potentially anything that's not using KSOptionParser.

# grep -L KSOptionParser pykickstart/commands/*py
authconfig.py
deviceprobe.py
displaymode.py
__init__.py
interactive.py
iscsiname.py
key.py
keyboard.py
lang.py
lilocheck.py
mediacheck.py
skipx.py
updates.py
zerombr.py

How would you feel about a patch that adds KSOptionParser parse_args
processing to each of these?

Thanks,
James

-- 
==========================================
 James Laska         -- jlaska at redhat.com
 Quality Engineering -- Red Hat, Inc.
==========================================
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
URL: <http://listman.redhat.com/archives/kickstart-list/attachments/20090129/b933b9a1/attachment.sig>


More information about the Kickstart-list mailing list