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

Fixing OSX makefile by fgb · Pull Request #16 · matplotlib/matplotlib · GitHub

Repository navigation

Fixing OSX makefile - #16

Closed
fgb wants to merge 5 commits into
matplotlib:masterfrom
fgb:v1.0.x
Closed

fgb wants to merge 5 commits into
matplotlib:masterfrom
fgb:v1.0.x

Conversation

fgb commented Feb 23, 2011

Copy link
Copy Markdown
Contributor

I updated a couple of links for fetch and added some stray files to clean. I'm also making a couple of suggestions regarding requiring PYVERSION in the command line and dropping the PREFIX from mpl_install. For this last one, I'm assuming that the dependencies are compile-time and matplotlib will work correctly from the relevant site-packages (I haven't seen any problems yet, although I've kept the dependencies in the PREFIX), but I welcome any comments or corrections.

fgb added 5 commits February 22, 2011 23:29
Although a hard-coded PYVERSION=2.6 is safe for snow leopard, it might make more sense to leave it blank, defaulting to the 'python' command that takes precedence in the PATH. Also, suggesting in the documentation that it can be included in the command line might prompt users to choose the right version without having to edit the makefile.
richbwood mentioned this pull request Dec 19, 2012
magnunor pushed a commit to magnunor/matplotlib that referenced this pull request Dec 5, 2013
phobson pushed a commit to phobson/matplotlib that referenced this pull request Sep 14, 2016
swfiua pushed a commit to swfiua/matplotlib that referenced this pull request Mar 17, 2017
This pull request was closed.
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.

1 participant


Back | FazBrowse Home | New Git URL