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

Update riscv-and-xtensa.md #149

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update riscv-and-xtensa.md #149

wants to merge 1 commit into from

Conversation

ryankopf
Copy link

The old statement was incorrect, because if the installer is not run as administrator, it fails to permanently set the environment variables without telling you. I have also submitted a PR to display a clearer error message in espup.

@ryankopf
Copy link
Author

After a bit more investigation, the change is still warranted, but the reason may actually be related to Windows's undocumented caching of registry values. However the problem is easily reproduceable by deleting your environment variables with Regedit, restarting your PC, and running the espup install command.

@SergioGasquez
Copy link
Member

After a bit more investigation, the change is still warranted, but the reason may actually be related to Windows's undocumented caching of registry values. However the problem is easily reproduceable by deleting your environment variables with Regedit, restarting your PC, and running the espup install command.

But did you have to use admin privileges when running the espup command to get the environment automatically picked up?

@ryankopf
Copy link
Author

I am going to have to test again later, but no it seemed like being administrator wasn't a majaor factor, but rather there is some kind of registry caching that doesn't complete unless you completely restart your PC.

@SergioGasquez
Copy link
Member

Hi! Did you had a chance to look into this?

@ryankopf
Copy link
Author

I was unable to determine the final cause, but it seems like there is some kind of undocumented caching of these registry values on Windows (tried on both 10 & 11). So a note about encouraging users to restart, on Windows, if it doesn't work immediately would still be a good idea.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants