Skip to content
This repository has been archived by the owner on Oct 9, 2023. It is now read-only.

Commit

Permalink
fix makefile to read variables from environment and overrides
Browse files Browse the repository at this point in the history
Signed-off-by: Jeev B <jeevb@users.noreply.github.com>
  • Loading branch information
jeevb committed Apr 12, 2023
1 parent 074f12d commit 85a06c6
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,8 @@ docker_build_scheduler:
docker build -t $$FLYTE_SCHEDULER_REPOSITORY:$(GIT_HASH) -f Dockerfile.scheduler .

.PHONY: integration
integration: export CGO_ENABLED = 0
integration: export GOFLAGS = -count=1
integration: export CGO_ENABLED ?= 0
integration: export GOFLAGS ?= -count=1
integration:
go test -v -tags=integration ./tests/...

Expand Down Expand Up @@ -53,14 +53,14 @@ compile_scheduler_debug:


.PHONY: linux_compile
linux_compile: export CGO_ENABLED = 0
linux_compile: export GOOS = linux
linux_compile: export CGO_ENABLED ?= 0
linux_compile: export GOOS ?= linux
linux_compile:
go build -o /artifacts/flyteadmin -ldflags=$(LD_FLAGS) ./cmd/

.PHONY: linux_compile_scheduler
linux_compile_scheduler: export CGO_ENABLED = 0
linux_compile_scheduler: export GOOS = linux
linux_compile_scheduler: export CGO_ENABLED ?= 0
linux_compile_scheduler: export GOOS ?= linux
linux_compile_scheduler:
go build -o /artifacts/flytescheduler -ldflags=$(LD_FLAGS) ./cmd/scheduler/

Expand Down

0 comments on commit 85a06c6

Please sign in to comment.