Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
|Summary:||Review Request: weasyprint - Utility and Python library to render HTML and CSS to PDF|
|Product:||[Fedora] Fedora||Reporter:||Eric Smith <spacewar>|
|Component:||Package Review||Assignee:||Christopher Meng <i>|
|Status:||CLOSED ERRATA||QA Contact:||Fedora Extras Quality Assurance <extras-qa>|
|Version:||rawhide||CC:||i, notting, package-review, susi.lehtola|
|Fixed In Version:||weasyprint-0.22-1.el7||Doc Type:||Bug Fix|
|Doc Text:||Story Points:||---|
|Last Closed:||2013-08-07 23:03:08 UTC||Type:||---|
|oVirt Team:||---||RHEL 7.3 requirements from Atomic Host:|
|Cloudforms Team:||---||Target Upstream Version:|
|Bug Depends On:||986630, 986634, 986715|
Description Eric Smith 2013-07-20 23:15:56 UTC
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
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.