Re: [PATCH] [For 1.2.15] spec: fixes for recently introduced libxl conf files

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



 Jim,

since the default build is with driver modules, I went ahead and made
the release, this can be carried in the stabilization branches, and
the next release is only one month away after all :-)

Daniel

On Fri, May 01, 2015 at 02:27:26PM -0600, Jim Fehlig wrote:
> Slight subject tweak to highlight a patch that might be a good candidate
> for 1.2.15.
> 
> Regards,
> Jim
> 
> Jim Fehlig wrote:
> > Commit 198cc1d3 introduced libxl-lockd and libxl-sanlock config
> > files but forgot to add them to the spec file.  Follow-up commit
> > 62b18d98 added the files to daemon-driver-libxl, but missed adding
> > them to the daemon package when configuring libvirt
> > --without-driver-modules.  In addition, commit 62b18d98 added
> > libxl-sanlock to daemon-driver-libxl, but it should be included
> > in lock-sanlock when libvirt is configured --with-sanlock.
> >
> > Signed-off-by: Jim Fehlig <jfehlig@xxxxxxxx>
> > ---
> >
> > Given the problems I've caused with the original commit 198cc1d3,
> > I'm reluctant to push this as a build-breaker without another
> > set of eyes reviewing it.
> >
> >  libvirt.spec.in | 5 ++++-
> >  1 file changed, 4 insertions(+), 1 deletion(-)
> >
> > diff --git a/libvirt.spec.in b/libvirt.spec.in
> > index f95e31f..0c343d6 100644
> > --- a/libvirt.spec.in
> > +++ b/libvirt.spec.in
> > @@ -2007,6 +2007,7 @@ exit 0
> >          %endif
> >          %if %{with_libxl}
> >  %config(noreplace) %{_sysconfdir}/libvirt/libxl.conf
> > +%config(noreplace) %{_sysconfdir}/libvirt/libxl-lockd.conf
> >  %dir %attr(0700, root, root) %{_localstatedir}/log/libvirt/libxl/
> >  %ghost %dir %{_localstatedir}/run/libvirt/libxl/
> >  %dir %attr(0700, root, root) %{_localstatedir}/lib/libvirt/libxl/
> > @@ -2130,7 +2131,6 @@ exit 0
> >  %defattr(-, root, root)
> >  %config(noreplace) %{_sysconfdir}/libvirt/libxl.conf
> >  %config(noreplace) %{_sysconfdir}/libvirt/libxl-lockd.conf
> > -%config(noreplace) %{_sysconfdir}/libvirt/libxl-sanlock.conf
> >  %{_datadir}/augeas/lenses/libvirtd_libxl.aug
> >  %{_datadir}/augeas/lenses/tests/test_libvirtd_libxl.aug
> >  %dir %attr(0700, root, root) %{_localstatedir}/log/libvirt/libxl/
> > @@ -2183,6 +2183,9 @@ exit 0
> >      %if %{with_qemu}
> >  %config(noreplace) %{_sysconfdir}/libvirt/qemu-sanlock.conf
> >      %endif
> > +    %if %{with_libxl}
> > +%config(noreplace) %{_sysconfdir}/libvirt/libxl-sanlock.conf
> > +    %endif
> >  %attr(0755, root, root) %{_libdir}/libvirt/lock-driver/sanlock.so
> >  %{_datadir}/augeas/lenses/libvirt_sanlock.aug
> >  %{_datadir}/augeas/lenses/tests/test_libvirt_sanlock.aug
> >   
> 
> --
> libvir-list mailing list
> libvir-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/libvir-list

-- 
Daniel Veillard      | Open Source and Standards, Red Hat
veillard@xxxxxxxxxx  | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | virtualization library  http://libvirt.org/

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]