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

Build: allow local static png dependency #1970

Closed
wants to merge 1 commit into from
Closed

Build: allow local static png dependency #1970

wants to merge 1 commit into from

Conversation

GBillotey
Copy link
Contributor

To start with, I am very new to building with windows SDK - so this issue is probably related with a bad practise: please tell.

Since commit 386b4e5 I have been unable to build using a local static png dependency (the one generously provided online by: http://www.lfd.uci.edu/~gohlke/pythonlibs/#matplotlib). Previously it would build even with png version = None ; now a CheckFailed error is raised.

As a workaround I ended adding the same try / catch pattern in check method of
Png class as in the LibAgg class of setupext.py. I am really unsure this the good thing to do ; however in this case I would be happy to learn.

@mdboom
Copy link
Member

mdboom commented May 3, 2013

I'm Cc'ing @cgohlke as our resident Windows expert.

I'm not sure I follow what you're trying to do here. At present when something in matplotlib says "using local copy", it means using a copy of the library that we include with the matplotlib source code. We don't include libpng in our source tree, so it doesn't make sense to say we are "using local copy".

I guess the issue here is that libpng isn't found with pkg-config, but it should continue because the other default values of include and library paths should find it from Christoph's packages. That seems right -- but perhaps the message should be "not found with pkg-config", rather than "using local copy".

In any event, I'll defer to @cgohlke for the best solution on this one.

@cgohlke
Copy link
Contributor

cgohlke commented May 3, 2013

I can confirm the issue and that this PR works. Perhaps the error message could be ending with "Using unknown version", consistent with freetype, so the png status reads png: yes [pkg-config information for 'libpng' could not be found. Using unknown version.]

@mdboom
Copy link
Member

mdboom commented May 3, 2013

Perfect, and thanks for the confirmation. I'll go ahead and make the change.

@ghost ghost assigned mdboom May 3, 2013
@mdboom
Copy link
Member

mdboom commented May 6, 2013

Merged manually.

@mdboom mdboom closed this May 6, 2013
@pelson
Copy link
Member

pelson commented May 7, 2013

Merged manually.

For the record (84a7a1f and 620117f)

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

4 participants