Conditional ACK. Comment inline.
On 02/24/2011 02:51 PM, jprovazn(a)redhat.com wrote:
From: Jan Provaznik<jprovazn(a)redhat.com>
Log dir is linked to /var/log/aeolus-conductor in pre-install.
---
aeolus-conductor.spec.in | 5 ++++-
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/aeolus-conductor.spec.in b/aeolus-conductor.spec.in
index 45bbe9f..cb0cec8 100644
--- a/aeolus-conductor.spec.in
+++ b/aeolus-conductor.spec.in
@@ -108,7 +108,7 @@ make
%{__cp} -R src/* %{buildroot}/%{app_root}
# we need to make sure there are no leftover logfiles getting pulled in
-rm -f %{buildroot}/%{app_root}/log/*
+%{__rm} -rf %{buildroot}/%{app_root}/log
# Remove this directory as the make install below will install the correct files to
this dir.
%{__rm} -rf %{buildroot}%{app_root}/classad_plugin
@@ -175,6 +175,9 @@ touch
%{buildroot}%{_localstatedir}/log/%{name}/image_builder_service.log
#rake spec
%pre daemons
+[ -d %{app_root}/log ]&& %{__rm} -rf %{app_root}/log
+%{__ln_s} %{_localstatedir}/log/%{name} %{app_root}/log
+
I thought you were going to link in the other direction, keeping what is
in %{app_root}. I think we should figure out a different way to
accomplish the logging before pushing the code in.
getent group aeolus>/dev/null || /usr/sbin/groupadd -g 451 -r
aeolus 2>/dev/null || :
getent passwd aeolus>/dev/null || \
/usr/sbin/useradd -u 451 -g aeolus -c "aeolus" \