`dotnet tool run incrementalist`: "Error: Option 'c, config' is defined multiple times."

[Aaronontheweb]

Reproduction:

dotnet tool run incrementalist --config -r -- build -c Release

Produces the following error:

ERROR(S):
  Option 'c, config' is defined multiple times.

Looks like we need to add some integration test support to make sure there can't be a conflict between incrementalist commands and the dotnet commands

[Aaronontheweb]

Interesting - this doesn't happen in our integration tests (dotnet run Incrementalist.Cmd) nor does it happen in our unit tests, but it does happen when we call dotnet tool incrementalist [args] - so this makes me wonder if the way arguments are escaped is different under those scenarios.

[Aaronontheweb]

So it looks like this is a bit of a nothingburger, despite all of the heavy lifting I did to reproduce this in #379

There are two ways around this issue - and we'll update the README.md accordingly:

Use just dotnet incrementalist

dotnet incrementalist --config -r -- build -c Release

Works fine with local tools.

Use -- prior to passing in the local tool arguments

dotnet tool run incrementalist -- --config -r -- build -c Release

Also works!

Resolution

I think we need to check if command line argument parsing returned a non-zero result and then log a warning with a link back to this issue.