-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Configure CLI11 to stuff all unknown positionals into the cmdline #4388
Configure CLI11 to stuff all unknown positionals into the cmdline #4388
Conversation
This commit fixes an issue where "wt -d C: wsl -d Alpine" would be parsed as "wt -d C: -d Alpine wsl" and rejected as invalid due to the repeated -d. It also fixes support for the option parsing terminator, --, in all command lines. Fixes #4277.
ae4f786
to
bfa6ade
Compare
The new test passes, but the |
what? no, I would never not run the tests like that O_o
/s
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess if I had to have a complaint, those comments in AppCommandlineArgs.cpp
are pretty W I D E but that's a nit.
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
I learned from the CLI11 folks that we don't actually need the string+vector version |
🎉 Handy links: |
This commit fixes an issue where "wt -d C: wsl -d Alpine" would be
parsed as "wt -d C: -d Alpine wsl" and rejected as invalid due to the
repeated -d. It also fixes support for the option parsing terminator,
--, in all command lines.
Fixes #4277.
References
CLIUtils/CLI11#417
PR Checklist
--startingDirectory
to a WSLwt.exe
command #4277Validation Steps
Tests, manual wailing.