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

Uninstalling and updating an optional workload #14955

Closed
8 tasks done
marcpopMSFT opened this issue Dec 12, 2020 · 4 comments
Closed
8 tasks done

Uninstalling and updating an optional workload #14955

marcpopMSFT opened this issue Dec 12, 2020 · 4 comments
Assignees
Milestone

Comments

@marcpopMSFT
Copy link
Member

marcpopMSFT commented Dec 12, 2020

.NET SDK managed workload update:

.NET SDK managed workload uninstall:


  • Need to have an option to check all (workload, SDk, runtime) at once. We need an explicit command for sure.
  • Check the last manifest from nuget manifest feed for workload
  • Check release.json (convert it to nuget package?) for SDK and runtime (dotnet sdk check)
  • Garbage collection for the packs (given SDKs and what’s one disk and manifests as input. There could be multiple SDKs)
@sfoslund
Copy link
Member

sfoslund commented Jun 3, 2021

Removing "Implicit check for update (check periodically or tie to a command. Check for update of project and check for SDK should be able to be distinguished by the user) below cutline." as it's tracked by #17394

@sfoslund
Copy link
Member

sfoslund commented Jun 3, 2021

Check update with explicit command. Same logic for both non admin and VS/standalone installer since there is no permission issue. (add to dotnet --info too?)

@wli3 does the list PR you have out now cover this? Or does that only advertise when the print urls option is set?

@wli3
Copy link

wli3 commented Jun 4, 2021

@sfoslund it is more for #17394 if we decide not to a background check, but an explicit check instead. I will just remove the line here.

@sfoslund
Copy link
Member

sfoslund commented Jun 4, 2021

Now that cross band update has been merged, all of these items are finished, closing.

@sfoslund sfoslund closed this as completed Jun 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants