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

[Chris Lumens <clumens redhat com>]

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