Bug 1251766
| Summary: | texinfo does not require perl-Storable | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Cole Robinson <crobinso> |
| Component: | texinfo | Assignee: | Vitezslav Crhonek <vcrhonek> |
| Status: | CLOSED RAWHIDE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | unspecified | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | rawhide | CC: | cweyl, emmanuel, iarnell, jplesnik, kasal, nmavrogi, orion, perl-devel, pertusus, ppisar, psabata, rc040203, tcallawa, vcrhonek |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | texinfo-6.0-2.fc24 | Doc Type: | Bug Fix |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2015-08-10 11:00:18 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Cole Robinson
2015-08-09 19:07:19 UTC
(In reply to Cole Robinson from comment #0) > > Previously it looks like 'perl' would pull in perl-Storable, but not > anymore. Correct. This is as of perl-5.22.0-350.fc24. > Though maybe the dep should be in texinfo or something... You are correct (rebuilding the current texinfo srpm fails with the same error message). texinfo is incorrectly filtering out the perl(Storable) requires. My proposed fix:
diff --git a/texinfo.spec b/texinfo.spec
index f40d7cd..428cdc2 100644
--- a/texinfo.spec
+++ b/texinfo.spec
@@ -1,21 +1,15 @@
-%global _use_internal_dependency_generator 0
-
%global tex_texinfo %{_datadir}/texmf/tex/texinfo
Summary: Tools needed to create Texinfo format documentation files
Name: texinfo
Version: 6.0
-Release: 1%{?dist}
+Release: 2%{?dist}
License: GPLv3+
Group: Applications/Publishing
Url: http://www.gnu.org/software/texinfo/
Source0: ftp://ftp.gnu.org/gnu/texinfo/texinfo-%{version}.tar.xz
Source1: ftp://ftp.gnu.org/gnu/texinfo/texinfo-%{version}.tar.xz.sig
Source2: info-dir
-# Source3: script for filtering out false perl requires
-Source3: filter-requires-texinfo.sh
-# Source4: script for filtering out false perl provides
-Source4: filter-provides-texinfo.sh
# Source5: macro definitions
Source5: macros.info
Patch0: texinfo-4.12-zlib.patch
@@ -26,9 +20,11 @@ Requires: perl >= 5.7.3, perl(Text::Unidecode)
Requires: perl(Unicode::EastAsianWidth), perl(Data::Dumper), perl(Locale::Messages)
BuildRequires: zlib-devel, ncurses-devel, help2man, perl(Data::Dumper)
BuildRequires: perl(Locale::Messages), perl(Unicode::EastAsianWidth), perl(Text::Unidecode)
+BuildRequires: perl(Storable)
-%global __find_requires %{SOURCE3}
-%global __find_provides %{SOURCE4}
+# Texinfo perl packages are not installed in default perl library dirs
+%global __provides_exclude ^perl\\(.*Texinfo.*\\)$
+%global __requires_exclude ^perl\\(.*Texinfo.*\\)$
%description
Texinfo is a documentation system that can produce both online
@@ -170,6 +166,9 @@ fi
%{_mandir}/man1/pdftexi2dvi.1*
%changelog
+* Sun Aug 9 2015 Orion Poplawski <orion.com> - 6.0-2
+- Add BR on perl(Storable), fix perl requires (bug #1251766)
+
* Tue Jul 14 2015 Vitezslav Crhonek <vcrhonek> - 6.0-1
- Update to texinfo-6.0
Resolves: #1236254
*** Bug 1251876 has been marked as a duplicate of this bug. *** Applied, thanks! |