Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add the MSBuild 17.3.4 prebuilt to the reference packages #952

Merged
merged 2 commits into from
May 7, 2024

Conversation

marcpopMSFT
Copy link
Member

Trying to fix the source build here: dotnet/roslyn#73283

Did I run the tooling correctly?

Co-authored-by: Matt Thalman <mthalman@microsoft.com>
@MichaelSimons MichaelSimons requested a review from mthalman May 6, 2024 20:55
@mthalman mthalman merged commit 2ae8497 into dotnet:release/8.0 May 7, 2024
6 checks passed
@marcpopMSFT
Copy link
Member Author

/backport to main

Copy link
Contributor

Copy link
Contributor

@marcpopMSFT backporting to main failed, the patch most likely resulted in conflicts:

$ git am --3way --ignore-whitespace --keep-non-patch changes.patch

Applying: Add the MSBuild 17.3.4 prebuilt to the reference packages
Using index info to reconstruct a base tree...
M	eng/Build.props
Falling back to patching base and 3-way merge...
Auto-merging eng/Build.props
CONFLICT (content): Merge conflict in eng/Build.props
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Add the MSBuild 17.3.4 prebuilt to the reference packages
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
Error: The process '/usr/bin/git' failed with exit code 128

Please backport manually!

Copy link
Contributor

@marcpopMSFT an error occurred while backporting to main, please check the run log for details!

Error: git am failed, most likely due to a merge conflict.

@MichaelSimons MichaelSimons mentioned this pull request May 16, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants