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. See: #87
mysociety · Dec 5, 2023 · caadeeb · caadeeb
1 parent 70e8d36
commit caadeeb
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
@@ -337,7 +337,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() {
@@ -669,5 +669,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
@@ -337,7 +337,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() {