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

Update to latest System.CommandLine package #66929

Merged
merged 1 commit into from
Mar 21, 2022

Conversation

jkotas
Copy link
Member

@jkotas jkotas commented Mar 21, 2022

No description provided.

@jkotas
Copy link
Member Author

jkotas commented Mar 21, 2022

/azp run runtime-coreclr crossgen2 outerloop

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jkotas jkotas marked this pull request as ready for review March 21, 2022 15:35
@jkotas jkotas requested review from trylek and agocke March 21, 2022 15:35
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!

@jkotas jkotas merged commit ca82541 into dotnet:main Mar 21, 2022
@jkotas jkotas deleted the System.CommandLine branch March 21, 2022 22:36
@am11
Copy link
Member

am11 commented Mar 28, 2022

@jkotas, now the performance has improved, should we revert 4731fdb?

@trylek
Copy link
Member

trylek commented Mar 28, 2022

Considering we've been churning this bit for the last two and a half years, should we at least wait until System.CommandLine proceeds from the "beta" phase to some real shipping version?

@jkotas
Copy link
Member Author

jkotas commented Mar 28, 2022

The System.CommandLine APIs have changed since 4731fdb so it would not be a simple revert.

It would nice to give it a try and measure the performance, give feedback to System.CommandLine team if it is still slowing down crossgen2 too much.

radekdoulik pushed a commit to radekdoulik/runtime that referenced this pull request Mar 30, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Apr 28, 2022
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