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

Add windows back. #9642

Merged
merged 1 commit into from
Dec 15, 2014
Merged

Add windows back. #9642

merged 1 commit into from
Dec 15, 2014

Conversation

jessfraz
Copy link
Contributor

Post Major Release: Now it can be part of the master builds again :)

ping @tianon @ahmetalpbalkan

Docker-DCO-1.1-Signed-off-by: Jessica Frazelle <jess@docker.com> (github: jfrazelle)
@tianon
Copy link
Member

tianon commented Dec 12, 2014

LGTM

@tianon
Copy link
Member

tianon commented Dec 12, 2014

Let's fix the issues that caused us to remove this in time for 1.5, yeah? 😉 👍

@jessfraz
Copy link
Contributor Author

ya!

On Fri, Dec 12, 2014 at 2:29 PM, Tianon Gravi notifications@github.com
wrote:

Let's fix the issues that caused us to remove this in time for 1.5, yeah? [image:
😉] [image: 👍]


Reply to this email directly or view it on GitHub
#9642 (comment).

@ahmetb
Copy link
Contributor

ahmetb commented Dec 13, 2014

LGTM. We've new folks to work on this on board now and I'll be on vacation for almost a month. I hope we can make it to 1.5! 😉

@jessfraz
Copy link
Contributor Author

Have a great vacation!!

On Saturday, December 13, 2014, Ahmet Alp Balkan notifications@github.com
wrote:

LGTM. we've new folks more people to work on this on board now and I'll be
on vacation for almost a month. I hope we can make it to 1.5! [image:
😉]


Reply to this email directly or view it on GitHub
#9642 (comment).

@SvenDowideit
Copy link
Contributor

yay!

@@ -68,9 +68,8 @@ RUN cd /usr/local/go/src && ./make.bash --no-clean 2>&1
ENV DOCKER_CROSSPLATFORMS \
linux/386 linux/arm \
darwin/amd64 darwin/386 \
freebsd/amd64 freebsd/386 freebsd/arm
# windows is experimental for now
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isn't windows still experimental? :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes but this way people can build from the Dockerfile and we can generate
the binaries for master.dockerproject.com

On Mon, Dec 15, 2014 at 8:32 AM, Alexander Morozov <notifications@github.com

wrote:

In Dockerfile
#9642 (diff):

@@ -68,9 +68,8 @@ RUN cd /usr/local/go/src && ./make.bash --no-clean 2>&1
ENV DOCKER_CROSSPLATFORMS
linux/386 linux/arm
darwin/amd64 darwin/386 \

  • freebsd/amd64 freebsd/386 freebsd/arm
    -# windows is experimental for now

isn't windows still experimental? :)


Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/9642/files#r21836991.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I only removed it for the release script

On Mon, Dec 15, 2014 at 8:34 AM, Jessica Frazelle jess@docker.com wrote:

yes but this way people can build from the Dockerfile and we can generate
the binaries for master.dockerproject.com

On Mon, Dec 15, 2014 at 8:32 AM, Alexander Morozov <
notifications@github.com> wrote:

In Dockerfile
#9642 (diff):

@@ -68,9 +68,8 @@ RUN cd /usr/local/go/src && ./make.bash --no-clean 2>&1
ENV DOCKER_CROSSPLATFORMS
linux/386 linux/arm
darwin/amd64 darwin/386 \

  • freebsd/amd64 freebsd/386 freebsd/arm
    -# windows is experimental for now

isn't windows still experimental? :)


Reply to this email directly or view it on GitHub
https://github.com/docker/docker/pull/9642/files#r21836991.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I meant only comment :)

@LK4D4
Copy link
Contributor

LK4D4 commented Dec 15, 2014

LGTM

LK4D4 added a commit that referenced this pull request Dec 15, 2014
@LK4D4 LK4D4 merged commit 1bdf579 into moby:master Dec 15, 2014
@jessfraz jessfraz deleted the windows branch December 18, 2014 23:36
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.

None yet

5 participants