Skip to content

Adjust man pages for create start split#878

Merged
cyphar merged 1 commit intoopencontainers:masterfrom
hqhq:adjust_man_for_run_create
Aug 24, 2016
Merged

Adjust man pages for create start split#878
cyphar merged 1 commit intoopencontainers:masterfrom
hqhq:adjust_man_for_run_create

Conversation

@hqhq
Copy link
Copy Markdown
Contributor

@hqhq hqhq commented Jun 4, 2016

Signed-off-by: Qiang Huang h.huangqiang@huawei.com

Comment thread man/runc.8.md Outdated
run create and run a container
spec create a new specification file
start create and run a container
start start signals a created container to execute the user defined process
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

might be better phrased as start executes the user defined process in a created container

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The initial phrase seems more accurate about what start does, but I'm ok with both, since it's Michael's words, @crosbymichael WDYT?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As an aside, we shouldn't repeat the command name as the first word. So "signals a created container to start execution".

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

whatever is more clear is good with me if you want to change both instances of the sentence

Signed-off-by: Qiang Huang <h.huangqiang@huawei.com>
@hqhq hqhq force-pushed the adjust_man_for_run_create branch 2 times, most recently from c02590f to c669b8d Compare June 12, 2016 12:42
@hqhq
Copy link
Copy Markdown
Contributor Author

hqhq commented Jun 12, 2016

Updated. PTAL.

@crosbymichael
Copy link
Copy Markdown
Member

crosbymichael commented Aug 23, 2016

LGTM

Approved with PullApprove

1 similar comment
@cyphar
Copy link
Copy Markdown
Member

cyphar commented Aug 24, 2016

LGTM

Approved with PullApprove

@cyphar cyphar merged commit c669b8d into opencontainers:master Aug 24, 2016
cyphar added a commit that referenced this pull request Aug 24, 2016
Closes #878 [Adjust man pages for create start split].
@hqhq hqhq deleted the adjust_man_for_run_create branch August 24, 2016 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants