Re: Partitioning installclasses server.py

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

 



On the side - Why aren't you folks simply using a kickstart file for this?

Rgrds

On 4/13/05, Matthew Miller <mattdm@xxxxxxxxxx> wrote:
> On Wed, Apr 13, 2005 at 09:53:08AM +0200, fonteneau wrote:
> >         autorequests = [ ("/", "ext3", 512, 1024, 1, 1), ("/usr",
> > "ext3", 1024, 2048, 1, 1)...
> 
> I'm doing the same thing, and here's my patches to do it. These might not
> apply exactly cleanly to unmodified anaconda, but you'll get the idea.
> 
> First, the patch to actually make this a parameter:
> 
> --- anaconda-10.1.1.13.orig/installclass.py     2004-10-04 11:51:42.000000000 -0400
> +++ anaconda-10.1.1.13/installclass.py  2005-03-29 16:45:32.273176224 -0500
> @@ -528,8 +528,8 @@
>          id.setMouse(mouse)
> 
>      def setDefaultPartitioning(self, partitions, clear = CLEARPART_TYPE_LINUX,
> -                               doClear = 1):
> -        autorequests = [ ("/", None, 1024, None, 1, 1, 1) ]
> +                               doClear = 1,
> +                               autorequests = [ ("/", None, 1024, None, 1, 1, 1) ] ):
> 
>          bootreq = getAutopartitionBoot()
>          if bootreq:
> 
> And then, the patch to the server install class to take advantage of it:
> 
> --- anaconda-10.1.1.13.orig/installclasses/server.py    2005-03-29 16:31:04.000000000 -0500
> +++ anaconda-10.1.1.13/installclasses/server.py 2005-03-29 16:42:57.000000000 -0500
> @@ -34,7 +34,11 @@
>      def setInstallData(self, id):
>         BaseInstallClass.setInstallData(self, id)
>          BaseInstallClass.setDefaultPartitioning(self, id.partitions,
> -                                                CLEARPART_TYPE_ALL)
> +                                                CLEARPART_TYPE_ALL, 1,
> +                                                [ ("/", None, 1024, None, 0, 1, 1),
> +                                                  ("/usr", None, 2048, 8192, 1, 1, 1),
> +                                                  ("/var", None, 2048, 4096, 1, 1, 1),
> +                                                  ("/home", None, 512, None, 1, 1, 1) ] )
> 
>      def __init__(self, expert):
>         BaseInstallClass.__init__(self, expert)
> 
> Which will work, but then you'll get crashes if you go back and forth from
> the partition review screen to the one before it. So you also need this, to
> make it *not* clear the partition requests on you. I've tested this and it
> works, but it may have some consequence I don't understand.
> 
> --- anaconda-10.1.1.13/autopart.py.dontdothat   2005-04-01 13:31:04.824378465 -0500
> +++ anaconda-10.1.1.13/autopart.py      2005-04-01 13:32:45.599676940 -0500
> @@ -1197,7 +1197,6 @@
>          diskset.refreshDevices()
>          partitions.setFromDisk(diskset)
>          partitions.setProtected(dispatch)
> -        partitions.autoPartitionRequests = []
>          return
> 
>      # if no auto partition info in instclass we bail
> 
> --
> Matthew Miller           mattdm@xxxxxxxxxx        <http://www.mattdm.org/>
> Boston University Linux      ------>                <http://linux.bu.edu/>
> 
> _______________________________________________
> Anaconda-devel-list mailing list
> Anaconda-devel-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list
> 


-- 
Nonchalantly yours
GobbledeGeek
[Every thing but Gobbledegook.. !!]


[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