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

Create manifest installers and VS manifests #56308

Merged
merged 3 commits into from
Jul 27, 2021

Conversation

directhex
Copy link
Contributor

This is a copy of dotnet/emsdk#50

@directhex directhex added the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Jul 26, 2021
@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@steveisok steveisok requested a review from lewing July 26, 2021 15:51
@directhex directhex removed the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Jul 26, 2021
@directhex directhex marked this pull request as ready for review July 26, 2021 21:49
@directhex
Copy link
Contributor Author

The right things are happening in the private build https://dev.azure.com/dnceng/internal/_build/results?buildId=1259362&view=results

@steveisok
Copy link
Member

/backport to release/6.0-preview7

@github-actions
Copy link
Contributor

Started backporting to release/6.0-preview7: https://github.com/dotnet/runtime/actions/runs/1069220553

@github-actions
Copy link
Contributor

@steveisok backporting to release/6.0-preview7 failed, the patch most likely resulted in conflicts:

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

Applying: Create manifest installers and VS manifests
Using index info to reconstruct a base tree...
M	eng/Version.Details.xml
M	eng/Versions.props
M	src/workloads/workloads.csproj
Falling back to patching base and 3-way merge...
Auto-merging src/workloads/workloads.csproj
Auto-merging eng/Versions.props
CONFLICT (content): Merge conflict in eng/Versions.props
Auto-merging eng/Version.Details.xml
CONFLICT (content): Merge conflict in eng/Version.Details.xml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Create manifest installers and VS manifests
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!

@steveisok steveisok merged commit 62503dd into dotnet:main Jul 27, 2021
steveisok pushed a commit to steveisok/runtime that referenced this pull request Jul 27, 2021
mmitche pushed a commit that referenced this pull request Jul 27, 2021
Co-authored-by: Jo Shields <directhex@apebox.org>
@ghost ghost locked as resolved and limited conversation to collaborators Aug 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants