mock: fix dynamic buildrequires unnecesarry loop

per discussion here: rpm-software-management#434 Mock should accept return code 0 from rpm. Relates: rpm-software-management#434
hrnciar · Feb 20, 2020 · c8c4021 · c8c4021
1 parent 6c9682c
commit c8c4021
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/mock/py/mockbuild/backend.py b/mock/py/mockbuild/backend.py
@@ -712,14 +712,12 @@ def get_command(mode):
                 except Error as e:
                     if e.resultcode != 11:
                         raise e
+                finally:
                     max_loops -= 1
                     self.buildroot.root_log.info("Dynamic buildrequires detected")
                     self.buildroot.root_log.info("Going to install missing buildrequires")
                     buildreqs = glob.glob(bd_out + '/SRPMS/*.buildreqs.nosrc.rpm')
                     self.installSrpmDeps(*buildreqs)
-                    packages_after = self.buildroot.all_chroot_packages()
-                    if packages_after == packages_before:
-                        success = True
                     for f_buildreqs in buildreqs:
                         os.remove(f_buildreqs)
                     if not sc: