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

Unable to find MSBuild when specifying msbuild-architecture #104

Closed
akunzai opened this issue Dec 13, 2022 · 2 comments
Closed

Unable to find MSBuild when specifying msbuild-architecture #104

akunzai opened this issue Dec 13, 2022 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@akunzai
Copy link

akunzai commented Dec 13, 2022

The setup-msbuild v1.2 Unable to find MSBuild when specifying msbuild-architecture

uses: microsoft/setup-msbuild@v1.2
with:
    msbuild-architecture: x64
Run microsoft/setup-msbuild@v1.2
  with:
    msbuild-architecture: x64
  env:
    DOTNET_ROOT: C:\Program Files\dotnet
C:\ProgramData\Chocolatey\bin\vswhere.exe -products * -requires Microsoft.Component.MSBuild -property installationPath -latest
C:\Program Files\Microsoft Visual Studio\2022\Enterprise
Error: Unable to find MSBuild.

see https://github.com/akunzai/GSS.Authentication.CAS/actions/runs/3679955445/jobs/6228417729

But CI workflow will passing after I removed the msbuild-architecture parameter

uses: microsoft/setup-msbuild@v1.2
Run microsoft/setup-msbuild@v1.2
  with:
    msbuild-architecture: x86
  env:
    DOTNET_ROOT: C:\Program Files\dotnet
C:\ProgramData\Chocolatey\bin\vswhere.exe -products * -requires Microsoft.Component.MSBuild -property installationPath -latest
C:\Program Files\Microsoft Visual Studio\2022\Enterprise

see https://github.com/akunzai/GSS.Authentication.CAS/actions/runs/3681629139/jobs/6228509271

@timheuer timheuer self-assigned this Dec 13, 2022
@timheuer timheuer added the bug Something isn't working label Dec 13, 2022
@timheuer
Copy link
Member

Taking a look now -- please revert to @v1.1 tag in the meantime.

@timheuer
Copy link
Member

Fixed in v1.3 and updated v1 tag. Apologies for the mistake @akunzai

kcgen added a commit to dosbox-staging/dosbox-staging that referenced this issue Dec 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants