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 the net7.0 build when only net6.0 workloads are installed #81390

Merged
merged 1 commit into from
Feb 9, 2023

Conversation

lewing
Copy link
Member

@lewing lewing commented Jan 30, 2023

Fixes an issue found by internal testing when building using a net7.0.xx sdk with only the wasm-tools-net6 workload installed.

fix for https://github.com/aspnet/AspNetCore-ManualTests/issues/1722#issuecomment-1407320124

Customer Impact

Fixes a scenario where the customer has installed the wasm-tools-net6 but has not installed wasm-tools on a net7 sdk and gets an error requiring the net7 workload when it is not strictly required

Testing

Manual, this is difficult/impossible to test in CI on release branches with stable branding due to downlevel version mismatch.

Risk

Low.

…emsdk turns that into requiring the workload
@lewing lewing force-pushed the net6-triggering-net7 branch from 3524d24 to 88ae49e Compare January 30, 2023 23:04
@lewing lewing changed the title Fix the case where net6 AutoImports sets WasmNativeWorkload true and … [release/7.0][wasm] Fix the case where net6 AutoImports sets WasmNativeWorkload true and … Jan 30, 2023
@lewing lewing marked this pull request as ready for review January 31, 2023 03:03
@lewing lewing requested a review from radical as a code owner January 31, 2023 03:03
@lewing lewing changed the title [release/7.0][wasm] Fix the case where net6 AutoImports sets WasmNativeWorkload true and … [release/7.0][wasm] Fix the net7.0 build when only net6.0 workloads are installed Jan 31, 2023
@carlossanlop
Copy link
Member

carlossanlop commented Feb 9, 2023

Tell mode @lewing ?

@lewing
Copy link
Member Author

lewing commented Feb 9, 2023

Tell mode @lewing ?

yes

@carlossanlop
Copy link
Member

Great. CI failure is #81544. Merging now.

@carlossanlop carlossanlop merged commit ca44b0d into dotnet:release/7.0 Feb 9, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Mar 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants