Skip to content

Commit 0674632

Browse files
committed
tests/docker: set HOST_ARCH if we don't have ARCH
As the docker rules want to be able to be run on a virgin unconfigured checkout add a fallback and use it if we need to. Signed-off-by: Alex Bennée <[email protected]>
1 parent 2efabbe commit 0674632

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

.shippable.yml

-2
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,6 @@ env:
2727
TARGET_LIST=ppc64-softmmu,ppc64-linux-user,ppc64abi32-linux-user
2828
build:
2929
pre_ci:
30-
# usually host ARCH is set by configure
31-
- echo "ARCH=$(uname -m)" > config-host.mak
3230
- make docker-image-${IMAGE} V=1
3331
pre_ci_boot:
3432
image_name: qemu

tests/docker/Makefile.include

+4-2
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@
22

33
.PHONY: docker docker-test docker-clean docker-image docker-qemu-src
44

5+
HOST_ARCH = $(if $(ARCH),$(ARCH),$(shell uname -m))
6+
57
DOCKER_SUFFIX := .docker
68
DOCKER_FILES_DIR := $(SRC_PATH)/tests/docker/dockerfiles
79
# we don't run tests on intermediate images (used as base by another image)
@@ -88,7 +90,7 @@ endif
8890

8991
# Enforce dependencies for composite images
9092
docker-image-debian9-mxe: docker-image-debian9
91-
ifeq ($(ARCH),x86_64)
93+
ifeq ($(HOST_ARCH),x86_64)
9294
docker-image-debian-amd64: docker-image-debian9
9395
DOCKER_PARTIAL_IMAGES += debian-amd64-cross
9496
else
@@ -106,7 +108,7 @@ docker-image-debian-win32-cross: docker-image-debian9-mxe
106108
docker-image-debian-win64-cross: docker-image-debian9-mxe
107109

108110
# For non-x86 hosts not all cross-compilers have been packaged
109-
ifneq ($(ARCH),x86_64)
111+
ifneq ($(HOST_ARCH),x86_64)
110112
DOCKER_PARTIAL_IMAGES += debian-mips-cross debian-mipsel-cross debian-mips64el-cross
111113
DOCKER_PARTIAL_IMAGES += debian-ppc64el-cross
112114
DOCKER_PARTIAL_IMAGES += debian-s390x-cross

0 commit comments

Comments
 (0)