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

Improve utilities command line interfaces #21

Open
moralismercatus opened this issue Feb 24, 2017 · 0 comments
Open

Improve utilities command line interfaces #21

moralismercatus opened this issue Feb 24, 2017 · 0 comments

Comments

@moralismercatus
Copy link
Collaborator

moralismercatus commented Feb 24, 2017

Utilities such as crete-dispatch, crete-vm-node, and crete-svm-node could all use some improvement in their command line interfaces.

All:

  1. Don't report exception is thrown when, for example, --help is used, or an incorrect option. Simply fixed by throwing a special purpose exception that is caught in a special purpose catch block e.g., something like crete::CmdLineArgException or crete::EarlyExitException

crete-dispatch:

  1. Include the ability to give a timeout as a cmdline argument e.g., --time-out/-t. A commonly requested feature.

crete-vm-node:

  1. -n erroneously states "number of svm instances". It should be "... vm instances"
  2. Provide complementary --vm option to crete-svm-node's --svm option that allows the user to specify the path of VM to use. I believe the reason this option was omitted originally is because the VM is architecture dependent; however, if that behavior is still desired at the cmdline, I don't see why it can't be done as --vm-x86, --vm-x64 options. Note that crete-svm-node uses platform agnostic --translator.
  3. Change option --port-master to simply --port. Originally done to distinguish from --guest-port before ports for guest were automatically selected.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant