Use ":" as user/group separator in chown

Long ago, this became standard to allow for the use of "." in group names. As of more recent distributions this has started to generate warnings.
mysociety · Dec 5, 2023 · 76ff007 · 76ff007
1 parent 34c3a42
commit 76ff007
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/bin/install-site-core.sh b/bin/install-site-core.sh
@@ -175,5 +175,5 @@ update_apt_sources
 # Remove one crippling package, if it's installed:
 apt-get -qq remove -y --purge apt-xapian-index >/dev/null || true
 clone_or_update_repository
-chown -R "$UNIX_USER"."$UNIX_USER" "$DIRECTORY"
+chown -R "$UNIX_USER":"$UNIX_USER" "$DIRECTORY"
 run_site_specific_script
diff --git a/bin/install-site.sh b/bin/install-site.sh
@@ -336,7 +336,7 @@ install_postgis() {
 make_log_directory() {
     LOG_DIRECTORY="$DIRECTORY/logs"
     mkdir -p "$LOG_DIRECTORY"
-    chown -R "$UNIX_USER"."$UNIX_USER" "$LOG_DIRECTORY"
+    chown -R "$UNIX_USER":"$UNIX_USER" "$LOG_DIRECTORY"
 }
 
 add_website_to_nginx() {
@@ -668,5 +668,5 @@ update_apt_sources
 # Remove one crippling package, if it's installed:
 apt-get -qq remove -y --purge apt-xapian-index >/dev/null || true
 clone_or_update_repository
-chown -R "$UNIX_USER"."$UNIX_USER" "$DIRECTORY"
+chown -R "$UNIX_USER":"$UNIX_USER" "$DIRECTORY"
 run_site_specific_script
diff --git a/shlib/installfns b/shlib/installfns
@@ -336,7 +336,7 @@ install_postgis() {
 make_log_directory() {
     LOG_DIRECTORY="$DIRECTORY/logs"
     mkdir -p "$LOG_DIRECTORY"
-    chown -R "$UNIX_USER"."$UNIX_USER" "$LOG_DIRECTORY"
+    chown -R "$UNIX_USER":"$UNIX_USER" "$LOG_DIRECTORY"
 }
 
 add_website_to_nginx() {