spec: fix Requires/BuildRequires on git

For a long time we use 'dirxz' as ARCHIVE_FORMAT (we distribute
archive.dir.tar.xz) in fedora package.  That's been this way
because 'xz' is historically installed by default in minimal
buildroot and because that's the preferred distribution method
(smallest size of the archive file).

Because 'dirxz' is preferred anyway, remove the redundant
--disable-git %configure option.  But also to avoid surprises in
future, let's make the 'dirxz' explicit by having 'BuildRequires:
xz' and using '--enable-xz' configure option.

As it was just a redundant dependency, from now on we don't depend
on 'git'.

Related: rhbz#1161284, rhbz#1416691
Version: 0.19.8.1-7
This commit is contained in:
Pavel Raiskup 2017-01-26 12:55:53 +01:00
parent 63036c0a24
commit 83c226dd02

View File

@ -1,7 +1,6 @@
%bcond_with jar
%bcond_with java
%bcond_without check
%bcond_without git
%global tarversion 0.19.8.1
%global archiveversion 0.19.8
@ -9,7 +8,7 @@
Summary: GNU libraries and utilities for producing multi-lingual messages
Name: gettext
Version: 0.19.8.1
Release: 6%{?dist}
Release: 7%{?dist}
License: GPLv3+ and LGPLv2+
Group: Development/Tools
URL: http://www.gnu.org/software/gettext/
@ -41,10 +40,10 @@ BuildRequires: zip, unzip
%endif
# for po-mode.el
BuildRequires: emacs
%if %{with git}
# for autopoint:
# for autosetup
BuildRequires: git
%endif
# ensure 'ARCHIVE_FORMAT=dirxz'
BuildRequires: xz
BuildRequires: chrpath
# following suggested by DEPENDENCIES:
BuildRequires: ncurses-devel
@ -96,10 +95,6 @@ Requires: %{name}-libs = %{version}-%{release}
Requires: %{name}-common-devel = %{version}-%{release}
Requires(post): info
Requires(preun): info
%if %{with git}
# for autopoint
Requires: git
%endif
Requires: xz
Obsoletes: gettext-autopoint < 0.18.1.1-3
Provides: gettext-autopoint = %{version}-%{release}
@ -167,10 +162,7 @@ export CFLAGS="$RPM_OPT_FLAGS -D__SUPPORT_SNAN__"
%else
--disable-java --disable-native-java \
%endif
%if %{without git}
--disable-git \
%endif
%{nil}
--with-xz
make %{?_smp_mflags} %{?with_java:GCJFLAGS="-findirect-dispatch"}
@ -358,6 +350,10 @@ fi
%{_mandir}/man1/msghack.1*
%changelog
* Thu Jan 26 2017 Pavel Raiskup <praiskup@redhat.com> - 0.19.8.1-7
- really remove Requires: git from gettext-devel (rhbz#1161284)
- make the BuildRequires unconditional (rhbz#1416691)
* Wed Dec 21 2016 Pavel Raiskup <praiskup@redhat.com> - 0.19.8.1-6
- disable test-lock for 'gettext-tool' subdir too (rhbz#1406031)