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

manifest.md: use base layer instead of base image #463

Merged
merged 1 commit into from
Nov 30, 2016

Conversation

coolljt0725
Copy link
Member

We doesn't define a base image, use base layer is
correct. This issue first being addressed in pr #312,
and then being addressed in pr #318, and then in pr #407,
but landing #407 has a long way to go. We could address
this first to avoid confusing.

Signed-off-by: Lei Jitang leijitang@huawei.com

We doesn't define a `base image`, use `base layer` is
correct. This issue first being addressed in pr opencontainers#312,
and then being addressed in pr opencontainers#318, and then in pr opencontainers#407,
but landing opencontainers#407 has a long way to go. We could addressed
this first to avoid confusing.

Signed-off-by: Lei Jitang <leijitang@huawei.com>
@wking
Copy link
Contributor

wking commented Nov 18, 2016 via email

@wking
Copy link
Contributor

wking commented Nov 18, 2016

And note that this is exactly the same patch as #312. That PR was
closed in the naïve expectation that landing optional layers would be
fairly straightforward 1. Since that has proven to not be the case,
I think it makes sense to come back and handle this smaller cleanup
independently (as this PR is doing).

@jonboulle
Copy link
Contributor

jonboulle commented Nov 25, 2016

LGTM

Approved with PullApprove

1 similar comment
@vbatts
Copy link
Member

vbatts commented Nov 30, 2016

LGTM

Approved with PullApprove

@vbatts vbatts merged commit e0a5ba5 into opencontainers:master Nov 30, 2016
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.

4 participants