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

1.4.0 RC1: --ftversion vs --version freetype version #3262

Merged
merged 1 commit into from Aug 6, 2014

Conversation

mdboom
Copy link
Member

@mdboom mdboom commented Aug 6, 2014

In RC1 of mpl 1.4.0 the freetype version is now determined correctly using freetype-config --ftversion (before --version which returned the libtool version 9.22.3 in my case). This however now leads to the case of matplotlib rc1 not installing on my machine as I have freetype 2.3.11.

My question is: Is freetype2 2.4 really required? Due to the setup bug in mpl 1.3.1 I used mpl with freetype 2.3.11 without any problems for several months now. It could be possible though that I didn't use all relevant features which would actually require 2.4 if that's really the case.

@letmaik letmaik changed the title RC1: --ftversion vs --version freetype version 1.4.0 RC1: --ftversion vs --version freetype version Jul 15, 2014
@tacaswell
Copy link
Member

@mdboom ping on this too, I know very little about this section of the code.

@mdboom
Copy link
Member

mdboom commented Aug 6, 2014

The core freetype API is actually quite stable, so I don't see why this wouldn't work.

tacaswell added a commit that referenced this pull request Aug 6, 2014
BLD : reduces the minimum version of freetype

1.4.0 RC1: --ftversion vs --version freetype version
@tacaswell tacaswell merged commit 40b6020 into matplotlib:master Aug 6, 2014
@mdboom mdboom deleted the min-freetype-version branch August 7, 2014 13:52
tacaswell added a commit that referenced this pull request Aug 26, 2014
BLD : reduces the minimum version of freetype

1.4.0 RC1: --ftversion vs --version freetype version
@tacaswell
Copy link
Member

cherry-picked (too late) as 6ed9211

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