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

Long alias parsing with boost 1.67 #2776

Closed

Conversation

guilhermelawless
Copy link
Contributor

Support for multiple long names was only introduced in 1.68, but the extra_parser can be used which calls a function on each element (example). Each element is a command, values are not affected.

This simply translates --debug_validate_blocks into validate_blocks, everything else should be untouched.

Support for multiple long names was only introduced in [1.68](https://www.boost.org/users/history/version_1_68_0.html), but the `extra_parser` can be used which calls a function on each element ([example](https://www.boost.org/doc/libs/1_67_0/libs/program_options/example/custom_syntax.cpp)).

This simply translates `--debug_validate_blocks` into `validate_blocks`, everything else should be untouched.
@guilhermelawless guilhermelawless added bug cli Changes related to the Command Line Interface labels May 18, 2020
@guilhermelawless guilhermelawless added this to the V21.0 milestone May 18, 2020
@guilhermelawless guilhermelawless self-assigned this May 18, 2020
Copy link
Contributor

@wezrule wezrule left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested a few CLI commands and options, worked fine.

@argakiig argakiig mentioned this pull request May 19, 2020
@guilhermelawless guilhermelawless deleted the cli/alias-parser branch May 22, 2020 16:27
@zhyatt zhyatt removed this from the V21.0 milestone Jun 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug cli Changes related to the Command Line Interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants