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

[release/7.0][wasm] Fix reference to Microsoft.Build.NoTargets #75472

Merged
merged 1 commit into from
Sep 13, 2022

Conversation

radical
Copy link
Member

@radical radical commented Sep 12, 2022

Backport of the fix from #75397

Customer impact

None. This is a tests only change.

Testing

CI wasm runtimetests run, since this is a CI/tests only change.

Risk

None, as long as the tests pass

.. in `WasmTestRunner.proj` used for runtime tests.

This project requires an explicit SDK version number because it is used
on Helix, and `global.json` is not available there. The version matches what we
have in `global.json`.

```
  Starting:    Loader.classloader.XUnitWrapper (parallel test collections = on, max threads = 2)
    Loader/classloader/DefaultInterfaceMethods/genericmethods/genericmethods/genericmethods.sh [FAIL]

...

      /datadisks/disk1/work/C25609EE/p/wasm-test-runner/WasmTestRunner.proj : error : Failed to resolve SDK 'Microsoft.Build.NoTargets/1.0.53'. Package restore was successful but a package with the ID of "Microsoft.Build.NoTargets" was not installed.
      /datadisks/disk1/work/C25609EE/p/wasm-test-runner/WasmTestRunner.proj : error MSB4236: The SDK 'Microsoft.Build.NoTargets/1.0.53' specified could not be found.
```
@ghost
Copy link

ghost commented Sep 12, 2022

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

Backport of the fix from #75397

Customer impact

None. This is a tests only change.

Testing

CI wasm runtimetests run, since this is a CI/tests only change.

Risk

None, as long as the tests pass

Author: radical
Assignees: -
Labels:

arch-wasm, area-Build-mono, test-failure

Milestone: -

@radical radical added this to the 7.0.0 milestone Sep 12, 2022
@radical
Copy link
Member Author

radical commented Sep 13, 2022

@carlossanlop ready for merge

@carlossanlop
Copy link
Member

@carlossanlop ready for merge

Let's do this. :shipit:

@carlossanlop carlossanlop merged commit a7da979 into dotnet:release/7.0 Sep 13, 2022
@radical radical deleted the fix-no-targets-7 branch September 13, 2022 17:11
@ghost ghost locked as resolved and limited conversation to collaborators Oct 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants