Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Build failure on Launchpad #3684

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Oct 23, 2014
Merged

Build failure on Launchpad #3684

merged 2 commits into from
Oct 23, 2014

Conversation

mdboom
Copy link
Member

@mdboom mdboom commented Oct 20, 2014

This started a few days ago (on the 18th, I think), and has been consistent since then.

building 'matplotlib._png' extension
i686-linux-gnu-gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv -O2 -Wall -fPIC -DPY_ARRAY_UNIQUE_SYMBOL=MPL_matplotlib__png_ARRAY_API -DNPY_NO_DEPRECATED_API=NPY_1_7_API_VERSION -I/usr/lib/python2.7/dist-packages/numpy/core/include -I/usr/include/libpng12 -I/usr/include -I. -I/usr/include/python2.7 -c src/_png.cpp -o build/temp.linux-i686-2.7/src/_png.o
In file included from /usr/include/libpng12/png.h:540:0,
                 from src/_png.cpp:21:
/usr/include/libpng12/pngconf.h:371:12: error: '__pngconf' does not name a type
            __pngconf.h__ in libpng already includes setjmp.h;
            ^
/usr/include/libpng12/pngconf.h:372:12: error: '__dont__' does not name a type
            __dont__ include it again.;
            ^

@pelson
Copy link
Member

pelson commented Oct 20, 2014

cc: @mdboom

@mdboom
Copy link
Member

mdboom commented Oct 20, 2014

I think the attached will fix this, but I have not tested it on Ubuntu specifically.

@mdboom mdboom mentioned this pull request Oct 21, 2014
Closed
pelson added a commit that referenced this pull request Oct 23, 2014
@pelson
Merge pull request #3684 from mdboom/fix-order-of-includes
101eecc 
Build failure on Launchpad
@pelson pelson merged commit 101eecc into matplotlib:master Oct 23, 2014
@pelson
Copy link
Member

pelson commented Oct 23, 2014

Merging as this looks fine. We can iterate if this isn't quite right.

@takluyver
Copy link
Contributor Author

And we have a successful build again. Thanks!

@pelson
Copy link
Member

pelson commented Oct 27, 2014

👍 Thanks Thomas. Good job @mdboom.

@mdboom mdboom deleted the fix-order-of-includes branch March 3, 2015 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mdboom @pelson @takluyver