Skip to content
This repository has been archived by the owner on May 10, 2023. It is now read-only.

Add commandline options and shorthands #6

Open
gbraad opened this issue Feb 28, 2021 · 4 comments
Open

Add commandline options and shorthands #6

gbraad opened this issue Feb 28, 2021 · 4 comments

Comments

@gbraad
Copy link
Contributor

gbraad commented Feb 28, 2021

Allow the use of:

  • --shell [shell] / -s [shell]
  • --distro [distro] / -d [distro]
  • --login / -l
  • etc

This is a follow-up to #5 (comment)

@cerebrate
Copy link
Member

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
Copy link

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
Copy link

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
Copy link

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.

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

No branches or pull requests

2 participants