Re: [PATCH] The script sections should operate on an AnacondaKSScript instance (#728468).

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

 



On Tue, 2011-08-09 at 14:07 -0400, Chris Lumens wrote:
> This is necessary to make sure they have a run method.
> ---
>  pyanaconda/kickstart.py |    8 +++++++-
>  1 files changed, 7 insertions(+), 1 deletions(-)
> 
> diff --git a/pyanaconda/kickstart.py b/pyanaconda/kickstart.py
> index 9a8b636..15a46b2 100644
> --- a/pyanaconda/kickstart.py
> +++ b/pyanaconda/kickstart.py
> @@ -51,7 +51,7 @@ from pykickstart.base import KickstartCommand, BaseData
>  from pykickstart.constants import *
>  from pykickstart.errors import formatErrorMsg, KickstartError, KickstartValueError, KickstartParseError
>  from pykickstart.parser import Group, KickstartParser, Packages, Script
> -from pykickstart.sections import PreScriptSection, NullSection
> +from pykickstart.sections import *

Why do you use import * so much?
Otherwise everything looks good.

>  from pykickstart.version import returnClassForVersion
>  
>  import gettext
> @@ -1259,6 +1259,12 @@ class AnacondaKSParser(KickstartParser):
>          self.handler.add(retval)
>          return retval
>  
> +    def setupSections(self):
> +        self.registerSection(PreScriptSection(self.handler, dataObj=AnacondaKSScript))
> +        self.registerSection(PostScriptSection(self.handler, dataObj=AnacondaKSScript))
> +        self.registerSection(TracebackScriptSection(self.handler, dataObj=AnacondaKSScript))
> +        self.registerSection(PackageSection(self.handler))
> +
>  def doKickstart(anaconda):
>      storage.storageInitialize(anaconda)
>      # Having initialized storage, we can apply all the other kickstart commands.

-- 
Martin Gracik <mgracik@xxxxxxxxxx>

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list


[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux