[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 4/7] docker: print warning if EXECUTABLE is not s
From: |
Sascha Silbe |
Subject: |
[Qemu-devel] [PATCH v2 4/7] docker: print warning if EXECUTABLE is not set when building debootstrap image |
Date: |
Wed, 24 Aug 2016 20:31:00 +0200 |
Building the debian-debootstrap image will usually fail if EXECUTABLE
isn't set (when using the Makefile). Warn the user in this case so
they know why it's failing.
Signed-off-by: Sascha Silbe <address@hidden>
---
tests/docker/Makefile.include | 3 +++
1 file changed, 3 insertions(+)
diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
index 1b20db0..19d4cc7 100644
--- a/tests/docker/Makefile.include
+++ b/tests/docker/Makefile.include
@@ -44,6 +44,9 @@ docker-image: ${DOCKER_TARGETS}
# General rule for building docker images
docker-image-%: $(DOCKER_FILES_DIR)/%.docker
+ @if test "$@" = docker-image-debian-bootstrap -a -z "$(EXECUTABLE)";
then \
+ echo WARNING: EXECUTABLE is not set, debootstrap may fail. 2>&1
; \
+ fi
$(call quiet-command,\
$(SRC_PATH)/tests/docker/docker.py build qemu:$* $< \
$(if $V,,--quiet) $(if $(NOCACHE),--no-cache) \
--
1.9.1
- [Qemu-devel] [PATCH v2 0/7] docker tests fixes, Sascha Silbe, 2016/08/24
- [Qemu-devel] [PATCH v2 3/7] docker: debian-bootstrap.pre: print helpful message if DEB_ARCH/DEB_TYPE unset, Sascha Silbe, 2016/08/24
- [Qemu-devel] [PATCH v2 4/7] docker: print warning if EXECUTABLE is not set when building debootstrap image,
Sascha Silbe <=
- [Qemu-devel] [PATCH v2 5/7] docker: make sure debootstrap is at least 1.0.67, Sascha Silbe, 2016/08/24
- [Qemu-devel] [PATCH v2 7/7] docker: silence debootstrap when --quiet is given, Sascha Silbe, 2016/08/24
- [Qemu-devel] [PATCH v2 1/7] docker.py: don't hang on large docker output, Sascha Silbe, 2016/08/24
- [Qemu-devel] [PATCH v2 2/7] docker: avoid dependency on 'realpath' package, Sascha Silbe, 2016/08/24
- [Qemu-devel] [PATCH v2 6/7] docker: build debootstrap after cloning, Sascha Silbe, 2016/08/24