[libvirt] [PATCH 1/3] spec: always autoreconf when building rpm
Daniel P. Berrange
berrange at redhat.com
Mon Dec 5 17:51:59 UTC 2011
On Mon, Dec 05, 2011 at 10:38:49AM -0700, Eric Blake wrote:
> Over time, Fedora and RHEL RPMs have often backported upstream
> patches that touched configure.ac and/or Makefile.am; this
> necessitates rerunning the autotools for the patch to be effective.
> Making this part of the spec file will make it easier for future
> backports to pull patches without thinking about this issue.
>
> * libvirt.spec.in (BuildRequires): Add autotools.
> (%build): Use them before configure.
> ---
> libvirt.spec.in | 4 ++++
> 1 files changed, 4 insertions(+), 0 deletions(-)
>
> diff --git a/libvirt.spec.in b/libvirt.spec.in
> index 06c949b..462421a 100644
> --- a/libvirt.spec.in
> +++ b/libvirt.spec.in
> @@ -334,6 +334,9 @@ Requires: libcgroup
> %endif
>
> # All build-time requirements
> +BuildRequires: autoconf
> +BuildRequires: automake
> +BuildRequires: libtool
> BuildRequires: python-devel
> %if %{with_systemd}
> BuildRequires: systemd-units
> @@ -721,6 +724,7 @@ of recent versions of Linux (and other OSes).
> %define init_scripts --with-init_script=redhat
> %endif
>
> +autoreconf -if
> %configure %{?_without_xen} \
> %{?_without_qemu} \
> %{?_without_openvz} \
NACK, we really shouldn't do this by default IMHO - regenerating autotools
has not always been foolproof when newer autotools are released. If we want
to include this for help of downstream, then it should be protected by a
conditional statement, so it is off by default and you can set
%define enable_autoconf 1
at the top of the spec to turn it on.
Regards,
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
More information about the libvir-list
mailing list