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

Refresh PATH after Windows installation #676

Closed
garrytrinder opened this issue Apr 25, 2024 · 3 comments
Closed

Refresh PATH after Windows installation #676

garrytrinder opened this issue Apr 25, 2024 · 3 comments
Labels
enhancement New feature or request needs peer review Issue needs review from other team members

Comments

@garrytrinder
Copy link
Contributor

winget doesn't support refreshing the current session when updating PATH after installation, so when Microsoft.DevProxy package is installed, users must start a new session to use the devproxy alias, if they try to run proxy an error will be thrown that devproxy cannot be found, this is also the same for Microsoft.DevProxy.Beta packages and devproxy-beta alias.

We should investigate into ways that we can handle this for the user so they can use proxy immediately after install.

@garrytrinder garrytrinder added the enhancement New feature or request label Apr 25, 2024
@waldekmastykarz
Copy link
Collaborator

It seems like a known issue with winget that many folks struggle with. There are some workarounds. Since it's not specific to us, I suggest that we document it but don't try to fix it ourselves and rather align with winget.

@waldekmastykarz waldekmastykarz added needs peer review Issue needs review from other team members help wanted We'd appreciate your help needs spec Issue needs specification and removed needs peer review Issue needs review from other team members help wanted We'd appreciate your help labels Apr 25, 2024
@waldekmastykarz
Copy link
Collaborator

After exploring a couple of options, we haven't found a way that's working and automatically registers the new path with the current terminal session. As such, I suggest we document the current behavior and close this issue.

@waldekmastykarz waldekmastykarz added needs peer review Issue needs review from other team members and removed needs spec Issue needs specification needs peer review Issue needs review from other team members labels Apr 29, 2024
@garrytrinder
Copy link
Contributor Author

A remark has been added to docs.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs peer review Issue needs review from other team members
Projects
None yet
Development

No branches or pull requests

2 participants