summary refs log tree commit diff
diff options
context:
space:
mode:
authorJianxun Zhang <jianxun.zhang@intel.com>2019-10-04 15:32:45 -0700
committerCommit Bot <commit-bot@chromium.org>2019-10-10 18:33:47 +0000
commita462f74b2688379d04d4acf4337d07df58fc08e6 (patch)
tree169fe98aabca7087eba106f6e7ae231c3088da18
parent34533fe3ee2a0e0c6100c1fda43c9f29ff8f2222 (diff)
downloadcrosvm-a462f74b2688379d04d4acf4337d07df58fc08e6.tar
crosvm-a462f74b2688379d04d4acf4337d07df58fc08e6.tar.gz
crosvm-a462f74b2688379d04d4acf4337d07df58fc08e6.tar.bz2
crosvm-a462f74b2688379d04d4acf4337d07df58fc08e6.tar.lz
crosvm-a462f74b2688379d04d4acf4337d07df58fc08e6.tar.xz
crosvm-a462f74b2688379d04d4acf4337d07df58fc08e6.tar.zst
crosvm-a462f74b2688379d04d4acf4337d07df58fc08e6.zip
docker: fix a failure when building crosvm
What fixed by this change failed crosvm build with docker on
non-Nvidia platforms.

BUG=None
TEST=./build_crosvm_base.sh
./build_crosvm.sh
./wrapped_smoke_test.sh

Change-Id: Ib2cc70d5bd98bcad25410ba54c0492491a3ea799
Signed-off-by: Jianxun Zhang <jianxun.zhang@intel.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/1842519
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Zach Reizner <zachr@chromium.org>
Reviewed-by: Daniel Verkamp <dverkamp@chromium.org>
Commit-Queue: Daniel Verkamp <dverkamp@chromium.org>
-rw-r--r--docker/Dockerfile.crosvm2
1 files changed, 1 insertions, 1 deletions
diff --git a/docker/Dockerfile.crosvm b/docker/Dockerfile.crosvm
index 77ede70..60ba5a9 100644
--- a/docker/Dockerfile.crosvm
+++ b/docker/Dockerfile.crosvm
@@ -15,7 +15,7 @@ RUN export uid=$UID gid=$GID && \
 
 # Install nvidia proprietary drivers if requested.
 ARG NVIDIA_VERSION=
-RUN if [ -n NVIDIA_VERSION ]; then \
+RUN if [ -n "${NVIDIA_VERSION}" ]; then \
         nvidia_url=http://us.download.nvidia.com/XFree86/Linux-x86_64 && \
         nvidia_binary=NVIDIA-Linux-x86_64-"${NVIDIA_VERSION}".run && \
         curl -LO "${nvidia_url}"/"${NVIDIA_VERSION}"/"${nvidia_binary}" && \