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

Pass RID to msbuild task #103508

Merged
merged 2 commits into from
Jun 25, 2024
Merged

Pass RID to msbuild task #103508

merged 2 commits into from
Jun 25, 2024

Conversation

am11
Copy link
Member

@am11 am11 commented Jun 15, 2024

First commit is reverting #103497, second commit has the fix for official build.

@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Jun 15, 2024
@am11 am11 added area-Infrastructure and removed area-Infrastructure-libraries community-contribution Indicates that the PR has been added by a community member labels Jun 15, 2024
@am11
Copy link
Member Author

am11 commented Jun 15, 2024

@MichalStrehovsky, I was able to repro it locally (with command from #103469 (comment)).

@MichalStrehovsky
Copy link
Member

Thanks! I'll test this out by doing an official build from a branch once the infra is stable enough, but for now marking no merge.

@jkoritzinsky could you please have a look at the ddf4666 commit? The rest was already reviewed and is a revert of a revert.

@MichalStrehovsky MichalStrehovsky added the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Jun 17, 2024
@am11
Copy link
Member Author

am11 commented Jun 24, 2024

runtime builds are now flowing normally through the SDK. No rush, but would be good to avoid conflicts if we can, as testing it requires spending time on multiple VMs / containers (and at some point I'd like to restart my computer 😅).

@MichalStrehovsky
Copy link
Member

/ba-g binaryformatter CI issues were fixed in the meantime and is unrelated

@MichalStrehovsky MichalStrehovsky merged commit 4e1ad9c into dotnet:main Jun 25, 2024
154 of 156 checks passed
@MichalStrehovsky MichalStrehovsky removed the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Jun 25, 2024
@MichalStrehovsky
Copy link
Member

Thank you! I'll watch the official builds...

@am11 am11 deleted the patch-1 branch June 25, 2024 06:16
@am11
Copy link
Member Author

am11 commented Jun 25, 2024

Picked up in dotnet/sdk#41793. 🙏

MichalStrehovsky added a commit to MichalStrehovsky/runtime that referenced this pull request Jun 27, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Jul 26, 2024
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.

4 participants