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

changes to CLI #86

Open
stdweird opened this issue Sep 30, 2016 · 4 comments
Open

changes to CLI #86

stdweird opened this issue Sep 30, 2016 · 4 comments

Comments

@stdweird
Copy link
Member

stdweird commented Sep 30, 2016

Propose changes to the CLI

  • --all implies --configure, no reason to add it
  • --skip should support comma-sperated list of components, and should work without specifiying --all

New alias

  • --only equal to --configure --nodeps --noautodeps to run only the listed components (this might to dangerous to add, as --configure --nodeps --noautodeps clearly indicates --i-know-what-i-am-doing)
@stdweird stdweird added this to the 16.10 milestone Sep 30, 2016
@piojo-zz
Copy link
Member

piojo-zz commented Oct 5, 2016

So, if I get --skip right, you mean:

ncm-ncd --co sudo useraccess --skip spma,accounts

it runs sudo and useraccess and their dependencies, but doesn't run spma or accounts, and yet it marks them as successful so that it execution reaches sudo and useraccess?

@ned21
Copy link
Contributor

ned21 commented Oct 5, 2016

--all should still require --configure as discussed in #80

--skip allowing you to skip dependencies when used without --all => agreed. Also support more than one.

--only seems like something that should be an alias in your shell config.

If we want to do any clean up then reducing the confusion between --nodeps --noautodeps and --ignore-errors-from-dependencies would be a good thing.

@jouvin
Copy link
Contributor

jouvin commented Oct 5, 2016

--nodeps seems misleading according to the man... Should be --ignore-missing-dependencies...

@jrha
Copy link
Member

jrha commented Oct 19, 2016

Discussed at workshop and broadly agreed, but no sign of a PR for 16.10 (unless something was buried in #81 or #88).

@jrha jrha removed this from the 16.10 milestone Oct 19, 2016
ttyS4 pushed a commit to ttyS4/ncm-ncd that referenced this issue Dec 15, 2017
ccm-initialise: fix bug after code cleanup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

5 participants