search

valhalla / docker

docker
MIT License
17 stars 22 forks source link

Refactor: Creates 'install-shared-deps.sh' for common dependencies #47

Closed purew closed 4 years ago

purew commented 4 years ago

Ah my bad. Still had a diff locally:

diff --git a/install-shared-deps.sh b/install-shared-deps.sh
index a7e601b..877571a 100644
--- a/install-shared-deps.sh
+++ b/install-shared-deps.sh
@@ -57,6 +57,6 @@ curl -o- https://raw.githubusercontent.com/creationix/nvm/v0.33.11/install.sh |
     && nvm use default

 # install cmake
-https://cmake.org/files/v3.11/cmake-3.11.2-Linux-$(uname --machine).sh /tmp/cmake.sh
+curl https://cmake.org/files/v3.11/cmake-3.11.2-Linux-$(uname --machine).sh > /tmp/cmake.sh
 sh /tmp/cmake.sh --prefix=/usr/local --skip-license && /bin/rm /tmp/cmake.sh
 cmake --version

Fixing with new PR.