From: Daniel Kahn Gillmor <dkg@fifthhorseman.net>
Date: Wed, 11 Nov 2009 22:16:30 +0000 (-0500)
Subject: abstract away the choice of placement of homedir for non-standard systems
X-Git-Url: https://gitweb.fluxo.info/?a=commitdiff_plain;h=4c74e5346f55dfe5f22f0d8a25a3a712ff27b6cf;p=kvm-manager.git

abstract away the choice of placement of homedir for non-standard systems
---

diff --git a/kvm-creator b/kvm-creator
index 744cb96..e2c0ca9 100755
--- a/kvm-creator
+++ b/kvm-creator
@@ -45,9 +45,8 @@ destroy() {
 
     update-service --remove "/etc/sv/kvm/$NAME"
     rm -rf "/etc/sv/kvm/$NAME"
-    deluser "$NAME"
+    deluser --remove-home "$NAME"
     lvremove "$VG/$NAME"
-    rm -rf "/home/$NAME"
 
 }
 
@@ -69,9 +68,13 @@ create() {
     set -e
     validate
     adduser "$NAME" --disabled-password --gecos "$NAME KVM user,,,"
-    mkdir -p "/home/$NAME/"{.ssh,vms,"vms/$NAME"}
-    chown "$NAME:$NAME" "/home/$NAME/"{.ssh,vms,"vms/$NAME"}
-    cp /root/.ssh/authorized_keys "/home/$NAME/.ssh/"
+    USERHOMEDIR=$(getent passwd "$NAME" | cut -f6 -d:)
+    mkdir -p "$USERHOMEDIR/"{.ssh,vms,"vms/$NAME"}
+    chown "$NAME:$NAME" "$USERHOMEDIR/"{.ssh,vms,"vms/$NAME"}
+    if [ -f /root/.ssh/authorized_keys ]; then
+        # is this really the right thing to do?
+        cp /root/.ssh/authorized_keys "$USERHOMEDIR/.ssh/"
+    fi
     lvcreate --name "$NAME" --size "$SIZE" $VG
     mkdir "/etc/sv/kvm/$NAME"{,/log,/env}
     cat > "/etc/sv/kvm/$NAME/log/run" <<EOF