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

Plugins: handle VS C++ Windows compiler issue #1794

Open
dcdenu4 opened this issue Feb 19, 2025 · 1 comment · May be fixed by #1795
Open

Plugins: handle VS C++ Windows compiler issue #1794

dcdenu4 opened this issue Feb 19, 2025 · 1 comment · May be fixed by #1795

Comments

@dcdenu4
Copy link
Member

dcdenu4 commented Feb 19, 2025

In later versions of micromamba for Windows, it's expected the install system has the VS C++ compiler on path. There's a few options for us to consider here:

  • Can we rollback Micromamba to before there was need for a Windows compiler?
  • Including the installer as part of the Windows build?
  • Install it for Windows when trying to install a plugin?

Some context from James on our previous history with VCRedist

RE: VCRedist, this is not surprising, but it is frustrating. We used to have to do this for InVEST as a part of our NSIS installed (not sure if it was technically legal for us to do so, though), so maybe we will just need to figure out vcredist installation as a part of the micromamba install?
Related discussion on github, along with a scripted solution

I was able to use the following script silently, except that it prompted a pop up of whether I wanted to allow this program to make changes to my comptuer.

Invoke-WebRequest -URI "https://aka.ms/vs/17/release/vc_redist.x64.exe" -OutFile "$env:Temp\vc_redist.x64.exe"; Start-Process "$env:Temp\vc_redist.x64.exe" -ArgumentList "/quiet /norestart" -Wait; Remove-Item "$env:Temp\vc_redist.x64.exe"
@emlys
Copy link
Member

emlys commented Feb 19, 2025

Confirmed we cannot use micromamba<1.5.2 to avoid the issue. Earlier versions have issues with path spaces.

@emlys emlys linked a pull request Feb 19, 2025 that will close this issue
3 tasks
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 a pull request may close this issue.

2 participants