quicklisp-client icon indicating copy to clipboard operation
quicklisp-client copied to clipboard

Re: Issue #147

Open obicons opened this issue 7 years ago • 1 comments

  • UNINSTALL can now accept a list of systems to remove
  • UNINSTALL can now optionally and safely remove a system's dependencies

obicons avatar Apr 30 '18 13:04 obicons

Thanks for the quick review!

I went ahead and did some refactoring to try and address your comments, specifically by removing the call and definition of FILTER and factored out the behavior of UNINSTALL-SYSTEM-DEPENDENCIES. I also generally improved the behavior.

To test this, I installed the modified client in a container. Then, I loaded two systems that share several dependencies (caveman and drakma) and verified that uninstalling one left the dependencies of the other intact.

Let me know if you have any other comments/concerns 👍

obicons avatar May 02 '18 15:05 obicons