Thanks to visit codestin.com
Credit goes to github.com

Skip to content

builder: some small fixups + fix a bug where empty entrypoints would not override inheritance.#8744

Merged
LK4D4 merged 1 commit into
moby:masterfrom
erikh:empty_entrypoint_inheritance
Oct 24, 2014
Merged

builder: some small fixups + fix a bug where empty entrypoints would not override inheritance.#8744
LK4D4 merged 1 commit into
moby:masterfrom
erikh:empty_entrypoint_inheritance

Conversation

@erikh

@erikh erikh commented Oct 24, 2014

Copy link
Copy Markdown
Contributor

closes #8732. I also tacked on some spelling fixes and a formatting change to debug logging. There is a test as well.

The crux of the issue is multi-tiered:

  • in the parser, in single word cases (remember, [] evaluates to no content), the attributes and "original" line did not propagate to the evaluator.
  • in runconfig, merging the image settings into the new builder settings would not distinguish between nil and an empty array.
  • in the entrypoint evaluator, the switch statement would eagerly treat nil and empty arrays as equal regardless of how they were defined.

…not override inheritance.

Docker-DCO-1.1-Signed-off-by: Erik Hollensbe <[email protected]> (github: erikh)
@vieux

vieux commented Oct 24, 2014

Copy link
Copy Markdown
Contributor

LGTM

@crosbymichael crosbymichael added this to the 1.3.1 milestone Oct 24, 2014
@jessfraz

Copy link
Copy Markdown
Contributor

LGTM

1 similar comment
@LK4D4

LK4D4 commented Oct 24, 2014

Copy link
Copy Markdown
Contributor

LGTM

LK4D4 added a commit that referenced this pull request Oct 24, 2014
builder: some small fixups + fix a bug where empty entrypoints would not override inheritance.
@LK4D4 LK4D4 merged commit c681bea into moby:master Oct 24, 2014
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.

ENTRYPOINT [] not resetting parent image entrypoint

5 participants