1 files changed, 30 insertions, 11 deletions
diff --git a/cloudmasterd/extra/cloudmasterd.spec b/cloudmasterd/extra/cloudmasterd.spec
index 7b938d1..eb822d8 100644
--- a/cloudmasterd/extra/cloudmasterd.spec
+++ b/cloudmasterd/extra/cloudmasterd.spec
@@ -18,7 +18,7 @@ Requires: genome-styling
 Requires: genome-autostarter
 Requires: koan
 Requires: python-virtinst
-Requires: libvirt
+Requires: /usr/bin/virsh
 Requires: rubygems
 Requires: rubygem(picnic) 
 Requires: rubygem(activesupport) 
@@ -26,6 +26,7 @@ Requires: rubygem(activerecord)
 Requires: rubygem(sqlite3-ruby) 
 Requires: rubygem(reststop) >= 0.2.1
 Requires: genome-styling
+Requires(post): chkconfig
 BuildRequires: rubygems
 BuildArch: noarch
 Provides: rubygem(%{gemname}) = %{version}
@@ -68,24 +69,35 @@ to the cloudmaster context.
 
 %install
 rm -rf %{buildroot}
-mkdir -p %{buildroot}%{gemdir} %{buildroot}/etc/init.d %{buildroot}/etc/cloudmasterd
+mkdir -p %{buildroot}%{gemdir} %{buildroot}%{_sysconfdir}/init.d %{buildroot}%{_sysconfdir}/cloudmasterd
 gem install --local --install-dir %{buildroot}%{gemdir} \
             --force --rdoc %{SOURCE0}
 mkdir -p %{buildroot}/%{_bindir}
 mv %{buildroot}%{gemdir}/bin/* %{buildroot}/%{_bindir}
-mv %{buildroot}%{geminstdir}/extra/cloudmasterd.init %{buildroot}/etc/init.d/cloudmasterd
-mv %{buildroot}%{geminstdir}/extra/config.yml %{buildroot}/etc/cloudmasterd/config.yml
+mv %{buildroot}%{geminstdir}/extra/cloudmasterd.init %{buildroot}%{_sysconfdir}/init.d/cloudmasterd
+mv %{buildroot}%{geminstdir}/extra/config.yml %{buildroot}%{_sysconfdir}/cloudmasterd/config.yml
 find %{buildroot}%{geminstdir}/bin -type f | xargs chmod a+x
 
 # Put in the apache rules
-mkdir -p %{buildroot}/etc/httpd/conf.d
-mv %{buildroot}%{geminstdir}/extra/cloudmasterd.apache %{buildroot}/etc/httpd/conf.d/cloudmasterd.conf
-mv %{buildroot}%{geminstdir}/extra/cloudmasterd-redirect.apache %{buildroot}/etc/httpd/conf.d/cloudmasterd-redirect.conf
+mkdir -p %{buildroot}%{_sysconfdir}/httpd/conf.d
+mv %{buildroot}%{geminstdir}/extra/cloudmasterd.apache %{buildroot}%{_sysconfdir}/httpd/conf.d/cloudmasterd.conf
+mv %{buildroot}%{geminstdir}/extra/cloudmasterd-redirect.apache %{buildroot}%{_sysconfdir}/httpd/conf.d/cloudmasterd-redirect.conf
+
+%post
+# Register the httpd service
+/sbin/chkconfig --add cloudmasterd
+
+%preun
+if [ $1 = 0 ]; then
+        /sbin/service cloudmasterd stop > /dev/null 2>&1
+        /sbin/chkconfig --del cloudmasterd
+fi
 
 %clean
 rm -rf %{buildroot}
 
 %files
+%config(noreplace) %{_sysconfdir}/cloudmasterd/config.yml
 %{_bindir}/cloudmasterd
 %{_bindir}/cloudmasterd-ctl
 %{_bindir}/cloudmasterd-sync
@@ -95,17 +107,24 @@ rm -rf %{buildroot}
 %doc %{geminstdir}/README
 %{gemdir}/cache/%{gemname}-%{version}.gem
 %{gemdir}/specifications/%{gemname}-%{version}.gemspec
-%config /etc/cloudmasterd/config.yml
 
 %defattr(755, root, root, -)
-/etc/init.d/cloudmasterd
+%{_sysconfdir}/init.d/cloudmasterd
 
 %files -n httpd-cloudmasterd
-/etc/httpd/conf.d/cloudmasterd.conf
+%doc %{gemdir}/doc/%{gemname}-%{version}
+%doc %{geminstdir}/LICENSE
+%doc %{geminstdir}/README
+%config(noreplace) %{_sysconfdir}/httpd/conf.d/cloudmasterd.conf
 
 %files -n httpd-cloudmasterd-redirect
-/etc/httpd/conf.d/cloudmasterd-redirect.conf
+%doc %{gemdir}/doc/%{gemname}-%{version}
+%doc %{geminstdir}/LICENSE
+%doc %{geminstdir}/README
+%config(noreplace) %{_sysconfdir}/httpd/conf.d/cloudmasterd-redirect.conf
 
 %changelog
+* Mon Jul 21 2008 Jeroen van Meeuwen <kanarip@fedoraproject.org> - 1.0.0-14
+- Some cleaning up for package review
 * Tue Jun 24 2008  <mhicks@redhat.com> - 1.0.0-1
 - Initial package