[anaconda][rhel7-branch][PATCH 00/14] Document Anaconda CLI options & improve option handling (#1088459)

Martin Kolman mkolman at redhat.com
Mon Aug 25 14:43:24 UTC 2014


On Mon, 2014-08-25 at 16:33 +0200, Vratislav Podzimek wrote:
> On Thu, 2014-08-21 at 22:10 +0200, Martin Kolman wrote:
> > A while ago I asked on our IRC channel how to go about backporting the
> > patches that add help texts to all Anaconda boot options and also generally
> > improve our CLI option handling. The general consensus was that I should try
> > to backport as much of the improvements as possible - so here it is! :)
> > 
> > I've started by backporting the 20+ patches to rhel7-branch, followed
> > by aggressive reordering and squashing, with the aim of squashing
> > related stuff (all help texts in a single patch, all option removals
> > in one patch, etc.) and giving the set a logical order (removals before
> > adding help texts and booleanizing flag assignments, etc.).
> > After defusing a lot of amusing merge conflicts I got this 14-piece patch set. :)
> > 
> > Strictly speaking only the patches that add the help texts and the fix for help text parsing
> > are needed to satisfy the requirements of fixing bug 1088459, but the rest improve our
> > option handling quite a lot and also harmonize it with what we have in Fedora right now
> > (argparse, version detection, flags are bools, etc.) - this should make option related
> > backports easier in the future.
> > 
> > Also, any opinions on removing the disused options vs the rhel7 lifecycle ?
> > Most of these options haven't been doing anything for ages, so if they are
> > passed as boot options, they will continue to do just that - Anaconda ignore them
> > as unknown options. If they are passed on CLI, there might be change in
> > behavior (Anaconda reporting the option is unknown) instead of accepting
> > the option but not doing anything with it.
> > 
> > 
> > 
> > Brian C. Lane (1):
> >   Fix --kickstart option
> > 
> > David Shea (2):
> >   Allow the location of anaconda_options.txt to be overridden
> >   Add anaconda_options.txt to makeupdates
> > 
> > Martin Kolman (10):
> >   Return CLI help text at once
> >   Don't overwrite function argument when parsing help texts
> >   Switch Anaconda to argparse
> >   Format the help text to properly fit to the terminal window
> >   Parse boot options before parsing CLI options (#1101341)
> >   Short-circuit initialization when printing out Anaconda version
> >   Move Anaconda version detection from isys to Python code
> >   Remove obsolete and disused Anaconda options
> >   Use booleans and direct assignments for flags
> >   Add help texts to Anaconda CLI options
> > 
> > Vratislav Podzimek (1):
> >   Move the Anaconda class to a proper module
> > 
> >  .gitignore                      |   1 +
> >  anaconda                        | 289 +++++++++++++++++----------------
> >  configure.ac                    |   1 +
> >  data/anaconda_options.txt       | 202 +++++++++++++++++++++++
> >  docs/driverdisc.txt             |   4 +-
> >  pyanaconda/Makefile.am          |   6 +-
> >  pyanaconda/__init__.py          | 229 --------------------------
> >  pyanaconda/anaconda.py          | 230 ++++++++++++++++++++++++++
> >  pyanaconda/anaconda_argparse.py | 346 ++++++++++++++++++++++++++++++++++++++++
> >  pyanaconda/anaconda_optparse.py | 249 -----------------------------
> >  pyanaconda/exception.py         |   4 +-
> >  pyanaconda/flags.py             |  18 +--
> >  pyanaconda/isys/Makefile.am     |   3 +-
> >  pyanaconda/isys/__init__.py     |   4 +-
> >  pyanaconda/isys/isys.c          |   6 -
> >  pyanaconda/startup_utils.py     |  80 ++++++++++
> >  pyanaconda/version.py.in        |   1 +
> >  scripts/makeupdates             |   2 +
> >  18 files changed, 1023 insertions(+), 652 deletions(-)
> >  create mode 100644 pyanaconda/anaconda.py
> >  create mode 100644 pyanaconda/anaconda_argparse.py
> >  delete mode 100644 pyanaconda/anaconda_optparse.py
> >  create mode 100644 pyanaconda/startup_utils.py
> >  create mode 100644 pyanaconda/version.py.in
> Apart from the already posted comments these all look good to me. Nice
> job!
Thanks! :)
> 




More information about the anaconda-patches mailing list