Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: build PDF with Weasyprint 53 #725

Closed
wants to merge 1 commit into from
Closed

Conversation

dkg
Copy link
Contributor

@dkg dkg commented Feb 17, 2022

This patch is from @kitterma, i'm just forwarding it from https://trac.ietf.org/trac/xml2rfc/ticket/696 (it's also at https://bugs.debian.org/1001824)

Weasyprint 53 and later no longer use cairo and pango, so xml2rfc tests for PDF generation capability need updating. Also, it appears that test_text_ content is obsolete with the new Weasyprint. The test fails even though PDF generation succeeds.

On debian and debian-derived systems, the suggestions that remain in run.py when weasyprint isn't available are rather over-complicated,but debian can continue to patch in more streamlined suggestions for its distribution.

(originally at https://trac.ietf.org/trac/xml2rfc/ticket/696)
(also at https://bugs.debian.org/1001824)

Weasyprint 53 and later no longer use cairo and pango, so xml2rfc tests for
PDF generation capability need updating.  Also, it appears that test_text_
content is obsolete with the new Weasyprint.  The test fails even though
PDF generation succeeds.

On debian and debian-derived systems, the suggestions that remain in
run.py when weasyprint isn't available are rather over-complicated,
but debian can continue to patch in more streamlined suggestions for
its distribution.
@NGPixel NGPixel changed the title Build PDF with Weasyprint 53 fix: build PDF with Weasyprint 53 Feb 18, 2022
@dkg dkg mentioned this pull request Feb 22, 2022
@kesara
Copy link
Member

kesara commented May 9, 2022

@dkg, @kitterma, Thank you for the PR and apologies for taking long to reply.
Because this change can affect the PDF output, we will have to regenerate a few RFCs to compare and discuss with RPC.
I'll arrange this.

@kesara
Copy link
Member

kesara commented Jun 28, 2022

Closing this PR in favour of #802.
weasyprint 53+ changes required further changes to the test suite.
Thank you for your support.

@kesara kesara closed this Jun 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants