From 97e389f73f2791d2eb4df57b4c75bdf1d4582bfa Mon Sep 17 00:00:00 2001
From: Nils Philippsen <nils@redhat.com>
Date: Wed, 24 Mar 2021 15:08:12 +0100
Subject: [PATCH] fix-home-fedora-ownerships: fix syntax errors

Signed-off-by: Nils Philippsen <nils@redhat.com>
---
 .../fix-home-fedora-ownerships.yml             | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/playbooks/manual/noggin-deployment/fix-home-fedora-ownerships.yml b/playbooks/manual/noggin-deployment/fix-home-fedora-ownerships.yml
index f5e2a16c16..55e704cb66 100644
--- a/playbooks/manual/noggin-deployment/fix-home-fedora-ownerships.yml
+++ b/playbooks/manual/noggin-deployment/fix-home-fedora-ownerships.yml
@@ -19,21 +19,21 @@
     # assumes that directory name == user name
     shell:
       chdir: /home/fedora
+      # skip directories owned by root and verify that the directory actually is the home directory
+      # of the same-named user
       cmd: >-
         for dname in */; do
-          dname="${dname%%/}"
-          # skip directories owned by root
-          downer="$(stat --format %U "$dname")"
+          dname="${dname%%/}";
+          downer="$(stat --format %U "$dname")";
           if [ "$downer" = "root" ]; then
             continue
-          fi
-          # verify the directory actually is the home directory of the same-named user
-          IFS=":" read -r _ _ _ _ _ homedir _ < <(getent passwd "$dname")
+          fi;
+          IFS=":" read -r _ _ _ _ _ homedir _ < <(getent passwd "$dname");
           if [ "$homedir" != "/home/fedora/$dname" ]; then
             continue
-          fi
-          echo "fixing ownership: $dname"
-          chown -R "$dname:" "$dname"
+          fi;
+          echo "fixing ownership: $dname";
+          chown -R "$dname:" "$dname";
         done
     register: chown_res
     changed_when: "'fixing ownership:' in chown_res.stdout"