On Wed, 08/24 20:30, Sascha Silbe wrote:
> The debian-bootstrap image doesn't choose a default architecture and
> distribution version, instead the user has to set both DEB_ARCH and
> DEB_TYPE in the environment. Print a reasonably helpful message if
> either of them isn't set instead of complaining about "qemu-" being
> missing or erroring out because we cannot cd to the mirror URL.
>
> Signed-off-by: Sascha Silbe <si...@linux.vnet.ibm.com>
> ---
> tests/docker/dockerfiles/debian-bootstrap.pre | 13 +++++++++++++
> 1 file changed, 13 insertions(+)
>
> diff --git a/tests/docker/dockerfiles/debian-bootstrap.pre
> b/tests/docker/dockerfiles/debian-bootstrap.pre
> index 5d9c8d5..2ae363f 100755
> --- a/tests/docker/dockerfiles/debian-bootstrap.pre
> +++ b/tests/docker/dockerfiles/debian-bootstrap.pre
> @@ -15,6 +15,19 @@ exit_and_skip()
> if [ -z $FAKEROOT ]; then
> echo "Please install fakeroot to enable bootstraping"
> exit_and_skip
> +
> +fi
> +
> +if [ -z "${DEB_ARCH}" ]; then
> + echo "Please set DEB_ARCH to choose an architecture (e.g. armhf)"
> + exit_and_skip
> +
> +fi
> +
> +if [ -z "${DEB_TYPE}" ]; then
> + echo "Please set DEB_TYPE to a Debian archive name (e.g. testing)"
> + exit_and_skip
> +
> fi
>
> # We check in order for
> --
> 1.9.1
>
Can you also update the three 'echo' commands to output the message to stderr?
Fam
