tito icon indicating copy to clipboard operation
tito copied to clipboard

Published 20 hours ago • verified publisher icon rpm-software-management

Update CLI library

Open ashcrow opened this issue 8 years ago • 5 comments

Wishlist: Modernize CLI library using ArgumentParser, click, etc...

ashcrow avatar Jun 12 '17 23:06 ashcrow

Hmm e.g. argparse cannot be used as long as we want to support EL6. Personally I am ok with dropping EL6 support, but I am not sure about other users. BTW with EOL of EL5 we can likely remove the enforcemen of MD5 hashes during rpm build.

xsuchy avatar Jun 14 '17 21:06 xsuchy

Good points. I'd be ok with dropping el6 as well, just leaving what's there.

dgoodwin avatar Jun 14 '17 21:06 dgoodwin

Enjoying click and it's API in the project we have it in FWIW.

stevekuznetsov avatar Jun 14 '17 22:06 stevekuznetsov

Really like click as well, however IIRC they have some opinions on positional arguments and what parent or sub-command they relate to that might complicate keeping the tito CLI exactly as it is today. If that could be worked around, or we could agree it's safe to alter the CLI slightly then it would be a good option.

dgoodwin avatar Jun 15 '17 11:06 dgoodwin

I'm a fan of click as well. I would think it's safe to alter the CLI if the version is bumped appropriately.

ashcrow avatar Jun 15 '17 13:06 ashcrow