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

Warn broken cairo version #423

Merged
merged 1 commit into from Feb 7, 2017
Merged

Warn broken cairo version #423

merged 1 commit into from Feb 7, 2017

Conversation

andreas-p
Copy link

Since the broken cairo version is all around (e.g. Debian Jessie), IMHO a warning might help other users.

Copy link
Member

@liZe liZe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good idea, thanks for the PR.

I'd prefer to avoid warnings at the import of the module, because they sometimes appear at the wrong time (for example when you launch weasyprint --help). It may be better in Document._render, what do you think?

@andreas-p
Copy link
Author

I see your point, but the Pango>1.38 warning appears at import time of fonts.py as well. Putting it in Document._render would have it appear possibly multiple times.

@liZe
Copy link
Member

liZe commented Feb 7, 2017

I see your point, but the Pango>1.38 warning appears at import time of fonts.py as well.

True (and I did that 😨).

Putting it in Document._render would have it appear possibly multiple times.

I think that it's not a bad thing.

So let's merge, I'll try to find a better solution before releasing the next version.

@liZe liZe merged commit 11bf0c7 into Kozea:master Feb 7, 2017
@liZe liZe added this to the 0.34 milestone Feb 7, 2017
jsonn pushed a commit to jsonn/pkgsrc that referenced this pull request Mar 3, 2017
Version 0.36
------------

Released on 2017-02-25.

New features:

* `#407 <https://github.com/Kozea/WeasyPrint/pull/407>`_:
  Handle ::first-letter.
* `#423 <https://github.com/Kozea/WeasyPrint/pull/423>`_:
  Warn user about broken cairo versions.

Bug fixes:

* `#411 <https://github.com/Kozea/WeasyPrint/pull/411>`_:
  Typos fixed in command-line help.


Version 0.35
------------

Released on 2017-02-25.

Bug fixes:

* `#410 <https://github.com/Kozea/WeasyPrint/pull/410>`_:
  Fix AssertionError in split_text_box.
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.

None yet

2 participants