[libvirt] PATCH Fix BuildRequires in spec file

Fixes error from rpmbuild RPM build errors: File not found: /var/tmp/libvirt-0.6.3-1.el5-root/usr/libexec/libvirt_parthelper diff -u -r libvirt.spec.orig libvirt.spec --- libvirt.spec.orig 2009-04-24 08:58:29.000000000 -0500 +++ libvirt.spec 2009-05-29 09:57:56.000000000 -0500 @@ -59,7 +59,7 @@ Source: libvirt-%{version}.tar.gz BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root URL: http://libvirt.org/ -BuildRequires: python python-devel +BuildRequires: python python-devel e2fsprogs-devel Requires: libxml2 Requires: readline Requires: ncurses

On Fri, May 29, 2009 at 10:07:14AM -0500, Trent Johnson wrote:
Fixes error from rpmbuild
RPM build errors: File not found: /var/tmp/libvirt-0.6.3-1.el5-root/usr/libexec/libvirt_parthelper
NACk this program does not use e2fsprogs. It only uses parted. If you have a build problem without e2fsprogs installed, then the bug is in parted's RPM not libvirt's. I expect parted-devel, needs a dependancy on e2fsprogs-devel.
diff -u -r libvirt.spec.orig libvirt.spec --- libvirt.spec.orig 2009-04-24 08:58:29.000000000 -0500 +++ libvirt.spec 2009-05-29 09:57:56.000000000 -0500 @@ -59,7 +59,7 @@ Source: libvirt-%{version}.tar.gz BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root URL: http://libvirt.org/ -BuildRequires: python python-devel +BuildRequires: python python-devel e2fsprogs-devel Requires: libxml2 Requires: readline Requires: ncurses
Daniel -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|

The problem seems to be that when you build libvirt without e2fsprogs-devel installed, then ./configure for libvirt will detect with_storage_disk=no Once that happens libvirt will not create the libvirt_parthelper program. Not sure how this can be fixed then. Thanks, Trent Daniel P. Berrange wrote:
On Fri, May 29, 2009 at 10:07:14AM -0500, Trent Johnson wrote:
Fixes error from rpmbuild
RPM build errors: File not found: /var/tmp/libvirt-0.6.3-1.el5-root/usr/libexec/libvirt_parthelper
NACk this program does not use e2fsprogs. It only uses parted. If you have a build problem without e2fsprogs installed, then the bug is in parted's RPM not libvirt's. I expect parted-devel, needs a dependancy on e2fsprogs-devel.
diff -u -r libvirt.spec.orig libvirt.spec --- libvirt.spec.orig 2009-04-24 08:58:29.000000000 -0500 +++ libvirt.spec 2009-05-29 09:57:56.000000000 -0500 @@ -59,7 +59,7 @@ Source: libvirt-%{version}.tar.gz BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root URL: http://libvirt.org/ -BuildRequires: python python-devel +BuildRequires: python python-devel e2fsprogs-devel Requires: libxml2 Requires: readline Requires: ncurses
Daniel

On Mon, Jun 01, 2009 at 10:42:53AM -0500, Trent Johnson wrote:
The problem seems to be that when you build libvirt without e2fsprogs-devel installed, then ./configure for libvirt will detect with_storage_disk=no
Once that happens libvirt will not create the libvirt_parthelper program.
Not sure how this can be fixed then.
That doesn't need fixing in libvirt, that's intended. If the parted library fails to link, then we disable the parthelper tool
Daniel P. Berrange wrote:
On Fri, May 29, 2009 at 10:07:14AM -0500, Trent Johnson wrote:
Fixes error from rpmbuild
RPM build errors: File not found: /var/tmp/libvirt-0.6.3-1.el5-root/usr/libexec/libvirt_parthelper
NACk this program does not use e2fsprogs. It only uses parted. If you have a build problem without e2fsprogs installed, then the bug is in parted's RPM not libvirt's. I expect parted-devel, needs a dependancy on e2fsprogs-devel.
diff -u -r libvirt.spec.orig libvirt.spec --- libvirt.spec.orig 2009-04-24 08:58:29.000000000 -0500 +++ libvirt.spec 2009-05-29 09:57:56.000000000 -0500 @@ -59,7 +59,7 @@ Source: libvirt-%{version}.tar.gz BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root URL: http://libvirt.org/ -BuildRequires: python python-devel +BuildRequires: python python-devel e2fsprogs-devel Requires: libxml2 Requires: readline Requires: ncurses
Danielk -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|

Should there be some logic added to the spec file then, as it tries to package libvirt_parthelper even it if was not built. Daniel P. Berrange wrote:
On Mon, Jun 01, 2009 at 10:42:53AM -0500, Trent Johnson wrote:
The problem seems to be that when you build libvirt without e2fsprogs-devel installed, then ./configure for libvirt will detect with_storage_disk=no
Once that happens libvirt will not create the libvirt_parthelper program.
Not sure how this can be fixed then.
That doesn't need fixing in libvirt, that's intended. If the parted library fails to link, then we disable the parthelper tool
Daniel P. Berrange wrote:
Fixes error from rpmbuild
RPM build errors: File not found: /var/tmp/libvirt-0.6.3-1.el5-root/usr/libexec/libvirt_parthelper NACk this program does not use e2fsprogs. It only uses parted. If you have a build problem without e2fsprogs installed, then the bug is in
On Fri, May 29, 2009 at 10:07:14AM -0500, Trent Johnson wrote: parted's RPM not libvirt's. I expect parted-devel, needs a dependancy on e2fsprogs-devel.
diff -u -r libvirt.spec.orig libvirt.spec --- libvirt.spec.orig 2009-04-24 08:58:29.000000000 -0500 +++ libvirt.spec 2009-05-29 09:57:56.000000000 -0500 @@ -59,7 +59,7 @@ Source: libvirt-%{version}.tar.gz BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root URL: http://libvirt.org/ -BuildRequires: python python-devel +BuildRequires: python python-devel e2fsprogs-devel Requires: libxml2 Requires: readline Requires: ncurses
Danielk
participants (2)
-
Daniel P. Berrange
-
Trent Johnson