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

Allow nesting of '--' #1162

Merged
merged 1 commit into from
Feb 4, 2018
Merged

Allow nesting of '--' #1162

merged 1 commit into from
Feb 4, 2018

Conversation

da-x
Copy link

@da-x da-x commented Feb 3, 2018

Suppose there is a program that executes another program, then it is convenient to take the arguments of the subprogram after --, as illustrated in the 22_stop_parsing_with_--.rs example. However, if the subprogram itself wants to receive --, then clap-rs should allow it.

Fixes #1161.


This change is Reviewable

Suppose there is a program that executes another program, then it is
convenient to take the arguments of the subprogram after `--`, as
illustrated in the 22_stop_parsing_with_--.rs example. However, if
the subprogram itself wants to receive `--`, then clap-rs should
allow it.

Fixes clap-rs#1161.
@mention-bot
Copy link

@da-x, thanks for your PR! By analyzing the history of the files in this pull request, we identified @kbknapp to be a potential reviewer.

@kbknapp
Copy link
Member

kbknapp commented Feb 3, 2018

Wow, fast! Thanks for going ahead and knocking this out! Once the tests pass I'll merge. There a few other minor issues I'll knock out shortly then put out v2.29.3

👍

@kbknapp kbknapp merged commit 8ff685e into clap-rs:master Feb 4, 2018
willmurphyscode added a commit to willmurphyscode/clap-rs that referenced this pull request Mar 1, 2018
Copy the fix from clap-rs#1162 into
the v3 parser, and add a test to protect against regressions in the
expected behavior of example 22.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants