Skip to content

[CrossGen] Condition MibcArgs for source-build#54225

Merged
MichaelSimons merged 1 commit into
dotnet:mainfrom
MichaelSimons:ArPow-PatchRemoval-35
Jun 18, 2021
Merged

[CrossGen] Condition MibcArgs for source-build#54225
MichaelSimons merged 1 commit into
dotnet:mainfrom
MichaelSimons:ArPow-PatchRemoval-35

Conversation

@MichaelSimons

Copy link
Copy Markdown
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 ghost added the area-Infrastructure-coreclr Only use for closed issues label Jun 15, 2021
@ghost

ghost commented Jun 15, 2021

Copy link
Copy Markdown

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
Copy Markdown
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.

@MichaelSimons MichaelSimons requested a review from trylek June 17, 2021 21:55

@trylek trylek left a comment

Copy link
Copy Markdown
Member

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.

Labels

area-Infrastructure-coreclr Only use for closed issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants