mirror of https://github.com/docker/compose.git
ci: don't use `-race` on Windows
I misunderstood the cause of the symptom in #10261 - thought that we'd explicitly turned off CGO for Windows with some of the build changes recently, but we don't even have `gcc` on the CI node, so it's actually just `-race` entirely that's the trouble. For right now, going the easy route and disabling it. We can look at getting a C toolchain on the Windows machine later. Signed-off-by: Milas Bowman <milas.bowman@docker.com>
This commit is contained in:
parent
d93da187f2
commit
5b043c4d59
9
Makefile
9
Makefile
|
@ -33,7 +33,12 @@ ifeq ($(DETECTED_OS),Windows)
|
||||||
BINARY_EXT=.exe
|
BINARY_EXT=.exe
|
||||||
endif
|
endif
|
||||||
|
|
||||||
TEST_COVERAGE_FLAGS = -race -coverprofile=coverage.out -covermode=atomic
|
TEST_COVERAGE_FLAGS = -coverprofile=coverage.out -covermode=atomic
|
||||||
|
ifneq ($(DETECTED_OS),Windows)
|
||||||
|
# go race detector requires gcc on Windows so not used by default
|
||||||
|
# https://github.com/golang/go/issues/27089
|
||||||
|
TEST_COVERAGE_FLAGS += -race
|
||||||
|
endif
|
||||||
TEST_FLAGS?=
|
TEST_FLAGS?=
|
||||||
E2E_TEST?=
|
E2E_TEST?=
|
||||||
ifeq ($(E2E_TEST),)
|
ifeq ($(E2E_TEST),)
|
||||||
|
@ -61,7 +66,7 @@ install: binary
|
||||||
|
|
||||||
.PHONY: e2e-compose
|
.PHONY: e2e-compose
|
||||||
e2e-compose: ## Run end to end local tests in plugin mode. Set E2E_TEST=TestName to run a single test
|
e2e-compose: ## Run end to end local tests in plugin mode. Set E2E_TEST=TestName to run a single test
|
||||||
CGO_ENABLED=1 go test $(TEST_FLAGS) $(TEST_COVERAGE_FLAGS) -count=1 ./pkg/e2e
|
go test $(TEST_FLAGS) $(TEST_COVERAGE_FLAGS) -count=1 ./pkg/e2e
|
||||||
|
|
||||||
.PHONY: e2e-compose-standalone
|
.PHONY: e2e-compose-standalone
|
||||||
e2e-compose-standalone: ## Run End to end local tests in standalone mode. Set E2E_TEST=TestName to run a single test
|
e2e-compose-standalone: ## Run End to end local tests in standalone mode. Set E2E_TEST=TestName to run a single test
|
||||||
|
|
Loading…
Reference in New Issue