Bug 986608 - Review Request: weasyprint - Utility and Python library to render HTML and CSS to PDF
Summary: Review Request: weasyprint - Utility and Python library to render HTML and CS...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Christopher Meng
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 986630 986634 986715
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-07-20 23:15 UTC by Eric Smith
Modified: 2014-09-24 03:49 UTC (History)
4 users (show)

Fixed In Version: weasyprint-0.22-1.el7
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-08-07 23:03:08 UTC
Type: ---
Embargoed:
i: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Eric Smith 2013-07-20 23:15:56 UTC
Spec URL: http://fedorapeople.org/~brouhaha/weasyprint/weasyprint.spec
SRPM URL: http://fedorapeople.org/~brouhaha/weasyprint/weasyprint-0.19.2-1.fc19.src.rpm
Description: WeasyPrint can render HTML and CSS to PDF. It aims to support web standards for printing. Both a command line tool and a Python library are provided.
Fedora Account System Username: brouhaha

Comment 1 Eric Smith 2013-07-21 01:51:38 UTC
I'm closing this review request, because there are some missing (unpackaged) dependencies, which I didn't notice because somehow I had them on my machine.  I'll work on packaging them first, then reopen this review request.

Thanks!
Eric

Comment 2 Christopher Meng 2013-07-21 02:53:40 UTC
Hi, 

Don't close a bug unless it's stalled or invalid. 

Please package the dependencies and use "depends on" to mark them. 

Anyway, I'll leave this bug as NotReady in whiteboard.

Comment 3 Eric Smith 2013-07-21 21:12:10 UTC
I closed it because I didn't think it likely that I'd be able to package the missing dependencies any time soon, so I considered it stalled.  However, as it happens, I actually have been able to get the missing dependencies packaged and submitted for review.

Spec URL: http://fedorapeople.org/~brouhaha/weasyprint/weasyprint.spec
SRPM URL: http://fedorapeople.org/~brouhaha/weasyprint/weasyprint-0.19.2-2.fc19.src.rpm

Depends on copending reviews of:

python-cairocffi  #986715
python-pyphen     #986634
python-tinycss    #986630

and indirectly on python-cffi and python-pycparser.

It is not clear to me from the Fedora Package Review Process document whether I should remove the "NotReady" from the Whiteboard if the package to be reviewed is dependent on other packages which have not yet been reviewed.

Comment 4 Susi Lehtola 2013-07-21 21:15:23 UTC
(In reply to Eric Smith from comment #3)
> It is not clear to me from the Fedora Package Review Process document
> whether I should remove the "NotReady" from the Whiteboard if the package to
> be reviewed is dependent on other packages which have not yet been reviewed.

You can clear NotReady since the package can be reviewed and imported even if the requisites aren't yet included in Fedora. The thing is that you just can't build the package before the requisites are there.

Comment 5 Eric Smith 2013-07-26 21:37:50 UTC
All the prerequisites have been approved and submitted for updates, there are koji buildroot overrides, and koji scratch builds have succeeded for rawhide, f19, f18, and epel6.

Comment 6 Christopher Meng 2013-07-27 09:35:52 UTC
cairocffi is absent.

I'll handle this in the next week.

Comment 7 Eric Smith 2013-07-27 14:03:47 UTC
Absent? The python-cairocffi update is submitted and there is a koji build root override in place, so weasyprint builds successfully in koji.

Next week is fine. Thanks for reviewing!
Eric

Comment 8 Christopher Meng 2013-07-29 00:44:02 UTC
Package is fine, only one question:

tests/test_web/run.py:#!/usr/bin/env python

In the past I think we shouldn't let env works, we should use /usr/bin/python. But can you tell me if this time is OK?

Comment 9 Eric Smith 2013-07-29 01:21:11 UTC
Spec URL: http://fedorapeople.org/~brouhaha/weasyprint/weasyprint.spec
SRPM URL: http://fedorapeople.org/~brouhaha/weasyprint/weasyprint-0.19.2-3.fc19.src.rpm

I'm not sure whether the policy on /usr/bin/env was actually adopted, but I've updated the spec to eliminate both uses of it.  Thanks!

Comment 10 Christopher Meng 2013-07-29 01:22:54 UTC
APPROVED.

Comment 11 Eric Smith 2013-07-29 01:34:11 UTC
New Package SCM Request
=======================
Package Name: weasyprint
Short Description: Utility and Python library to render HTML and CSS to PDF
Owners: brouhaha
Branches: f18 f19 el6
InitialCC:

Comment 12 Gwyn Ciesla 2013-07-29 12:12:28 UTC
Git done (by process-git-requests).

Comment 13 Fedora Update System 2013-07-29 16:08:16 UTC
weasyprint-0.19.2-3.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/weasyprint-0.19.2-3.fc19

Comment 14 Fedora Update System 2013-07-29 16:15:32 UTC
weasyprint-0.19.2-3.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/weasyprint-0.19.2-3.fc18

Comment 15 Fedora Update System 2013-07-29 16:26:43 UTC
weasyprint-0.19.2-3.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/weasyprint-0.19.2-3.el6

Comment 16 Fedora Update System 2013-07-30 17:45:54 UTC
weasyprint-0.19.2-3.fc19 has been pushed to the Fedora 19 testing repository.

Comment 17 Fedora Update System 2013-08-07 23:03:08 UTC
weasyprint-0.19.2-3.fc19 has been pushed to the Fedora 19 stable repository.

Comment 18 Fedora Update System 2013-08-07 23:09:27 UTC
weasyprint-0.19.2-3.fc18 has been pushed to the Fedora 18 stable repository.

Comment 19 Fedora Update System 2013-08-16 19:51:18 UTC
weasyprint-0.19.2-3.el6 has been pushed to the Fedora EPEL 6 stable repository.

Comment 20 Eric Smith 2014-08-21 03:59:01 UTC
Package Change Request
======================
Package Name: weasyprint
New Branches: epel7
Owners: brouhaha

Comment 21 Gwyn Ciesla 2014-08-21 11:51:33 UTC
Git done (by process-git-requests).

Comment 22 Fedora Update System 2014-08-30 02:02:10 UTC
weasyprint-0.22-1.el7 has been submitted as an update for Fedora EPEL 7.
https://admin.fedoraproject.org/updates/weasyprint-0.22-1.el7

Comment 23 Fedora Update System 2014-09-24 03:49:27 UTC
weasyprint-0.22-1.el7 has been pushed to the Fedora EPEL 7 stable repository.


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