Skip to content

Better check for required dependency libpng#3477

Merged
tacaswell merged 1 commit into
matplotlib:masterfrom
cimarronm:setupext_check_for_libpng
Sep 8, 2014
Merged

Better check for required dependency libpng#3477
tacaswell merged 1 commit into
matplotlib:masterfrom
cimarronm:setupext_check_for_libpng

Conversation

@cimarronm

Copy link
Copy Markdown
Contributor

This fixes a bug where the setup incorrectly may state that libpng is present and continue building which will eventually error out.

Current behavior

No libpng installed at all

REQUIRED DEPENDENCIES AND EXTENSIONS
...
...
                   png: yes [pkg-config information for 'libpng' could not
                        be found. Using unknown version.]
...
...
<builds until it eventually errors out>

Updated behavior

libpng pkgconfig exists

REQUIRED DEPENDENCIES AND EXTENSIONS
...
...
                   png: yes [version x.y.z]
...
...
<builds>

no libpng pkgconfig bug png.h include exists

REQUIRED DEPENDENCIES AND EXTENSIONS
...
...
                   png: yes [pkg-config information for 'libpng' could not
                        be found. Using unknown version found on system.]
...
...
<builds>

no png.h include at all

REQUIRED DEPENDENCIES AND EXTENSIONS
...
...
                   png: no  [pkg-config information for 'libpng' could not
                        be found.]
...
...
============================================================================
                        * The following required packages can not be built:
                        * png
<does not build>

@cimarronm cimarronm changed the title BUG: If pkgconfig returns nothing for libpng at least check for header f... Better check required dependency libpng Sep 6, 2014
@cimarronm cimarronm changed the title Better check required dependency libpng Better check for required dependency libpng Sep 6, 2014
@tacaswell tacaswell added this to the v1.4.x milestone Sep 6, 2014
@tacaswell tacaswell modified the milestones: v1.4.1, v1.4.x Sep 8, 2014
tacaswell added a commit that referenced this pull request Sep 8, 2014
BLD : Better check for required dependency libpng
@tacaswell tacaswell merged commit a44673b into matplotlib:master Sep 8, 2014
tacaswell added a commit that referenced this pull request Sep 12, 2014
BLD : Better check for required dependency libpng
@tacaswell

Copy link
Copy Markdown
Member

cherry-picked as ef2ea6a

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.

3 participants