FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Update for building matplotlib under Mac OS X 10.7 Lion and XCode > 4.2 · Pull Request #899 · matplotlib/matplotlib · GitHub

Repository navigation

Update for building matplotlib under Mac OS X 10.7 Lion and XCode > 4.2 - #899

Closed
ghost wants to merge 1 commit into
masterfrom
unknown repository
Closed

ghost wants to merge 1 commit into
masterfrom
unknown repository

Conversation

ghost commented May 25, 2012

Copy link
Copy Markdown

No description provided.

efiring commented May 26, 2012

Copy link
Copy Markdown
Member

Maybe we need to have more than one make.osx; I imagine quite a few people will still be building on older systems, so I don't think we want the older version of make.osx simply vanish. But we certainly do want to have a version that works on the latest.

Comment thread make.osx

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

As far as I can see, this has made the code more likely to fail. Currently PNGVERSION in this file is 1.5.4 but the 1.5.4 folder only exists in the older-releases folder

pelson commented Jun 24, 2012

Copy link
Copy Markdown
Member

@XIRON: Did you try using the setup.py? I know the preferred method states to use the make file, but I have had some success with the regular build process (10.7 and latest xcode, with some homebrew extras).

jkseppan commented Jul 2, 2012

Copy link
Copy Markdown
Member

I got building to work with homebrew on 10.6 without using make.osx at all, so we might not really need make.osx for user builds. I think it has been used to build the downloadable binaries to work on a wide variety of OS X versions, and that use is probably still valid.

Copy link
Copy Markdown
Member

What is the status of this PR? Do we need more testers? I have access to a user with this combination of specs who could do a test if needed.

pelson commented Jul 19, 2012

Copy link
Copy Markdown
Member

I don't have confidence that this change works (see my inline code comment), and @jkseppan and myself are awaiting to hear if using setup.py would work instead of using the make.osx (it does for me, and I have 10.7 with xcode 4.3.3, with additional brew installations).

If there were a pause/hold button on pull requests, I would have it pressed for this PR.

@XIRON: Does setup.py work for you? Is there any additional motivation for changing make.osx?

Copy link
Copy Markdown
Member

as an additional datapoint, the user I was referring to just yesterday built and installed matplotlib from the master branch just fine. I am pretty sure he is using homebrew as well.

pelson commented Aug 11, 2012

Copy link
Copy Markdown
Member

@XIRON: I'm closing this issue as I don't believe that the code can work anymore (the png library on the sourceforge site is no longer in that location). Additionally, it is currently best to build mpl via the standard setup.py`` so the need for themake.osx``` seems to be diminishing (perhaps it is still needed for some distribution builds?).

We can always re-open this issue if there are good reasons to do so. If you are unable to build for osx 10.7 feel free to get in touch via the users or devel mailing list and I would be happy to help.

pelson closed this Aug 11, 2012
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

4 participants


Back | FazBrowse Home | New Git URL