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

Update README to point to correct image #288

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update README to point to correct image #288

wants to merge 1 commit into from

Conversation

nvllsvm
Copy link

@nvllsvm nvllsvm commented May 30, 2017

The image should be alpine instead of gliderlabs/alpine.

@andyshinn
Copy link
Contributor

I'm not sure about this. This project is the gliderlabs/alpine image. We supply the tarballs for the official repository. But we were first gliderlabs/alpine and still are. The argument against this is because the specific official docs are at https://github.com/docker-library/docs/tree/master/alpine so those docs would reference alpine while ours would gliderlabs/alpine.

Though, I'll leave this open for more feedback. @tianon thoughts?

@nvllsvm
Copy link
Author

nvllsvm commented May 30, 2017

My confusion arose when I saw the latest tag in gliderlabs/alpine is 3.4 while the official alpine tag is the latest 3.6 - just like this repo. Will the gliderlabs/alpine image be kept up-to-date with the latest tags?

Additional confusion was felt after seeing that Glider Labs is mentioned as the maintainer in the official image.

@andyshinn
Copy link
Contributor

That is a good question. I think the last two releases were pull requests and maybe they just neglected to update both. I'll get them updated to they are in sync.

andyshinn pushed a commit that referenced this pull request May 31, 2017
tao12345666333 pushed a commit to tao12345666333/docker-alpine that referenced this pull request Jul 12, 2018
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.

2 participants