Bug 218768 - Review Request: poppler-extras - PDF rendering library extras (qt/qt4)
Summary: Review Request: poppler-extras - PDF rendering library extras (qt/qt4)
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Package Reviews List
URL:
Whiteboard:
Depends On:
Blocks: FE-DEADREVIEW
TreeView+ depends on / blocked
 
Reported: 2006-12-07 14:14 UTC by Rex Dieter
Modified: 2007-11-30 22:11 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-12-12 02:53:30 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Rex Dieter 2006-12-07 14:14:56 UTC
Spec URL: http://kde-redhat.unl.edu/apt/kde-redhat/SPECS/poppler-extras.spec
SRPM URL: http://kde-redhat.unl.edu/apt/kde-redhat/all/SRPMS.stable/poppler-extras-0.5.4-1.src.rpm
Description:
PDF rendering library extras, including qt and qt4 wrappers.

This is my first quick-n-dirty attempt at building the qt/qt4 poppler wrappers.  I've done some minimal testing of the -qt wrapper with kdegraphics' kpdf, and it *seems* to work. (:

Comment 1 Bernard Johnson 2006-12-08 09:15:31 UTC
Rex- 

> # yes, this file is owned by 2 packages, deal.  -- Rex

http://fedoraproject.org/wiki/Packaging/Guidelines?highlight=%28Packaging%29#head-a5931a7372c4a00065713430984fa5875513e6d4
Packages must not own files already owned by other packages. The rule of thumb
here is that the first package to be installed should own the files that other
packages may rely upon. This means, for example, that no package in Fedora
should ever share ownership with any of the files owned by the filesystem or man
package. If you feel that you have a good reason to own a file or that another
package owns, then please present that at package review time.

I think this is meant more to address having entirely different packages owning
the same file, rather than a collection of related packages sharing a file, but
I thought I'd get your take on it.

Comment 2 Rex Dieter 2006-12-08 12:59:12 UTC
Yes, I'm aware of the guideline.  This is a reasonable exception, which is why I
added the pithy comment. (:

Hold off on reviewing this... the poppler maintainer seems agreeable to adding
-qt to Core (see bug #158941 comment #11), so I'll probably be modifying this
(soon) to include only the -qt4 wrapper.

Comment 3 Bernard Johnson 2006-12-08 21:10:03 UTC
Ok, I will review it for you after it's modified.

Comment 4 Rex Dieter 2006-12-12 02:53:30 UTC
Since the qt bindings will soon be included in the Core package, and AFAIK, 
nothing currenty needs/uses the qt4 bindings (yet), I withdraw this Review 
request.

Comment 5 Laurent Rineau 2007-10-10 20:55:11 UTC
Anybody knows anything about poppler-qt4 in Fedora? Current poppler package in 
Rawhide does not seem to be build with Qt4 support (and I would love to test 
Okular...).


Comment 6 Rex Dieter 2007-10-11 14:09:59 UTC
Laurent, see bug #327691


Note You need to log in before you can comment on or make changes to this bug.