Skip to content

[release/8.0] [NativeAOT] Fix --make-repro-path ILC option - #90989

Merged
carlossanlop merged 2 commits into
release/8.0from
backport/pr-90935-to-release/8.0
Aug 23, 2023
Merged

[release/8.0] [NativeAOT] Fix --make-repro-path ILC option#90989
carlossanlop merged 2 commits into
release/8.0from
backport/pr-90935-to-release/8.0

Conversation

@github-actions

@github-actionsgithub-actionsBot commented Aug 23, 2023

Copy link
Copy Markdown
Contributor

Backport of #90935 to release/8.0

/cc @jkotas@ivanpovazan

Customer Impact

--make-repro-path command line option got broken by upgrading System.CommandLine package. It prevents customers from sending us self-contained repros for native AOT compiler issues.

Testing

Manual test. We will consider adding automated test to SDK.

Risk

IMPORTANT: If this backport is for a servicing release, please verify that:

  • The PR target branch is release/X.0-staging, not release/X.0.

  • If the change touches code that ships in a NuGet package, you have added the necessary package authoring and gotten it explicitly reviewed.

@carlossanlop

Copy link
Copy Markdown
Contributor

@jeffschwMSFT do we have your blessing?

@jeffschwMSFTjeffschwMSFT 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.

approved. we can merge once ready

@jeffschwMSFTjeffschwMSFT added the Servicing-approved Approved for servicing release label Aug 23, 2023
@carlossanlop
carlossanlop merged commit 303f1d3 into release/8.0Aug 23, 2023
@carlossanlop
carlossanlop deleted the backport/pr-90935-to-release/8.0 branch August 23, 2023 23:03
@ghostghost locked as resolved and limited conversation to collaborators Sep 23, 2023
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-ReadyToRunServicing-approvedApproved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@carlossanlop@jkotas@jeffschwMSFT@ivanpovazan