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

Add support for unquoted command strings after -- #16

Closed
Miserlou opened this issue Jun 27, 2018 · 3 comments
Closed

Add support for unquoted command strings after -- #16

Miserlou opened this issue Jun 27, 2018 · 3 comments

Comments

@Miserlou
Copy link
Owner

Miserlou commented Jun 27, 2018

via this comment:

Lovely idea. I have a small amount of "issue" with the way you're doing the command line though. It's more typical to see commandlines of the form:

loop --every 15s --until-success --num 5 ./do_thing.sh

i.e. where you give the command "last" and where if you want to give that command arguments, you use -- to separate:

loop --until-success -- ping -c5 myserver

As a long-time CLI user, having to put the command, and its arguments, first just feels wrong.

Well done on your project though, and +1 for using clap :-)

@mlindner
Copy link

+1 This is the normal way command line apps work.

@Miserlou
Copy link
Owner Author

Miserlou commented Oct 1, 2018

Fixed in 0.4.0.

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

No branches or pull requests

2 participants