Switch R2RDump over to the new command line parser (dotnet/coreclr#27440)
authorTomáš Rylek <trylek@microsoft.com>
Fri, 25 Oct 2019 22:45:19 +0000 (15:45 -0700)
committerGitHub <noreply@github.com>
Fri, 25 Oct 2019 22:45:19 +0000 (15:45 -0700)
commiteaf74a97f3502ec0c208da92d9d597e2a5045d83
tree14ab59198aa5882d39ec30be5df8466dfe73255c
parent0c8b7b5b171b4c45129fafe6aa66e24f5ae56930
Switch R2RDump over to the new command line parser (dotnet/coreclr#27440)

This change modifies R2RDump to use the new
System.CommandLine.Experimental module instead of the obsoleted
System.CommandLine parser we were using before. Apart from general
hygiene w.r.t. using up-to-date components one practical reason
for this switch-over is that the old parser is only available on
the unreliable myget.org nuget feed we plan to drop as part of
our planned repo consolidation.

During debugging of the change I noticed a bug in the implementation
of the "--verbose" option - it shouldn't be setting EntryPoints,
otherwise we actually end up with a super-terse output including
just the entry points. I have anecdotally verified that, after fixing
this in both the "new" and "old" R2RDump both produce identical output.

Thanks

Tomas

Commit migrated from https://github.com/dotnet/coreclr/commit/a0d03272dbe691005d74edff73a36c012d38fa1b
src/coreclr/src/tools/r2rdump/CommandLineOptions.cs [new file with mode: 0644]
src/coreclr/src/tools/r2rdump/R2RDump.cs
src/coreclr/src/tools/r2rdump/R2RDump.csproj