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

Help/USAGE message for hab sup load shows positional args with incorrect order #2221

Open
rhass opened this issue May 4, 2017 · 4 comments
Labels
Focus: CLI Related to the Habitat CLI (core/hab) component Type: Bug Issues that describe broken functionality
Milestone

Comments

@rhass
Copy link
Contributor

rhass commented May 4, 2017

When you run hab sup load --help it tells you:

USAGE:
    hab-sup load [FLAGS] [OPTIONS] <PKG_IDENT>

However if you do something like this: sudo hab sup load --bind database:postgresql.default myorgin/mypkg it throws the following error:

error: The following required arguments were not provided:
    <PKG_IDENT>

The working command is: sudo hab sup load myorigin/mypkg --bind database:postgresql.default

This seems like it should be a simple enough bug for me to fix on my own, but I haven't quite figured out where in the supervisor code these messages are being generated and displayed. This may be a bug in the upstream command line parser as well.

@reset reset added Bug labels May 4, 2017
@mwrock
Copy link
Contributor

mwrock commented May 9, 2017

The usage messages are controlled by the clap crate and not directly fromo the habitat code. My suspision after looking into this briefly is that there may be a bug in clap related to parameters that can take multiple values. I noticed that any of the params other than --bind worked. We should definitely investigate if this has been addressed in the clap crate and if not, file an issue there.

@stale
Copy link

stale bot commented Apr 3, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. We value your input and contribution. Please leave a comment if this issue still affects you.

@stale stale bot added the Stale label Apr 3, 2020
@christophermaier christophermaier added Focus: CLI Related to the Habitat CLI (core/hab) component and removed A-cli labels Jul 24, 2020
@stale stale bot removed the Stale label Jul 24, 2020
@christophermaier christophermaier added Type: Bug Issues that describe broken functionality and removed C-bug labels Jul 24, 2020
@stale
Copy link

stale bot commented Aug 12, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. We value your input and contribution. Please leave a comment if this issue still affects you.

1 similar comment
@stale
Copy link

stale bot commented Aug 12, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. We value your input and contribution. Please leave a comment if this issue still affects you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Focus: CLI Related to the Habitat CLI (core/hab) component Type: Bug Issues that describe broken functionality
Projects
None yet
Development

No branches or pull requests

6 participants