Skip to content

Commit

Permalink
Switch cross builds to use BUILD_IN_DOCKER
Browse files Browse the repository at this point in the history
minikube-linux-amd64 and minikube-darwin-amd64 cannot be cross built
natively on different platforms and must be built in the container.
  • Loading branch information
r2d4 committed Jun 29, 2017
1 parent 3ffc1e7 commit f998882
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion docs/contributors/releasing_minikube.md
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ Send a PR for the Makefile change and wait until it is merged. Once the commit
Run this command:

```shell
make cross checksum
BUILD_IN_DOCKER=y make cross checksum
```

## Add the version to the releases.json file
Expand Down
2 changes: 1 addition & 1 deletion hack/jenkins/minikube_cross_build_and_upload.sh
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ fi
export GOPATH=~/go

# Build all platforms (Windows, Linux, OSX)
make cross
BUILD_IN_DOCKER=y make cross

# Build the e2e test target for Darwin and Linux. We don't run tests on Windows yet.
# We build these on Linux, but run the tests on different platforms.
Expand Down
2 changes: 1 addition & 1 deletion hack/jenkins/release_build_and_upload.sh
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ cat Makefile | grep "VERSION_MINOR ?=" | grep $VERSION_MINOR
cat Makefile | grep "VERSION_BUILD ?=" | grep $VERSION_BUILD

# Build and upload
make cross checksum
BUILD_IN_DOCKER=y make cross checksum

gsutil cp out/minikube-linux-amd64 gs://$BUCKET/releases/$TAGNAME/
gsutil cp out/minikube-linux-amd64.sha256 gs://$BUCKET/releases/$TAGNAME/
Expand Down

0 comments on commit f998882

Please sign in to comment.