search

arkane-systems / RunInGenie

Quickly run a wsl command from a Windows prompt.
MIT License
33 stars 6 forks source link

Add commandline options and shorthands #6

Open gbraad opened 3 years ago

gbraad commented 3 years ago

Allow the use of:

This is a follow-up to https://github.com/arkane-systems/RunInGenie/pull/5#issuecomment-782603747

cerebrate commented 3 years ago

I've been thinking about this a little in the interim.

It's seeming to me that there are some downsides to allowing arbitrary options, insofar as there's no rule in Linux preventing options, etc. from having filenames starting with - or --, meaning there's a chance and a greater one with every new options that it's going to cause a name collision with something for someone somewhere.

It also requires options whose parameters, if any, are all strictly defined enough that there won't be ambiguity between what's an option and what's a command.

I'm thinking it may be better to make one breaking change now to head off future problems. Specifically, introduce a mandatory -- separator now when options are supplied; if found in the argument list, everything before it gets parsed as options to RunInGenie, and everything after it gets passed as the inside-WSL command line. If not found in the argument list, everything gets passed as the inside-WSL command-line as is, and RunInGenie doesn't try to parse it for options.

Not sure what to do with -h/--help in this model, since people aren't going to understand to use -h -- unless they've already read the -h output. Must think on this more.

github-actions[bot] commented 3 years ago

There has been no activity on this issue for over 60 days. If there is no further activity within 7 days from this message, it will be automatically closed.

github-actions[bot] commented 3 years ago

There has been no activity on this issue for over 60 days. If there is no further activity within 7 days from this message, it will be automatically closed.

github-actions[bot] commented 3 years ago

There has been no activity on this issue for over 60 days. If there is no further activity within 7 days from this message, it will be automatically closed.