[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

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



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 redhat com
 Quality Engineering -- Red Hat, Inc.
==========================================

Attachment: signature.asc
Description: This is a digitally signed message part


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]