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.

Sign up for GitHub

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

Jump to bottom

Disable Magical Version Handling #514

Closed
mitsuhiko opened this issue May 28, 2016 · 3 comments
Closed

Disable Magical Version Handling #514

mitsuhiko opened this issue May 28, 2016 · 3 comments
Labels
A-parsing Area: Parser's logic and needs it changed somehow. C-enhancement Category: Raise on the bar on expectations

Comments

@mitsuhiko
Copy link
Contributor

Currently you cannot implement --version yourself and what's worse is that you cannot disable the short alias for it (-V), you can only bind it to a different letter.

What i want is to not have version info at all on arguments and implement --version myself as i want to short some license info there as well.
@mitsuhiko
Copy link
Contributor Author

Currently if you disable the version flag with the setting and you add your own --version, clap starts handling it again and brings up it's own version screen.

@kbknapp
Copy link
Member

kbknapp commented May 30, 2016

I agree this would be a good change, I came across something I wanted to do muslef like this as well. I'll post back here as I start getting this implemented. 👍

@kbknapp kbknapp added C-enhancement Category: Raise on the bar on expectations T: new feature A-parsing Area: Parser's logic and needs it changed somehow. labels May 30, 2016
kbknapp added a commit that referenced this issue May 30, 2016
@kbknapp
fix: fixes bug where one can't override version flag
c9f8a3a 
Closes #514
@kbknapp kbknapp mentioned this issue May 30, 2016
Merged
@kbknapp
Copy link
Member

kbknapp commented May 30, 2016

#517 fixes this. Now you'd just need to specify an arg with a long of version which effectively disables the auto-generated one (to include the short).

homu added a commit that referenced this issue May 31, 2016
@homu
Auto merge of #517 - kbknapp:quick-issues, r=kbknapp
ed0fd8a 
Quick issues

Relates to #515 #516 #514
homu added a commit that referenced this issue May 31, 2016
@homu
Auto merge of #517 - kbknapp:quick-issues, r=kbknapp
33412b7 
Quick issues

Relates to #515 #516 #514
homu added a commit that referenced this issue May 31, 2016
@homu
Auto merge of #517 - kbknapp:quick-issues, r=kbknapp
34e608c 
Quick issues

Relates to #515 #516 #514
@homu homu closed this as completed in 90d7d6a May 31, 2016
kbknapp added a commit that referenced this issue Jun 4, 2016
@kbknapp
fix: fixes bug where one can't override version or help flags
8f630c6 
Closes #514
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-parsing Area: Parser's logic and needs it changed somehow. C-enhancement Category: Raise on the bar on expectations
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mitsuhiko @kbknapp