Previously, the package_cache was duplicated in import_and_invalidate before being modified. A recent change modified the code to use a passed object as-is. Because the same instance was also passed for packages_to_add, modifications to the cache resulted in changes to the list of packages to add.
The result was that no packages were ever removed from the target repo, effectively making the invalidation pattern inert.
Previously, the
package_cachewas duplicated inimport_and_invalidatebefore being modified. A recent change modified the code to use a passed object as-is. Because the same instance was also passed forpackages_to_add, modifications to the cache resulted in changes to the list of packages to add.The result was that no packages were ever removed from the target repo, effectively making the invalidation pattern inert.
Regression caused by #921