Re: [PATCH] libvirt.spec.in: BuildRequires: python-devel

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

 



On 02.12.2014 10:09, Jiri Denemark wrote:
On Tue, Dec 02, 2014 at 09:11:56 +0100, Michal Privoznik wrote:
On 01.12.2014 22:32, Eric Blake wrote:
On 12/01/2014 09:21 AM, Michal Privoznik wrote:
As of f48ab7d8 we are trying to use python-devel to check python's
version (or at least gnulib is doing that). However, the specfile
doesn't say anything about it and therefore bootstrapping may fail
on a minimal installation on rawhide.

What's the failure?  The whole point of f48ab7d8 is that the use of
'python-config' is optional, and should not be fatal if it is not found.
   Requiring it in the specfile feels like a step backwards, and probably
breaks on RHEL 5, where we still want 'make rpm' to work.

Do we?

[zippy@localhost libvirt.git]$ cat /etc/fedora-release
Fedora release 22 (Rawhide)
[zippy@localhost libvirt.git]$ rpm -q python-devel
package python-devel is not installed
[zippy@localhost libvirt.git]$ git clean -fxd; ./autogen.sh --system
Removing AUTHORS
Removing ChangeLog
Running ./configure with --prefix=/usr --sysconfdir=/etc
--localstatedir=/var --libdir=/usr/lib64
running bootstrap...
./bootstrap: Error: 'python-config' not found

./bootstrap: Please install the prerequisite programs
Failed to bootstrap, please investigate.
[zippy@localhost libvirt.git]$

But requiring python-devel in libvirt.spec is not going to help you with
this in any way. RPMs can be built only after you have bootstrapped
libvirt. And I guess at that point, python-devel is no longer required
or is it?

I think it's going to help. Let me show you how I install dependencies for libvirt. I download the latest src.rpm from koji web, and yum-builddep it. But that - for some reason - doesn't install all the necessary packages. So I need to go through couple of iterations of 'autogen.sh && make' just to install missing dependencies. Honestly, I don't see any downside of having BuildRequires for additional packages that are required to build from git. Upside is much easier dependency installation. We are already doing that btw: the section I'm adding the dependency to already contains libtool, autoconf, automake, etc. Packages that are not needed when building from released tar.gz rather than when building from git.

Michal

--
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]