Skip to content

Conversation

@mboersma
Copy link
Member

@mboersma mboersma commented Apr 5, 2016

The old URL for fetching mc was replaced by a new server and paths, which broke the deis/minio build. Additionally, go vet is no longer installed that way, so this removes that line from install.sh before building. This also reverts a change back to the last-known-good version of deis/pkg/aboutme.

@mboersma mboersma self-assigned this Apr 5, 2016
@mboersma mboersma added the bug label Apr 5, 2016
@mboersma mboersma added this to the v2.0-beta2 milestone Apr 5, 2016
@mboersma
Copy link
Member Author

mboersma commented Apr 5, 2016

This passed Travis CI but wouldn't re-build in Jenkins for some reason:

 [8] System error: exec: "/bin/sh": stat /bin/sh: not a directory

😕

@mboersma mboersma force-pushed the update-mc-location branch from 268c035 to fb9c82d Compare April 5, 2016 23:00
@mboersma mboersma force-pushed the update-mc-location branch from 7955208 to 8e5f5f4 Compare April 12, 2016 20:15
@mboersma mboersma changed the title fix(Dockerfile): update location of "mc" binary fix(*): resolve build and container start issues Apr 12, 2016
@kmala kmala added the LGTM1 label Apr 12, 2016
@mboersma mboersma merged commit 0c71217 into deis:master Apr 13, 2016
@mboersma mboersma deleted the update-mc-location branch April 13, 2016 21:32
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.

3 participants