jclouds / legacy-jclouds-cli

https://jclouds.apache.org
Apache License 2.0
17 stars 5 forks source link

Adding a note about required parameter order #25

Closed demobox closed 11 years ago

demobox commented 11 years ago

See this comment on issue 24

demobox commented 11 years ago

@andrewgaul Do you think this note (in the README) would have helped?

gaul commented 11 years ago

Looks good to me, although most users do not read documentation so we should improve the error messages/loosen argument parsing restrictions as well. I will take a crack at this eventually if someone else does not beat me to it.

@shrinandj ran into confusing argument parsing earlier today as well

demobox commented 11 years ago

although most users do not read documentation

Agreed. What does the "usage" help message look like? Perhaps a better place for this?

iocanel commented 11 years ago

Most of the confusion comes from the fact that we are using the "category" / "action" combination, which seems like an argument and after that we are passing options and finally real arguments.

I am feeling that we should provide a script per command and not use category/action at all. Something like:

./bin/jclouds-node-list --identity ????? --credential ????

instead of

./bin/jclouds node list --identity ????? --credential ????

jclouds commented 11 years ago

I don't like that idea. Git like cli is preferred and further reinforced by the new aws cli.