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

[CrossGen] Condition MibcArgs for source-build #54225

Merged
merged 1 commit into from
Jun 18, 2021

Conversation

MichaelSimons
Copy link
Member

This integrates this source-build patch.

This is reverting this change made in #50536. For source builds, there is no optimization data therefore the MibcArgs need to be excluded to avoid build errors.

Related to dotnet/source-build#2052

@ghost
Copy link

ghost commented Jun 15, 2021

Tagging subscribers to this area: @hoyosjs
See info in area-owners.md if you want to be subscribed.

Issue Details

This integrates this source-build patch.

This is reverting this change made in #50536. For source builds, there is no optimization data therefore the MibcArgs need to be excluded to avoid build errors.

Related to dotnet/source-build#2052

Author: MichaelSimons
Assignees: -
Labels:

area-Infrastructure-coreclr

Milestone: -

@MichaelSimons
Copy link
Member Author

@trylek, could you take a look at these changes? I see David is out for a few days and I noticed you reviewed the original change that I am proposing reverting a small part.

Copy link
Member

@trylek trylek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you!

@MichaelSimons MichaelSimons merged commit 8d0c263 into dotnet:main Jun 18, 2021
@MichaelSimons MichaelSimons deleted the ArPow-PatchRemoval-35 branch June 18, 2021 13:26
@ghost ghost locked as resolved and limited conversation to collaborators Jul 18, 2021
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.

3 participants