[PATCH] D37213: Changed Dockerfiles to install LLVM into /usr/local

[Ilya Biryukov via Phabricator via llvm-commits]

ilya-biryukov created this revision.

Previously, the installation path was simply '/'.
Using '/usr/local' would ensure that LLVM installation does not
conflict with software installed via package managers.


https://reviews.llvm.org/D37213

Files:
  utils/docker/debian8/release/Dockerfile
  utils/docker/example/release/Dockerfile
  utils/docker/nvidia-cuda/release/Dockerfile


Index: utils/docker/nvidia-cuda/release/Dockerfile
===================================================================
--- utils/docker/nvidia-cuda/release/Dockerfile
+++ utils/docker/nvidia-cuda/release/Dockerfile
@@ -18,6 +18,6 @@
 LABEL maintainer "LLVM Developers"
 
 # Unpack clang installation into this container.
-ADD clang.tar.gz /
+ADD clang.tar.gz /usr/local/
 
 # C++ standard library and binutils are already included in the base package.
Index: utils/docker/example/release/Dockerfile
===================================================================
--- utils/docker/example/release/Dockerfile
+++ utils/docker/example/release/Dockerfile
@@ -21,4 +21,4 @@
 
 # Unpack clang installation into this container.
 # It is copied to this directory by build_docker_image.sh script.
-ADD clang.tar.gz /
+ADD clang.tar.gz /usr/local/
Index: utils/docker/debian8/release/Dockerfile
===================================================================
--- utils/docker/debian8/release/Dockerfile
+++ utils/docker/debian8/release/Dockerfile
@@ -18,4 +18,4 @@
     rm -rf /var/lib/apt/lists/*
 
 # Unpack clang installation into this image.
-ADD clang.tar.gz /
+ADD clang.tar.gz /usr/local/


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37213.112897.patch
Type: text/x-patch
Size: 1194 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170828/9dacfb58/attachment.bin>