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

option ordering on config help is wrong #36

Closed
NeilW opened this issue Sep 30, 2013 · 2 comments
Closed

option ordering on config help is wrong #36

NeilW opened this issue Sep 30, 2013 · 2 comments

Comments

@NeilW
Copy link
Member

NeilW commented Sep 30, 2013

Synopsis is:

SYNOPSIS
brightbox [global options] config [client_list]
brightbox [global options] config [-a arg|--alias arg] [-t arg|--auth-url arg] client_add client-id secret [api_url, auth_url]
brightbox [global options] config client_default alias
brightbox [global options] config client_remove alias
brightbox [global options] config [-a arg|--alias arg] [-p arg|--password arg] user_add email client-id secret [api_url auth_url]

However putting the -a in that order gives:

ERROR: Unknown option -a

The -a and the other options have to be after the client_add or user_add.

@NeilW
Copy link
Member Author

NeilW commented Sep 30, 2013

This seems to be a general problem with the argument ordering.

e.g.
brightbox [global options] images [-a|--show-all] [-l arg|--account arg] [-s arg|--status arg] [-t arg|--type arg] [list] [image-id...]

$ brightbox images -t official list
ERROR: Unknown option -t

@tokengeek
Copy link
Contributor

This appears to have been resolved in GLI davetron5000/gli#162

brightbox help config
NAME
    config - Manages local configuration settings and credentials

SYNOPSIS
    brightbox [global options] config [client_list]
    brightbox [global options] config client_add [-a arg|--alias arg] [-t arg|--auth-url arg] client-id secret [api_url, auth_url]
    brightbox [global options] config client_default alias
    brightbox [global options] config client_remove alias
    brightbox [global options] config user_add [-a arg|--alias arg] [-p arg|--password arg] email client-id secret [api_url auth_url]

COMMANDS
    client_add     - Add new api client details to config
    client_default - Set the default api client in the config
    client_list    - Lists the api clients defined in the config (default)
    client_remove  - Remove api client details from config
    user_add       - Add new user credentials to config

Will update the GLI dependency to GLI v2.9.0 to fix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants