"swiftly uninstall all" command to remove all installed toolchains

[markdols]

Hello, I maintain the swiftly-bin AUR package, and I appreciate all of the work that has gone into this utility, as it has been very useful to use (and definitely more flexible than just using the swift-bin AUR package!)

The .install script I wrote for said package runs swiftly uninstall latest in a loop until the ~/.local/share/swiftly/toolchains folder is empty. This is prone to an infinite loop if a non-toolchain file or folder is placed in that directory.

To alleviate this, I propose some small changes to Uninstall.swift that allow for a one-command uninstallation of all installed toolchains with swiftly uninstall all.

I am new to contributing, so please let me know if there are any issues with this proposal.

Thank you!

[patrickfreed]

Hey, thanks for the PR, and also thanks for maintaining that package for swiftly!

I've gone ahead and made two slight changes to the PR:

• Ensured the in-use toolchain is uninstalled last to avoid swiftly spamming the output with "x toolchain is now in use" each time the in-use toolchain is uninstalled during an "uninstall all" command
• Added a unit test

Related question: is this functionality used for uninstalling swiftly? Or is it for cleaning up any prior swiftly installation before performing a fresh install? If it's the latter, note that this is already handled by the swiftly-install.sh script, if it's possible to use that as part of this installation process. If it's the former, then #84 will be useful for you in the future.

[patrickfreed]

@swift-server-bot add to allowlist

[markdols]

@patrickfreed This would be used for uninstalling swiftly. The package manager would be responsible for removing the binary; the .install script uninstalls all toolchains before the binary is uninstalled, which is where a swiftly uninstall all command would be useful.

[patrickfreed]

Makes sense, just merged it in now. Thanks for the contribution!