Re: [lorax 2/2] Allow compression type be specified in lorax.conf

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

 



> @@ -370,13 +379,18 @@ class Lorax(BaseLoraxClass):
>          factory = images.Factory()
>          imgclass = factory.get_class(self.basearch)
>  
> +        ctype = self.conf.get("compression", "type")
> +        cspeed = self.conf.get("compression", "speed")
> +
>          i = imgclass(kernellist=self.outputtree.kernels,
>                       installtree=self.installtree,
>                       outputroot=self.outputtree.root,
>                       product=self.product,
>                       version=self.version,
>                       treeinfo=treeinfo,
> -                     basearch=self.basearch)
> +                     basearch=self.basearch,
> +                     ctype=ctype,
> +                     cspeed=cspeed)
>  
>          # backup required files
>          i.backup_required(self.workdir)
> diff --git a/src/pylorax/images.py b/src/pylorax/images.py
> index 65cf1a6..ece8b00 100644
> --- a/src/pylorax/images.py
> +++ b/src/pylorax/images.py
> @@ -79,7 +79,7 @@ SPARCDIR = "boot"
>  class PPC(object):
>  
>      def __init__(self, kernellist, installtree, outputroot, product, version,
> -                 treeinfo, basearch):
> +                 treeinfo, basearch, ctype, cspeed):
>  
>          self.kernellist = kernellist
>          self.installtree = installtree

You're starting to build up pretty large lists of arguments here.  You
might want to consider making a change like I did to pyanaconda/users.py
in 0d91f1c1e080ba73f32378563fd5f123696a501e.

- Chris

_______________________________________________
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