Merge pull request #953 from haveneer-training/fix-installation-path
fix(install): protect path with whitespaces using quotes
This commit is contained in:
commit
0a92262fa0
|
@ -1,4 +1,5 @@
|
||||||
#!/usr/bin/env bash
|
#!/usr/bin/env bash
|
||||||
|
set -euo pipefail
|
||||||
|
|
||||||
echo "Let's get you set up with Rustlings!"
|
echo "Let's get you set up with Rustlings!"
|
||||||
|
|
||||||
|
@ -100,8 +101,8 @@ function vercomp() {
|
||||||
|
|
||||||
RustVersion=$(rustc --version | cut -d " " -f 2)
|
RustVersion=$(rustc --version | cut -d " " -f 2)
|
||||||
MinRustVersion=1.39
|
MinRustVersion=1.39
|
||||||
vercomp $RustVersion $MinRustVersion
|
vercomp "$RustVersion" $MinRustVersion || ec=$?
|
||||||
if [ $? -eq 2 ]
|
if [ ${ec:-0} -eq 2 ]
|
||||||
then
|
then
|
||||||
echo "ERROR: Rust version is too old: $RustVersion - needs at least $MinRustVersion"
|
echo "ERROR: Rust version is too old: $RustVersion - needs at least $MinRustVersion"
|
||||||
echo "Please update Rust with 'rustup update'"
|
echo "Please update Rust with 'rustup update'"
|
||||||
|
@ -112,9 +113,9 @@ fi
|
||||||
|
|
||||||
Path=${1:-rustlings/}
|
Path=${1:-rustlings/}
|
||||||
echo "Cloning Rustlings at $Path..."
|
echo "Cloning Rustlings at $Path..."
|
||||||
git clone -q https://github.com/rust-lang/rustlings $Path
|
git clone -q https://github.com/rust-lang/rustlings "$Path"
|
||||||
|
|
||||||
cd $Path
|
cd "$Path"
|
||||||
|
|
||||||
Version=$(curl -s https://api.github.com/repos/rust-lang/rustlings/releases/latest | ${PY} -c "import json,sys;obj=json.load(sys.stdin);print(obj['tag_name']);")
|
Version=$(curl -s https://api.github.com/repos/rust-lang/rustlings/releases/latest | ${PY} -c "import json,sys;obj=json.load(sys.stdin);print(obj['tag_name']);")
|
||||||
CargoBin="${CARGO_HOME:-$HOME/.cargo}/bin"
|
CargoBin="${CARGO_HOME:-$HOME/.cargo}/bin"
|
||||||
|
|
Loading…
Reference in a new issue