[PATCH] RPM spec updates

# HG changeset patch # User Dan Smith <danms@us.ibm.com> # Date 1197478221 28800 # Node ID de0e1bd9377b3e3325eec951ed0b0bf5100f565c # Parent c9edd5bc7202c0ee76bfba68aac98fd97ca87b34 RPM spec updates - Restart the CIMOM after base schema installation - Installed file cleanups - Permissions cleanups - Require correct version of libvirt - Require unzip, since we need it for base schema unpacking - Require explicit verison of libxml2 (for rpmlint). I have no idea what version is actually required, so I took the F8 version. If someone has an opinion or complaint about this, I'm happy to change it. Signed-off-by: Dan Smith <danms@us.ibm.com> diff -r c9edd5bc7202 -r de0e1bd9377b libvirt-cim.spec.in --- a/libvirt-cim.spec.in Wed Dec 12 08:42:39 2007 -0800 +++ b/libvirt-cim.spec.in Wed Dec 12 08:50:21 2007 -0800 @@ -9,10 +9,11 @@ Source: libvirt-cim-%{version}.tar.gz Source: libvirt-cim-%{version}.tar.gz BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root URL: http://libvirt.org/CIM/ -Requires: libxml2 -Requires: libvirt >= 0.2.3 +Requires: libxml2 >= 2.6.30 +Requires: libvirt >= 0.3.2 +Requires: unzip BuildRequires: tog-pegasus-devel -BuildRequires: libvirt-devel >= 0.2.3 +BuildRequires: libvirt-devel >= 0.3.2 BuildRequires: e2fsprogs-devel BuildRequires: libxml2-devel BuildRequires: libcmpiutil-devel @@ -26,7 +27,8 @@ platforms with a single provider. %prep %setup -q -chmod -x src/* libxkutil/* schema/* +chmod -x src/* libxkutil/* schema/* README doc/* base_schema/README* +chmod +X src/* libxkutil/* schema/* %build %configure --disable-werror @@ -40,6 +42,7 @@ rm -f $RPM_BUILD_ROOT%{_libdir}/*.a rm -f $RPM_BUILD_ROOT%{_libdir}/*.a rm -f $RPM_BUILD_ROOT%{_libdir}/cmpi/*.la rm -f $RPM_BUILD_ROOT%{_libdir}/cmpi/*.a +rm -f $RPM_BUILD_ROOT%{_libdir}/libxkutil.so %clean rm -fr $RPM_BUILD_ROOT @@ -57,22 +60,24 @@ rm -fr $RPM_BUILD_ROOT %{_datadir}/%{name}/install_base_schema.sh %{_datadir}/%{name} +/etc/init.d/tog-pegasus restart + %{_datadir}/%{name}/provider-register.sh -t pegasus \ -n /@CIM_VIRT_NS@ \ - -r %{REGISTRATION} -m %{SCHEMA} || true + -r %{REGISTRATION} -m %{SCHEMA} >/dev/null 2>&1 || true %preun %{_datadir}/%{name}/provider-register.sh -d -t pegasus \ -n /@CIM_VIRT_NS@ \ - -r %{REGISTRATION} -m %{SCHEMA} || true + -r %{REGISTRATION} -m %{SCHEMA} >/dev/null 2>&1 || true %postun -p /sbin/ldconfig %files %defattr(-, root, root) -%doc README doc/CodingStyle doc/SubmittingPatches -%doc provider-register.sh +%doc README COPYING doc/CodingStyle doc/SubmittingPatches +%doc base_schema/README.DMTF %doc doc/*.html %{_libdir}/lib*.so* %{_libdir}/cmpi/lib*.so*

On Wed, Dec 12, 2007 at 08:50:35AM -0700, Dan Smith wrote:
# HG changeset patch # User Dan Smith <danms@us.ibm.com> # Date 1197478221 28800 # Node ID de0e1bd9377b3e3325eec951ed0b0bf5100f565c # Parent c9edd5bc7202c0ee76bfba68aac98fd97ca87b34 RPM spec updates
- Restart the CIMOM after base schema installation - Installed file cleanups - Permissions cleanups - Require correct version of libvirt - Require unzip, since we need it for base schema unpacking - Require explicit verison of libxml2 (for rpmlint). I have no idea what version is actually required, so I took the F8 version. If someone has an opinion or complaint about this, I'm happy to change it.
just use something like libxml2 >= 2.6.0, there is no reason to break that build for example on a RHEL box because of a too tight libxml2 dep. [...]
+Requires: libxml2 >= 2.6.30
too tight IMHO :-) Daniel -- Red Hat Virtualization group http://redhat.com/virtualization/ Daniel Veillard | virtualization library http://libvirt.org/ veillard@redhat.com | libxml GNOME XML XSLT toolkit http://xmlsoft.org/ http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/

DV> just use something like libxml2 >= 2.6.0, there is no reason to DV> break that build for example on a RHEL box because of a too tight DV> libxml2 dep.
+Requires: libxml2 >= 2.6.30
DV> too tight IMHO :-) 'tis why I asked :) I'll change it. Thanks! -- Dan Smith IBM Linux Technology Center Open Hypervisor Team email: danms@us.ibm.com
participants (2)
-
Dan Smith
-
Daniel Veillard