Re: [PATCH 3/8] patch anaconda itself to use AnacondaKSParser

["Alford, Seth" <Seth_Alford adp com>]

On Fri, Jul 10, 2009 at 02:53:28PM -0500, Jeremy Katz wrote:
> On Friday, July 10 2009, Alford, Seth said:
> > We had to modify the anaconda script itself to include the
> > AnacondaKSParser module.  We did these modifications based on
> > Chris Lumens' "pykickstart Programmer's Guide", which currently
> > lives in /usr/share/doc/pykickstart-0.43.3/programmers-guide and
> > comes with pykickstart.
> 
> The current code in git master is very different here, so what are you
> trying to achieve with this so we can see if it's done or if it's
> something that's worth considering there
> 
> Jeremy

I'm trying to add support for the --interactive and -s flags for
pre/post install scripts.

To do that, I needed to modify either AnacondaKSParser in
kickstart.py, or KickstartParser in
/usr/lib/python2.4/site-packages/pykickstart/parser.py.

If I modified pykickstart.py, I have to rebuild another rpm
package, pykickstart.  If I modify kickstart.py, I have to
rebuild anaconda, which I'm doing already.  It seemed simpler to
jus modify AnacondaKSParser.

--Seth


This message and any attachments are intended only for the use of the addressee and may contain information that is privileged and confidential. If the reader of the message is not the intended recipient or an authorized representative of the intended recipient, you are hereby notified that any dissemination of this communication is strictly prohibited. If you have received this communication in error, please notify us immediately by e-mail and delete the message and any attachments from your system.