Hook up the "Remove Packages" button on the dep solving error screen.

All this does is set skip_broken and yum does all the rest.  Note that we
are still doing dep solving a second time, when the progress hub is up, but
adding this setting to the Payload object means skip_broken will be in effect
there too.  We still need proper error handling for other errors that might
occur at that point.

This should take care of #865073.  I can't think of any cases where you can
get past the first hub with a dep solving error.

2 files changed, 8 insertions, 0 deletions

diff --git a/pyanaconda/packaging/__init__.py b/pyanaconda/packaging/__init__.py
index 6462a286d..b7402c762 100644
--- a/pyanaconda/packaging/__init__.py
+++ b/pyanaconda/packaging/__init__.py
@@ -640,6 +640,10 @@ class ArchivePayload(ImagePayload):
     pass
 
 class PackagePayload(Payload):
+    def __init__(self, *args, **kwargs):
+        Payload.__init__(self, *args, **kwargs)
+        self.skipBroken = False
+
     """ A PackagePayload installs a set of packages onto the target system. """
     @property
     def kernelPackages(self):
diff --git a/pyanaconda/packaging/yumpayload.py b/pyanaconda/packaging/yumpayload.py
index e42d44368..108992c09 100644
--- a/pyanaconda/packaging/yumpayload.py
+++ b/pyanaconda/packaging/yumpayload.py
@@ -1101,6 +1101,10 @@ reposdir=%s
         log.info("checking software selection")
         self.txID = time.time()
 
+        if self.skipBroken:
+            log.info("running software check with skip_broken = True")
+            self._yum.conf.skip_broken = True
+
         self.release()
         self.deleteYumTS()