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

updated default -H docs #1576

Merged
merged 3 commits into from Aug 21, 2013
Merged

updated default -H docs #1576

merged 3 commits into from Aug 21, 2013

Conversation

matthewmueller
Copy link
Contributor

discussed: #938

@crosbymichael
Copy link
Contributor

@matthewmueller Does this need to be changed in the command line also?

@matthewmueller
Copy link
Contributor Author

@crosbymichael woops. thought the rst was being pulled in. updated.

@crosbymichael
Copy link
Contributor

@matthewmueller In the cli it says the unix socket part twice

@vieux
Copy link
Contributor

vieux commented Aug 19, 2013

@matthewmueller to be consistent with docker --help, I think commands.go should be:

help := fmt.Sprintf("Usage: docker [OPTIONS] COMMAND [arg...]\n -H=[unix://%s]: tcp://host:port to bind/connect to or unix://path/to/socket to use\n\nA self-sufficient runtime for linux containers.\n\nCommands:\n", DEFAULTUNIXSOCKET)

@matthewmueller
Copy link
Contributor Author

@vieux okay fixed help based on what you recommended, anything else?

@vieux
Copy link
Contributor

vieux commented Aug 21, 2013

@matthewmueller nope, thanks

LGTM

@crosbymichael
Copy link
Contributor

LGTM

crosbymichael added a commit that referenced this pull request Aug 21, 2013
@crosbymichael crosbymichael merged commit e0a7013 into moby:master Aug 21, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants