[Piuparts-commits] [SCM] piuparts git repository branch, develop, updated. 0.45-102-g4c48ac2

[Andreas Beckmann]

The following commit has been merged in the develop branch:
commit 1872c5f98faaef104c5be86b7282eabc24208d2d
Author: Andreas Beckmann <debian at abeckmann.de>
Date:   Sat Jul 28 08:35:46 2012 +0200

    p: --install-purge-install: use restore_selections()
    
    Use restore_selections instead of purge_packages() for the
    --install-purge-install test. purge_packages() fails miserably on
    packages with circular dependencies and on transitively essential
    packages that are already installed the base chroot. Purging
    works fine, but in the end there are various amounts of spuriously
    added, removed, or modified files.
    
    --install-purge-install does not work well on these two groups of
    packages, same as --warn-on-others. See #682744.
    
    Signed-off-by: Andreas Beckmann <debian at abeckmann.de>

diff --git a/piuparts.py b/piuparts.py
index 74c8d55..78d8292 100644
--- a/piuparts.py
+++ b/piuparts.py
@@ -2084,7 +2084,7 @@ def install_purge_test(chroot, chroot_state, package_files, packages):
 
     if settings.install_purge_install:
         file_owners = chroot.get_files_owned_by_packages()
-        chroot.purge_packages(packages)
+        chroot.restore_selections(chroot_state_with_deps["selections"], packages)
         logging.info("Validating chroot after purge")
         if not check_results(chroot, chroot_state_with_deps, file_owners, deps_info=deps_info):
             return False

-- 
piuparts git repository