[PATCH] If no partitioning commands are given, apply the UI selections (#490880).

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

 



First, always apply the default partitioning scheme in kickstart installs.
Then, always run clearPartitions since it knows best.  Finally, if any of
the various kickstart partitioning commands are given, unset doAutoPart so
the user-provided partitioning scheme takes effect.
---
 installclasses/fedora.py |   10 ++++------
 installclasses/rhel.py   |   10 ++++------
 kickstart.py             |    8 ++++++++
 storage/partitioning.py  |    3 +--
 4 files changed, 17 insertions(+), 14 deletions(-)

diff --git a/installclasses/fedora.py b/installclasses/fedora.py
index d2ddafa..69f198d 100644
--- a/installclasses/fedora.py
+++ b/installclasses/fedora.py
@@ -61,12 +61,10 @@ class InstallClass(BaseInstallClass):
 
     def setInstallData(self, anaconda):
 	BaseInstallClass.setInstallData(self, anaconda)
-
-        if not anaconda.isKickstart:
-            BaseInstallClass.setDefaultPartitioning(self,
-                                                    anaconda.id.storage,
-                                                    anaconda.platform,
-                                                    CLEARPART_TYPE_LINUX)
+        BaseInstallClass.setDefaultPartitioning(self,
+                                                anaconda.id.storage,
+                                                anaconda.platform,
+                                                CLEARPART_TYPE_LINUX)
 
     def setSteps(self, anaconda):
 	BaseInstallClass.setSteps(self, anaconda);
diff --git a/installclasses/rhel.py b/installclasses/rhel.py
index da25946..5217928 100644
--- a/installclasses/rhel.py
+++ b/installclasses/rhel.py
@@ -87,12 +87,10 @@ class InstallClass(BaseInstallClass):
 
     def setInstallData(self, anaconda):
 	BaseInstallClass.setInstallData(self, anaconda)
-
-        if not anaconda.isKickstart:
-            BaseInstallClass.setDefaultPartitioning(self, 
-                                                    anaconda.id.storage,
-                                                    anaconda.platform,
-                                                    CLEARPART_TYPE_LINUX)
+        BaseInstallClass.setDefaultPartitioning(self, 
+                                                anaconda.id.storage,
+                                                anaconda.platform,
+                                                CLEARPART_TYPE_LINUX)
 
     def setSteps(self, anaconda):
         dispatch = anaconda.dispatch
diff --git a/kickstart.py b/kickstart.py
index 844e58c..5ef435e 100644
--- a/kickstart.py
+++ b/kickstart.py
@@ -347,6 +347,8 @@ class LogVol(commands.logvol.F9_LogVol):
         storage = self.handler.id.storage
         devicetree = storage.devicetree
 
+        storage.doAutoPart = False
+
         if lvd.mountpoint == "swap":
             type = "swap"
             lvd.mountpoint = ""
@@ -576,6 +578,8 @@ class Partition(commands.partition.F9_Partition):
         devicetree = storage.devicetree
         kwargs = {}
 
+        storage.doAutoPart = False
+
         if pd.onbiosdisk != "":
             pd.disk = isys.doGetBiosDisk(pd.onbiosdisk)
 
@@ -724,6 +728,8 @@ class Raid(commands.raid.F9_Raid):
         devicetree = storage.devicetree
         kwargs = {}
 
+        storage.doAutoPart = False
+
         if rd.mountpoint == "swap":
             type = "swap"
             rd.mountpoint = ""
@@ -877,6 +883,8 @@ class VolGroup(commands.volgroup.FC3_VolGroup):
         storage = self.handler.id.storage
         devicetree = storage.devicetree
 
+        storage.doAutoPart = False
+
         # Get a list of all the physical volume devices that make up this VG.
         for pv in vgd.physvols:
             dev = devicetree.getDeviceByName(pv)
diff --git a/storage/partitioning.py b/storage/partitioning.py
index d824fa6..1bd7807 100644
--- a/storage/partitioning.py
+++ b/storage/partitioning.py
@@ -161,8 +161,7 @@ def doAutoPartition(anaconda):
     disks = []
     devs = []
 
-    if anaconda.id.storage.doAutoPart and not anaconda.isKickstart:
-        # kickstart uses clearPartitions even without autopart
+    if anaconda.id.storage.doAutoPart:
         clearPartitions(anaconda.id.storage)
 
     if anaconda.id.storage.doAutoPart:
-- 
1.6.1.3

_______________________________________________
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