Bug 507621 - Review Request: pypoppler - Python bindings for the Poppler PDF rendering library
Summary: Review Request: pypoppler - Python bindings for the Poppler PDF rendering lib...
Keywords:
Status: CLOSED DUPLICATE of bug 510962
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 497656 509484
TreeView+ depends on / blocked
 
Reported: 2009-06-23 14:12 UTC by Hicham HAOUARI
Modified: 2009-07-13 18:35 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-07-13 18:35:35 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Hicham HAOUARI 2009-06-23 14:12:37 UTC
Spec URL: http://rapidshare.com/files/247749721/pypoppler.spec.html
SRPM URL: http://rapidshare.com/files/247750206/pypoppler-0.10.0-1.fc11.src.rpm.html
Description: Python bindings for the Poppler PDF rendering library. It is needed to run programs written in Python and using Poppler set.

Comment 1 Hicham HAOUARI 2009-06-23 14:25:50 UTC
gedit-latex-plugin-0.2rc2-1 : New upstream rc version

this package depends on pypoppler package

Spec URL : http://rapidshare.com/files/247752065/gedit-latex-plugin.spec.html
SRPM URL : http://rapidshare.com/files/247754470/gedit-latex-plugin-0.2rc2-1.fc11.src.rpm.html

This new version have a nice preview pane for produced pdfs, which accelerate latex writing speed considerably

Comment 2 Jason Tibbitts 2009-07-01 21:00:27 UTC
Is this your first package for Fedora?  I don't see any account under your name in the account system, and I'd guess you aren't in the packager group because instead of fedoraproject.org space you're using rapidshare links which make it rather difficult to even take a look at your package.

Comment 3 Hicham HAOUARI 2009-07-02 13:30:26 UTC
Yes, it is right, I am not in the packager group, the procedure is just too long. I thought that my contributions are welcome, but anyway, I am setting up my own repo for people who want to test my packages.

And thanks for your gentle reply.

Comment 5 leigh scott 2009-07-02 17:05:23 UTC
You will need to remove the .la file 

https://fedoraproject.org/wiki/Packaging/Guidelines#Packaging_Static_Libraries



Also %{_libdir}/python2.6/site-packages is wrong

https://fedoraproject.org/wiki/Packaging/Python#System_Architecture


Here's an example

http://leigh123linux.fedorapeople.org/pub/SPECS/pypoppler.spec



P.S I'm not going to review your package as you need a sponsor.

Comment 6 Hicham HAOUARI 2009-07-02 17:30:46 UTC
Ok, I updated the spec file

Comment 7 Juan Manuel Rodriguez 2009-07-03 02:16:10 UTC
Hi Hicham, 

One of the first things you should do to any specfile, is run rpmlint on it. 

rpmlint will show any errors or warnings there are on your Specfile, like mixed use of spaces and tabs. 

rpmlint said: 
pypoppler.spec: W: mixed-use-of-spaces-and-tabs (spaces: line 3, tab: line 20)

Also, what Leigh said about the Static Libraries is right. 
On %configure, use --disable-static

and on %install, remove the la file, using: 
rm -rf $RPM_BUILD_ROOT%{python_sitelib}/*.la

I updated your specfile with these changes. 
http://proyectofedora.org/mexico/pypoppler.spec

Comment 8 Hicham HAOUARI 2009-07-03 02:37:03 UTC
Ok updated the spec file

Comment 9 Mamoru TASAKA 2009-07-03 15:10:37 UTC
Please post the full URLs of your spec/srpm files every
time you modified them. Also please make it sure that
you change the release number of the spec file to avoid
confusion.

Comment 11 Mamoru TASAKA 2009-07-13 07:04:27 UTC
*** Bug 510962 has been marked as a duplicate of this bug. ***

Comment 12 Fabian Affolter 2009-07-13 12:09:56 UTC
In Bug #510962 I made this package under the name of python-poppler and Jason reviewed and approved it.  But after that I renamed the package to pypoppler as Jason suggested and we run into this duplication.

Hicham, do you still want to go on with this package?

Comment 13 Hicham HAOUARI 2009-07-13 14:39:09 UTC
Hi Fabian,

No problem if u want to maintain it, what is important is that it must be in Fedora. I introduced it because I needed it to use the latest gedit-latex-plugin which displays a nice preview pane that requires pypoppler.

So my final answer is : if you want to take it, go ahead.

Comment 14 Fabian Affolter 2009-07-13 18:34:54 UTC
pypoppler is needed for pdfshuffler as well that was the reason because I made the spec file in April. 

Ok, I will reopen my review request and close this one.

Comment 15 Fabian Affolter 2009-07-13 18:35:35 UTC

*** This bug has been marked as a duplicate of bug 510962 ***


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