Bug 522621

Summary:	BuildRequires: cairo-devel needs to be versioned (higher)
Product:	[Fedora] Fedora	Reporter:	Moritz Barsnick <moritz>
Component:	poppler	Assignee:	Kristian Høgsberg <krh>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	low	Docs Contact:
Priority:	low
Version:	rawhide	CC:	jnovy, krh, rdieter
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	All
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2009-09-11 18:06:17 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Moritz Barsnick 2009-09-10 20:38:28 UTC

Description of problem:
poppler SRPM can't be rebuilt on F10 due to cairo-devel version mismatch, but BuildRequires fails to catch this.

Version-Release number of selected component (if applicable):
poppler-0.11.3-1

How reproducible:
on F10 (or any distro/version with cairo-devel < 1.8.4)

Steps to Reproduce:
1. rpmbuild --rebuild poppler-0.11.3-1.fc12.src.rpm
  
Actual results:
./configure fails due to cairo-devel 1.8.0 instead of >= 1.8.4 available.

Expected results:
rpm notices that cairo-devel >= 1.8.4 is required to build this package.

Additional info:
While I guess poppler-0.11/0.12 may not be planned for F10, this issue is still easily caught. I realize that policy is (officially?) that Requires/BuildRequires need not be versioned if the supported distributions fulfil that version anyway. Not sure whether F10 counts, or doesn't if this version isn't planned.

I also don't understand why F10 doesn't have a newer cairo. *shrug*

So I suggest this change to poppler.spec:
-BuildRequires: cairo-devel
+BuildRequires: cairo-devel >= 1.8.4
+Requires: cairo >= 1.8.4

Oh, and please bump Rawhide to 0.12.0. :)

Comment 1 Moritz Barsnick 2009-09-10 20:39:32 UTC

(Changed Summary, I had pressed return too early.)

Comment 2 Rex Dieter 2009-09-11 18:06:17 UTC

I'll add the BuildRequires at least, the explicit Requires, no.

cvs only, no rebuild for this.

p.s. poppler-0.12.0 was built for rawhide on 9/9