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

Remove Version Adjacent Older SDKs from Tracking List when Newer SDK Version in Same Feature Band / Major Minor is Installed #2172

Open
NicoleWang001 opened this issue Mar 12, 2025 · 3 comments
Labels
enhancement New feature or request good first issue Good for newcomers P3 ux

Comments

@NicoleWang001
Copy link
Member

Describe the bug.

The older .NET SDK info is still shown in the Uninstall .NET list after installing the new version .NET SDK.

Please attach log file(s) here if you have any.

logs.zip

Reproduction Steps

  1. On windows, run the sample command ‘Install .NET SDK Globally via .NET Install Tool (Former Runtime Extension)‘. or the command: .NET Install Tool - Install the .NET SDK System-Wide.
  2. Enter a version like 8.0.406 to install .NET SDK 8.0.406
  3. Upgrading to a new version like 8.0.407 with the same command
  4. run the ‘.NET Install Tool: Uninstall .NET’ command to observe the installed SDKs

Expected Behavior

Because the newer SDK with replaced the older on Windows, the older SDK info should not be shown in Uninstall .NET list.

Exceptions & Errors (if any)

The older .NET SDK info is still shown in the Uninstall .NET list after installing the new version .NET SDK.
Image

.NET Install Tool Version

v2.2.9-b

Your operating system? Please provide more detail, such as the version and or distro above.

Windows

VS Code Version

1.98.0

@NicoleWang001
Copy link
Member Author

On Linux, it shows updating info in output if performing SDK upgrade. @nagilson Could you help to confirm whether has the plan on Windows?
Here is the behavior on Ubuntu 24.04

Image

@nagilson
Copy link
Member

nagilson commented Mar 12, 2025 via email

@nagilson nagilson changed the title [NETE2ESDK][Windows] The older .NET SDK info should not be shown in the Uninstall .NET list after updating to new. Remove Version Adjacent Older SDKs from Tracking List when Newer SDK Version in Same Feature Band / Major Minor is Installed Mar 12, 2025
@nagilson nagilson added P3 and removed untriaged labels Mar 12, 2025
@nagilson
Copy link
Member

nagilson commented Mar 12, 2025

Thanks for reporting.

The list displayed in the UI is based on the InstallTrackerSingleton state which is only managed by the extension, so it is unaware of uninstallations that take place from external sources like another installer. However, if someone tried to uninstall this via the UI, it would realize it was uninstalled externally and remove it. The same is true if it was asked to install again and it was validating that it was still there. (Note: If thats not how it behaves that is very bad)

However, if we are in charge of the install, upon success we could untrack the versions in the same major minor feature band sdks. This is an annoying behavior but does not break anything so lower priority to fix and not blocking as it has been present in all versions.

The correct behavior is the Linux one above not the top portion.

@nagilson nagilson added ux enhancement New feature or request good first issue Good for newcomers labels Mar 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers P3 ux
Projects
None yet
Development

No branches or pull requests

2 participants