This repository has been archived by the owner. It is now read-only.

Fix for matplotlib and freetype 2.5.1 #65

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
4 participants
Contributor

claudiodsf commented Dec 5, 2013

Fixes #62.

Adds the upstream patch: matplotlib/matplotlib#2623

Let me know.

Contributor

claudiodsf commented Dec 10, 2013

Freetype has just been updated to 2.5.2 (Homebrew/legacy-homebrew@a9fbe6d).

This patched formula is still required and it compiles correctly.

Contributor

claudiodsf commented Dec 10, 2013

Pinging @samueljohn for comments.

zittix commented Jan 2, 2014

Note that installing with --HEAD works without this patch.

Contributor

claudiodsf commented Jan 3, 2014

@zittix You're right. The patch comes indeed from HEAD. I modified the formula accordingly.

Let's hope that some maintainer gives a look at this PR...

Contributor

claudiodsf commented Jan 21, 2014

Pinging again @samueljohn and @MikeMcQuaid on this.
Please note that matplotlib does not compile without this patch.
See also Homebrew/homebrew-versions#302

Owner

MikeMcQuaid commented Jan 21, 2014

I'm not a maintainer for this repo, really.

Contributor

claudiodsf commented Jan 21, 2014

@MikeMcQuaid Sorry, it's just that @samueljohn has kind of disappeared recently...

Owner

MikeMcQuaid commented Jan 21, 2014

Sure. One of the other maintainers may look at this now it's a Homebrew organisation repo.

Contributor

samueljohn commented Jan 21, 2014

Not disappeared but working more and therefore less brew time than during my PhD time. Tomorrow, I'll pull this in.

And a question: Do you want to help out at homebrew/python?

Contributor

claudiodsf commented Jan 22, 2014

@samueljohn I understand. I'm also undergoing a strong work period. That's why I have to decline your offer. Thank you anyway.
This will not prevent me, however, to contribute here and there with some pull request. 😉

Contributor

samueljohn commented Jan 23, 2014

Alright, no problem.

Thanks for your debugging and the addition of the patch!

Contributor

samueljohn commented Jan 23, 2014

added in c837d4d

@samueljohn samueljohn closed this Jan 23, 2014

@claudiodsf claudiodsf deleted the claudiodsf:mpl-freetpe_2.5.1 branch Jan 24, 2014

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.