Hi Steven,

this patch updates the spec file to use macros consistently across the
board and fix a major issue by including /var/lib/corosync in the rpm
(taking ownership of the directory).

Without this fix, in a super clean corosync installation, corosync will
fail to start because of the lack of /var/lib/corosync and purging
corosync rpm would leave random files in there, that's undesirable
behaviour.

Please consider also backporting to flatiron.

Fabio
Index: corosync.spec.in
===================================================================
--- corosync.spec.in	(revision 2373)
+++ corosync.spec.in	(working copy)
@@ -34,9 +34,9 @@
 %endif
 
 %{_configure}	CFLAGS="$(echo '%{optflags}')" \
-		--prefix=/usr \
-		--sysconfdir=/etc \
-		--localstatedir=/var \
+		--prefix=%{_prefix} \
+		--sysconfdir=%{_sysconfdir} \
+		--localstatedir=%{_localstatedir} \
 		--libdir=%{_libdir}
 
 %build
@@ -83,9 +83,9 @@
 %{_sbindir}/corosync-cfgtool
 %{_sbindir}/corosync-fplay
 %{_sbindir}/corosync-pload
-%dir /etc/corosync
-%dir /etc/corosync/uidgid.d
-%config(noreplace) /etc/corosync/corosync.conf.example
+%dir %{_sysconfdir}/corosync
+%dir %{_sysconfdir}/corosync/uidgid.d
+%config(noreplace) %{_sysconfdir}/corosync/corosync.conf.example
 %{_initddir}/corosync
 %dir %{_libexecdir}/lcrso
 %{_libexecdir}/lcrso/coroparse.lcrso
@@ -99,6 +99,7 @@
 %{_libexecdir}/lcrso/quorum_testquorum.lcrso
 %{_libexecdir}/lcrso/vsf_quorum.lcrso
 %{_libexecdir}/lcrso/vsf_ykd.lcrso
+%dir %{_localstatedir}/lib/corosync
 %{_mandir}/man8/corosync_overview.8*
 %{_mandir}/man8/corosync-objctl.8*
 %{_mandir}/man5/corosync.conf.5*
_______________________________________________
Openais mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/openais

Reply via email to