From 0bf0e5a6efb6d4718f18db11084295aaba2062e3 Mon Sep 17 00:00:00 2001 From: Doug Davis Date: Thu, 9 Jul 2015 09:57:42 -0700 Subject: [PATCH] Align 'docker commit' docs with the code It was missing some variants and 'maintainer' isn't actually supported. Also sorted the list of allowed cmds in the code just to make it easier to diff with the docs. Signed-off-by: Doug Davis --- builder/job.go | 11 +++++------ docs/reference/commandline/commit.md | 2 +- man/docker-commit.1.md | 2 +- 3 files changed, 7 insertions(+), 8 deletions(-) diff --git a/builder/job.go b/builder/job.go index 93273c7c20..09c290b3c8 100644 --- a/builder/job.go +++ b/builder/job.go @@ -32,16 +32,15 @@ const maxPreambleLength = 100 // whitelist of commands allowed for a commit/import var validCommitCommands = map[string]bool{ - "entrypoint": true, "cmd": true, - "user": true, - "workdir": true, + "entrypoint": true, "env": true, - "volume": true, "expose": true, - "onbuild": true, "label": true, - "maintainer": true, + "onbuild": true, + "user": true, + "volume": true, + "workdir": true, } type Config struct { diff --git a/docs/reference/commandline/commit.md b/docs/reference/commandline/commit.md index 18f3d32857..cec7de672c 100644 --- a/docs/reference/commandline/commit.md +++ b/docs/reference/commandline/commit.md @@ -35,7 +35,7 @@ undesired, set the 'p' option to false. The `--change` option will apply `Dockerfile` instructions to the image that is created. Supported `Dockerfile` instructions: -`CMD`|`ENTRYPOINT`|`ENV`|`EXPOSE`|`ONBUILD`|`USER`|`VOLUME`|`WORKDIR` +`CMD`|`ENTRYPOINT`|`ENV`|`EXPOSE`|`LABEL`|`ONBUILD`|`USER`|`VOLUME`|`WORKDIR` ## Commit a container diff --git a/man/docker-commit.1.md b/man/docker-commit.1.md index 875a9b7059..bc813ae71b 100644 --- a/man/docker-commit.1.md +++ b/man/docker-commit.1.md @@ -29,7 +29,7 @@ people. **-c** , **--change**=[] Apply specified Dockerfile instructions while committing the image - Supported Dockerfile instructions: `CMD`|`ENTRYPOINT`|`ENV`|`EXPOSE`|`ONBUILD`|`USER`|`VOLUME`|`WORKDIR` + Supported Dockerfile instructions: `CMD`|`ENTRYPOINT`|`ENV`|`EXPOSE`|`LABEL`|`ONBUILD`|`USER`|`VOLUME`|`WORKDIR` **--help** Print usage statement