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

trusty tag #30

Closed
wants to merge 1 commit into from
Closed

trusty tag #30

wants to merge 1 commit into from

Conversation

moander
Copy link

@moander moander commented Jul 17, 2015

Identical to version/wheezy/Dockerfile except FROM

@chorrell
Copy link
Contributor

Hi!

Thanks for this!

I'd like to hold off for a bit on adding new variants. Eventually this repo is going to move under the nodejs organization and there will be the forthcoming node.js/io.js merged release too. I'd like to wait until after either of those things happen and keep this open to discuss with the @nodejs/docker working group.

@retrohacker
Copy link
Contributor

Interested in revisiting this. I've built a lot of tooling to maintain a set of images that offer tags like these. They can be found at https://github.com/nodesource/docker-node

We could translate some of that work here if there is value in it.

@Starefossen
Copy link
Member

Is there some obvious benefit of using ubuntu over debian for Docker Images, or is this just a matter of personal preference?

@moander
Copy link
Author

moander commented Jan 26, 2016

I don't remember the exact case, but there was an issue with something running on Debian. Something was only going to work on trusty.

docker-library/buildpack-deps@71935ca

@chorrell
Copy link
Contributor

I'm wary of adding another variant that needs to be maintained unless there's a really strong use case for it.

@retrohacker
Copy link
Contributor

@nodejs/docker is it safe to close this PR?

@Starefossen
Copy link
Member

I would think so @retrohacker

@retrohacker retrohacker closed this Mar 4, 2016
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