Skip to content

Update the cp command docs according to commit d40a17ffc2#33548

Closed
boaz0 wants to merge 1 commit into
moby:masterfrom
boaz0:cp_permissions_docs
Closed

Update the cp command docs according to commit d40a17ffc2#33548
boaz0 wants to merge 1 commit into
moby:masterfrom
boaz0:cp_permissions_docs

Conversation

@boaz0
Copy link
Copy Markdown
Contributor

@boaz0 boaz0 commented Jun 6, 2017

related to #30877 and #28923

- What I did

Update the docs

- How I did it

Add a small paragraph regarding the ownership of the copied files.

@GordonTheTurtle
Copy link
Copy Markdown

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "cp_permissions_docs" git@github.com:ripcurld0/moby.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

@GordonTheTurtle GordonTheTurtle added dco/no Automatically set by a bot when one of the commits lacks proper signature status/0-triage and removed dco/no Automatically set by a bot when one of the commits lacks proper signature labels Jun 6, 2017
@thaJeztah
Copy link
Copy Markdown
Member

Thanks! I think this needs a mention of the new flag that was added (which makes the actual change);

-a, --archive       Archive mode (copy all uid/gid information)

Signed-off-by: Boaz Shuster <ripcurld.github@gmail.com>
@cpuguy83
Copy link
Copy Markdown
Member

Thanks @ripcurld0.

These docs were moved to github.com/docker/cli. Can you open this over there?
Closing this one since we can't merge here.

@cpuguy83 cpuguy83 closed this Jun 20, 2017
@boaz0 boaz0 deleted the cp_permissions_docs branch June 21, 2017 10:23
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.

4 participants