On 06/27/2012 04:32 PM, John Eckersberg wrote:
Previously the aeolus user's home directory was set to
/var/aeolus.
This was changed to /usr/share/aeolus-conductor in commit 923f894.
However this only catches new installations; if the aeolus user
previously existed (e.g. upgrade) then the home directory would not be
updated. This catches that case and ensures that old installations
using the old home directory are updated.
---
aeolus-conductor.spec.in | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/aeolus-conductor.spec.in b/aeolus-conductor.spec.in
index 6186651..15ace33 100644
--- a/aeolus-conductor.spec.in
+++ b/aeolus-conductor.spec.in
@@ -164,6 +164,10 @@ getent passwd aeolus >/dev/null || \
/usr/sbin/useradd -u 180 -g aeolus -c "aeolus" \
-s /sbin/nologin -r -d /usr/share/aeolus-conductor aeolus 2> /dev/null || :
+# Previously ~aeolus was set to /var/aeolus
+# If this is an upgrade scenario, fix the home directory
+test ~aeolus == /usr/share/aeolus-conductor || usermod --home
/usr/share/aeolus-conductor aeolus
+
%install
%{__mkdir} -p %{buildroot}
%{__mkdir} -p %{buildroot}%{app_root}
After building the rpms on F17 tried to install them but with this patch
in master I got the following error message:
usermod: user aeolus is currently logged in
With this patch reverted I was able to install the rpms.
Did I forget something or a new issue was introduced with this patch?
Imre