Skip to content

Commit

Permalink
Merge pull request #20287 from smarterclayton/ginkgo_build
Browse files Browse the repository at this point in the history
Must use `make build` in spec file to build ginkgo
  • Loading branch information
smarterclayton committed Jul 12, 2018
2 parents a439eee + 51cab21 commit 3af079d
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 4 deletions.
4 changes: 4 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,10 @@ build-router-e2e-test:
hack/build-go.sh test/end-to-end/end-to-end.test
.PHONY: build-router-e2e-test

build-docs:
hack/generate-docs.sh
.PHONY: build-docs

# Run core verification and all self contained tests.
#
# Example:
Expand Down
3 changes: 3 additions & 0 deletions hack/lib/constants.sh
Original file line number Diff line number Diff line change
Expand Up @@ -335,6 +335,9 @@ readonly OS_ALL_IMAGES=(
# os::build::check_binaries ensures that binary sizes do not grow without approval.
function os::build::check_binaries() {
platform=$(os::build::host_platform)
if [[ "${platform}" != "linux/amd64" && "${platform}" != "darwin/amd64" ]]; then
return 0
fi
# enforce that certain binaries don't accidentally grow too large
# IMPORTANT: contact Clayton or another master team member before altering this code
if [[ -f "${OS_OUTPUT_BINPATH}/${platform}/oc" ]]; then
Expand Down
8 changes: 4 additions & 4 deletions origin.spec
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
# this is the version we obsolete up to. The packaging changed for Origin
# 1.0.6 and OSE 3.1 such that 'openshift' package names were no longer used.
%global package_refactor_version 3.0.2.900
%global golang_version 1.9.1
%global golang_version 1.10
# %commit and %os_git_vars are intended to be set by tito custom builders provided
# in the .tito/lib directory. The values in this spec file will not be kept up to date.
%{!?commit:
Expand Down Expand Up @@ -229,7 +229,7 @@ of docker. Exclude those versions of docker.
%if 0%{make_redistributable}
# Create Binaries for all supported arches
%{os_git_vars} OS_BUILD_RELEASE_ARCHIVES=n make build-cross
%{os_git_vars} hack/build-go.sh vendor/github.com/onsi/ginkgo/ginkgo
%{os_git_vars} OS_BUILD_RELEASE_ARCHIVES=n make build WHAT=vendor/github.com/onsi/ginkgo/ginkgo
%else
# Create Binaries only for building arch
%ifarch x86_64
Expand All @@ -248,11 +248,11 @@ of docker. Exclude those versions of docker.
BUILD_PLATFORM="linux/s390x"
%endif
OS_ONLY_BUILD_PLATFORMS="${BUILD_PLATFORM}" %{os_git_vars} OS_BUILD_RELEASE_ARCHIVES=n make build-cross
OS_ONLY_BUILD_PLATFORMS="${BUILD_PLATFORM}" %{os_git_vars} hack/build-go.sh vendor/github.com/onsi/ginkgo/ginkgo
OS_ONLY_BUILD_PLATFORMS="${BUILD_PLATFORM}" %{os_git_vars} OS_BUILD_RELEASE_ARCHIVES=n make build WHAT=vendor/github.com/onsi/ginkgo/ginkgo
%endif

# Generate man pages
%{os_git_vars} hack/generate-docs.sh
%{os_git_vars} make build-docs
%endif

%install
Expand Down

0 comments on commit 3af079d

Please sign in to comment.