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

Windows usage example fails in Windows 10 #72

Closed
Wihmajster opened this issue May 7, 2022 · 1 comment · Fixed by #73
Closed

Windows usage example fails in Windows 10 #72

Wihmajster opened this issue May 7, 2022 · 1 comment · Fixed by #73
Labels
documentation Improvements or additions to documentation

Comments

@Wihmajster
Copy link
Contributor

Example usage for windows works neither in PowerShell nor in command prompt (tested in Windows 10 sandbox).

curl -LJO "https://github.com/aminya/setup-cpp/releases/download/v0.14.0/setup_cpp_windows.exe"

That works in command prompt. However, it fails in PowerShell. Curl is an alias for Invoke-WebRequest in PowerShell. Replacing curl with curl.exe should work:

curl.exe -LJO "https://github.com/aminya/setup-cpp/releases/download/v0.14.0/setup_cpp_windows.exe"

./setup_cpp_windows --compiler llvm --cmake true --ninja true --ccache true --vcpkg true

That works in PowerShell but fails in the command prompt. Replacing Forward Slash with Backslash should work:

.\setup_cpp_windows --compiler llvm --cmake true --ninja true --ccache true --vcpkg true

Moreover, RefreshEnv.cmd fails If chocolatey has just been installed by setup_cpp. I am not sure what is the best solution in this case.

\ProgramData\chocolatey\bin\RefreshEnv.cmd

The above command works unless you have changed chocolatey default install location. However, both fails when you don't have choco installed, and setup_cpp does not install it either.

@aminya aminya added the documentation Improvements or additions to documentation label May 7, 2022
@aminya
Copy link
Owner

aminya commented May 7, 2022

Thanks for letting me know! Do you want to make a pull request fixing the readme?

For refreshenv, I can add a short note.

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

Successfully merging a pull request may close this issue.

2 participants