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] Use the correct runtime pack version when the workload is not installed #75028

Merged
merged 6 commits into from
Sep 6, 2022

Conversation

radical
Copy link
Member

@radical radical commented Sep 2, 2022

WorkloadManifest.targets had the runtime pack version update conditioned on the workload being installed, which meant that the original version from the sdk was used if no workload was installed, ignoring the version from the manifest.

  • And update tests for this.

@radical radical added arch-wasm WebAssembly architecture area-Build-mono labels Sep 2, 2022
@ghost ghost assigned radical Sep 2, 2022
@ghost
Copy link

ghost commented Sep 2, 2022

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

Issue Details

WorkloadManifest.targets had the runtime pack version update conditioned on the workload being installed, which meant that the original version from the sdk was used if no workload was installed, ignoring the version from the manifest.

Author: radical
Assignees: -
Labels:

arch-wasm, area-Build-mono

Milestone: -

.. if no workload is installed. This ensures that the correct runtime
pack version (intended by the installed manifest) is used even if the
workload is not installed.
.. runtime pack version is being used.

To use the locally built runtime pack without the workload, we need the
nuget, so we use that from the artifacts.
@radical radical marked this pull request as ready for review September 3, 2022 02:10
@radical radical requested a review from lewing as a code owner September 3, 2022 02:10
@radical
Copy link
Member Author

radical commented Sep 3, 2022

/azp run runtime-wasm-perf

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@radical radical changed the title [wasm] Use the correct runtime pack version when the workload is not installed [release/7.0][wasm] Use the correct runtime pack version when the workload is not installed Sep 6, 2022
@carlossanlop
Copy link
Member

Ping @marek-safar we need an approval, please.

@carlossanlop
Copy link
Member

CI is green, approved and signed off. Ready to merge. :shipit:

@carlossanlop carlossanlop merged commit 9dda765 into dotnet:release/7.0 Sep 6, 2022
@radical radical deleted the fix-rpack-ver branch September 7, 2022 00:25
@ghost ghost locked as resolved and limited conversation to collaborators Oct 7, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
arch-wasm WebAssembly architecture area-Build-mono
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants