Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

uninstall -all shouldn't attempt to uninstall IPM itself. #707

Open
isc-shuliu opened this issue Jan 16, 2025 · 1 comment
Open

uninstall -all shouldn't attempt to uninstall IPM itself. #707

isc-shuliu opened this issue Jan 16, 2025 · 1 comment
Labels
bug Something isn't working prio: medium
Milestone

Comments

@isc-shuliu
Copy link
Collaborator

Currently, running uninstall -all will attempt to uninstall all packages, including IPM itself.

zpm:USER>uninstall -a

Uninstalling zpm
[USER|ZPM]	Clean START
[USER|ZPM]	Unconfigure START
[USER|ZPM]	Unconfigure SUCCESS
[ZPM]	Clean FAILURE
Error uninstalling zpm: ERROR #5002: ObjectScript error: <CLASS DOES NOT EXIST>%Clean+141^%IPM.Lifecycle.Base.1 *%IPM.StudioDocument.Module

<CLASS DOES NOT EXIST>DisplayError+2^%IPM.Main.1 *%IPM.Repo.UniversalSettings
USER 6d1>

We should explicitly exclude IPM from the uninstall list and there should be a separate implementation for uninstalling IPM itself.

@isc-shuliu isc-shuliu added bug Something isn't working prio: medium labels Jan 16, 2025
@isc-shuliu
Copy link
Collaborator Author

Also, unless this is already implemented, we need to figure out the uninstallation order based on the dependency graph (we might need to do a topological sort first)

@isc-shuliu isc-shuliu added this to the Backlog milestone Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working prio: medium
Projects
None yet
Development

No branches or pull requests

1 participant