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

Consider merge with rust-argparse #8

Closed
ctjhoa opened this issue Mar 19, 2015 · 2 comments
Closed

Consider merge with rust-argparse #8

ctjhoa opened this issue Mar 19, 2015 · 2 comments

Comments

@ctjhoa
Copy link
Contributor

ctjhoa commented Mar 19, 2015

Hi,

I think this module https://github.com/tailhook/rust-argparse as the same goal as your's.
Please consider a merge between both because I think a lot of module will emerged to handle cli args and that's not necessary a good thing for standardization.

cf : tailhook/rust-argparse#14

@evnm
Copy link
Owner

evnm commented Mar 19, 2015

Thanks for the heads-up. I agree that duplicating functionality (and maintenance) across separate libraries should be avoided wherever possible.

In their current forms, cli.rs and rust-argparse provide the same bells and whistles. argparse by definition covers argument-parsing and the obvious derivatives thereof (e.g. help output). My intention with cli.rs is to provide a collection of functions that are useful when developing CLI programs. Hypothetically, this could include functionality not related to arg-parsing, such as additional exit status helpers or some mechanism to load and parse configuration files (e.g. a dotfile in the user's home directory).

Maybe we should add a disclaimer in the cli.rs README to point this out and link to other similar libraries like argparse. I don't want to go so far as to deprecate cli.rs right now, but I'd like potential consumers to be informed as to alternatives.

@ctjhoa
Copy link
Contributor Author

ctjhoa commented Mar 20, 2015

@evnm Ok I got it. Maybe a solution could be to rely on rust-argparse for arg parsing parts and to evolve cli.rs in a much higher level module which bundle some functionalities like parse conf .... and include arg parse in those functionalities.

@ctjhoa ctjhoa closed this as completed May 29, 2015
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