Skip to content

fix(*): resolve build and container start issues#89

Merged
mboersma merged 4 commits into
deis:masterfrom
mboersma:update-mc-location
Apr 13, 2016
Merged

fix(*): resolve build and container start issues#89
mboersma merged 4 commits into
deis:masterfrom
mboersma:update-mc-location

Conversation

@mboersma

@mboersma mboersma commented Apr 5, 2016

Copy link
Copy Markdown
Member

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

mboersma commented Apr 5, 2016

Copy link
Copy Markdown
Member Author

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