refactor(scripts): use ln -P on non-darwin arch

jsjoeio · jsjoeio · commit dbf81fae4f69 · 2022-08-29T20:13:10.000Z
diff --git a/scripts/build_docker.sh b/scripts/build_docker.sh
@@ -90,8 +90,15 @@ fi
 # hardlinked from.
 cdroot
 temp_dir="$(TMPDIR="$(dirname "$input_file")" mktemp -d)"
-ln -P "$input_file" "$temp_dir/coder"
-ln -P Dockerfile "$temp_dir/"
+ln_flags=""
+
+# -P is an illegal option on darwin and will cause script to fail.
+if [[ $(uname | tr '[:upper:]' '[:lower:]') != 'darwin' ]]; then
+	ln_flags+=(-P)
+fi
+
+ln "${ln_flags[@]}" "$input_file" "$temp_dir/coder"
+ln "${ln_flags[@]}" Dockerfile "$temp_dir/"
 
 cd "$temp_dir"
 
diff --git a/scripts/package.sh b/scripts/package.sh
@@ -64,15 +64,21 @@ if [[ "$arch" == "arm" ]] || [[ "$arch" == "armv7" ]]; then
 	arch="armhf"
 fi
 
+ln_flags=""
+# -P is an illegal option on darwin and will cause script to fail.
+if [[ $(uname | tr '[:upper:]' '[:lower:]') != 'darwin' ]]; then
+	ln_flags+=(-P)
+fi
+
 # Make temporary dir where all source files intended to be in the package will
 # be hardlinked from.
 cdroot
 temp_dir="$(TMPDIR="$(dirname "$input_file")" mktemp -d)"
-ln -P "$input_file" "$temp_dir/coder"
-ln -P "$(realpath coder.env)" "$temp_dir/"
-ln -P "$(realpath coder.service)" "$temp_dir/"
-ln -P "$(realpath preinstall.sh)" "$temp_dir/"
-ln -P "$(realpath scripts/nfpm.yaml)" "$temp_dir/"
+ln "${ln_flags[@]}" "$input_file" "$temp_dir/coder"
+ln "${ln_flags[@]}" "$(realpath coder.env)" "$temp_dir/"
+ln "${ln_flags[@]}" "$(realpath coder.service)" "$temp_dir/"
+ln "${ln_flags[@]}" "$(realpath preinstall.sh)" "$temp_dir/"
+ln "${ln_flags[@]}" "$(realpath scripts/nfpm.yaml)" "$temp_dir/"
 
 cd "$temp_dir"
 
