-
Notifications
You must be signed in to change notification settings - Fork 5k
Use the regular build scripts for -vs coreclr.sln #114511
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
Conversation
This fixes an issue with passing the RID in for cDAC and also removes one more direct usage of build-runtime.cmd
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot wasn't able to review any files in this pull request.
Files not reviewed (1)
- eng/build.ps1: Language not supported
Tagging subscribers to this area: @hoyosjs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, as long as you have already verified creation of coreclr.sln
scenario.
Does this also resolve #113708? I've been using an old CoreClr.sln for a while because I can't generate a new one. |
I tested it locally and it worked for me, so I think so. |
This fixes an issue with passing the RID in for cDAC and also removes one more direct usage of build-runtime.cmd