Re: [libvirt] [PATCH 1/n] spec: fix rpm build when lxc disabled

2013-09-04 Thread Michal Privoznik
On 03.09.2013 23:27, Eric Blake wrote:
 'make rpm' failed if ~/.rpmmacros contains '%_without_lxc 1',
 which simulates the case of not having lxc available.
 
 RPM build errors:
 File not found: 
 /home/eblake/rpmbuild/BUILDROOT/libvirt-1.1.1-1.fc19.x86_64/etc/libvirt/virt-login-shell.conf
 File not found by glob: 
 /home/eblake/rpmbuild/BUILDROOT/libvirt-1.1.1-1.fc19.x86_64/usr/share/man/man1/virt-login-shell.1*
 File not found: 
 /home/eblake/rpmbuild/BUILDROOT/libvirt-1.1.1-1.fc19.x86_64/usr/bin/virt-login-shell
 make: *** [rpm] Error 1
 
 Reported by Dan Berrange.
 
 * libvirt.spec.in: Mark virt-login-shell as conditional on lxc.
 
 Signed-off-by: Eric Blake ebl...@redhat.com
 ---
 
 I'm working on a couple more that I spotted while in the area:
 we still unconditionally build vbox even though we recently
 moved it into libvirtd for licensing reasons, which means a
 client_only build will probably fall over on that; but while
 testing my hypothesis, I discoverd that a client_only build
 fails even earlier with:
 
 TEST: fdstreamtest
  1) Stream read blocking  ... OK
  2) Stream read non-blocking  ... 
 Unexpected EOF block 0 want 128
 FAILED
  3) Stream write blocking ... OK
  4) Stream write non-blocking ... 
 Failed to finish stream: internal error: libvirt:  error : cannot execute 
 binary 
 /home/eblake/rpmbuild/BUILD/libvirt-1.1.1/tests/../src/libvirt_iohelper: No 
 such file or directory
 
  libvirt.spec.in | 6 ++
  1 file changed, 6 insertions(+)

ACK

Michal

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


Re: [libvirt] [PATCH 1/n] spec: fix rpm build when lxc disabled

2013-09-04 Thread Eric Blake
On 09/04/2013 06:38 AM, Michal Privoznik wrote:
 On 03.09.2013 23:27, Eric Blake wrote:
 'make rpm' failed if ~/.rpmmacros contains '%_without_lxc 1',
 which simulates the case of not having lxc available.

 RPM build errors:
 File not found: 
 /home/eblake/rpmbuild/BUILDROOT/libvirt-1.1.1-1.fc19.x86_64/etc/libvirt/virt-login-shell.conf
 File not found by glob: 
 /home/eblake/rpmbuild/BUILDROOT/libvirt-1.1.1-1.fc19.x86_64/usr/share/man/man1/virt-login-shell.1*
 File not found: 
 /home/eblake/rpmbuild/BUILDROOT/libvirt-1.1.1-1.fc19.x86_64/usr/bin/virt-login-shell
 make: *** [rpm] Error 1

 Reported by Dan Berrange.

 * libvirt.spec.in: Mark virt-login-shell as conditional on lxc.


 
 ACK

Thanks; pushed.

-- 
Eric Blake   eblake redhat com+1-919-301-3266
Libvirt virtualization library http://libvirt.org



signature.asc
Description: OpenPGP digital signature
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH 1/n] spec: fix rpm build when lxc disabled

2013-09-03 Thread Eric Blake
'make rpm' failed if ~/.rpmmacros contains '%_without_lxc 1',
which simulates the case of not having lxc available.

RPM build errors:
File not found: 
/home/eblake/rpmbuild/BUILDROOT/libvirt-1.1.1-1.fc19.x86_64/etc/libvirt/virt-login-shell.conf
File not found by glob: 
/home/eblake/rpmbuild/BUILDROOT/libvirt-1.1.1-1.fc19.x86_64/usr/share/man/man1/virt-login-shell.1*
File not found: 
/home/eblake/rpmbuild/BUILDROOT/libvirt-1.1.1-1.fc19.x86_64/usr/bin/virt-login-shell
make: *** [rpm] Error 1

Reported by Dan Berrange.

* libvirt.spec.in: Mark virt-login-shell as conditional on lxc.

Signed-off-by: Eric Blake ebl...@redhat.com
---

I'm working on a couple more that I spotted while in the area:
we still unconditionally build vbox even though we recently
moved it into libvirtd for licensing reasons, which means a
client_only build will probably fall over on that; but while
testing my hypothesis, I discoverd that a client_only build
fails even earlier with:

TEST: fdstreamtest
 1) Stream read blocking  ... OK
 2) Stream read non-blocking  ... 
Unexpected EOF block 0 want 128
FAILED
 3) Stream write blocking ... OK
 4) Stream write non-blocking ... 
Failed to finish stream: internal error: libvirt:  error : cannot execute 
binary /home/eblake/rpmbuild/BUILD/libvirt-1.1.1/tests/../src/libvirt_iohelper: 
No such file or directory

 libvirt.spec.in | 6 ++
 1 file changed, 6 insertions(+)

diff --git a/libvirt.spec.in b/libvirt.spec.in
index e94901a..affd2ec 100644
--- a/libvirt.spec.in
+++ b/libvirt.spec.in
@@ -2014,17 +2014,23 @@ fi
 %doc AUTHORS ChangeLog.gz NEWS README COPYING COPYING.LESSER TODO

 %config(noreplace) %{_sysconfdir}/libvirt/libvirt.conf
+%if %{with_lxc}
 %config(noreplace) %{_sysconfdir}/libvirt/virt-login-shell.conf
+%endif
 %{_mandir}/man1/virsh.1*
 %{_mandir}/man1/virt-xml-validate.1*
 %{_mandir}/man1/virt-pki-validate.1*
 %{_mandir}/man1/virt-host-validate.1*
+%if %{with_lxc}
 %{_mandir}/man1/virt-login-shell.1*
+%endif
 %{_bindir}/virsh
 %{_bindir}/virt-xml-validate
 %{_bindir}/virt-pki-validate
 %{_bindir}/virt-host-validate
+%if %{with_lxc}
 %attr(4755, root, root) %{_bindir}/virt-login-shell
+%endif
 %{_libdir}/lib*.so.*

 %if %{with_dtrace}
-- 
1.8.3.1

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