From f22f095d8ab755b7a9d523d632550c0709a9538c Mon Sep 17 00:00:00 2001 From: Greg Warner Date: Mon, 27 Jan 2025 16:07:46 -0700 Subject: [PATCH] fix chown syntax Some `chown` binaries seem to support the `.` notation, but a few don't. I haven't looked into why that is. Since `:` is the arguably ubiquitous, conventional, and universal (?) syntax, I'm changing it to head in that direction. That said, usage of `copy.sh` should probably be phased out, anyway, and the from-rust-built binary `copy` should be used instead. --- scripts/copy.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/copy.sh b/scripts/copy.sh index fd039fa..52c23ed 100755 --- a/scripts/copy.sh +++ b/scripts/copy.sh @@ -25,7 +25,7 @@ dst_dir="${1}"; shift || die 2 "Usage: ${0} [-T|--template] SRC_DIR DST_DIR" echo cp -a "${src}" "${dst}" cp -a "${src}" "${dst}" || die 1 "Failed to copy file" # TODO: make this configurable - chown root.root "${dst}" || die 1 "Unable to set ownership" + chown root:root "${dst}" || die 1 "Unable to set ownership" chmod +w "${dst}" || die 1 "Unable to make writable" [ -z "${TEMPLATE}" ] && continue