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

OSX install #3238

Merged
merged 13 commits into from Jul 17, 2014
Merged

OSX install #3238

merged 13 commits into from Jul 17, 2014

Conversation

katyhuff
Copy link
Contributor

This makes some slight instruction changes for the README in the OSX README and adds a little updating in the INSTALL file. This is only a solution to #3209 for the OSX platform. Someone should still check and change the windows instructions, for example, as discussed in issue #2999.

frequently. Both of these packages include matplotlib and pylab, and
distributions that already provide matplotlib built-in. The Continuum.io
Python distribution `(Anaconda)
<https://store.continuum.io/cshop/anaconda/>`_ for Windows, OS X, or Redhat
Copy link
Member

Choose a reason for hiding this comment

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

Redhat -> linux

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah, I think you're right. This bit was already in there and I thought about changing it, but I thought maybe (due to the discussion of linux later) there was something subtle about matplotlib on debian systems or something. Not the case?

@@ -22,3 +29,9 @@ To install matplotlib from source, execute:
Example usage::

python setup.py install
Copy link
Member

Choose a reason for hiding this comment

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

So -- this is totally cool, but I think we maybe just discovered this doesn't work (at least it didn't for Michael Nagy and I). building matplotlib from source using conda as dependencies, it doesn't pick up freetype from conda since conda doesn't include pkg-config. Anyway, if it's working for you @katyhuff, it would be good to know so we can get to the bottom of the cause.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ahhhh... I have two sets of each library installed. Glad you caught this. Matplotlib is picking up my macports libpng rather than the conda one! Changing instructions back accordingly.

@tacaswell tacaswell added this to the v1.4.0 milestone Jul 12, 2014
@NelleV
Copy link
Member

NelleV commented Jul 12, 2014

By the way, libpng is listed as an optional dependencies in the main installation documentation (INSTALL in root). Do you mind taking care of that? It is just moving the description libpng to the build requirements.

For developers who want to build matplotlib from source, the recommended and
supported way to build is to use a third-party package manager to install the
required dependencies, and then install matplotlib from source using the
setup.py script. Three widely used package managers are homebrew, and
Copy link
Member

Choose a reason for hiding this comment

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

You mean "Two"... unless you want to include fink. I would just stick with homebrew and macports, though.

@matthew-brett
Copy link
Contributor

Any reason to remove python x, y from the list of "preferred" installers?

@mdboom
Copy link
Member

mdboom commented Jul 14, 2014

Re: Python (x, y): I mistakenly thought it was unmaintained. We can restore the recommendation if that's what you're suggesting...

@matthew-brett
Copy link
Contributor

Yes, thanks - that what I was suggesting...

@katyhuff
Copy link
Contributor Author

it's back in! :)

@matthew-brett
Copy link
Contributor

Thanks for that and for working on the docs.

On 7/14/14, Katy Huff notifications@github.com wrote:

it's back in! :)


Reply to this email directly or view it on GitHub:
#3238 (comment)

<http://www.pythonxy.com/>`_ which tends to be updated a bit more
frequently. Both of these packages include matplotlib and pylab, and
*lots* of other useful tools. matplotlib is also packaged for almost
<http://www.pythonxy.com/>`_ . matplotlib is also packaged for almost
Copy link
Member

Choose a reason for hiding this comment

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

Link is dead...

@dmcdougall
Copy link
Member

+1

I think this should be cherry-picked into 1.4.x too. Any objections?

@efiring
Copy link
Member

efiring commented Jul 16, 2014

No objections; we do need this in 1.4.x.

@tacaswell
Copy link
Member

Yes, who ever merges this should cherry-pick it.

dmcdougall added a commit that referenced this pull request Jul 17, 2014
@dmcdougall dmcdougall merged commit 448179a into matplotlib:master Jul 17, 2014
@dmcdougall
Copy link
Member

Thanks @katyhuff!

@dmcdougall
Copy link
Member

Cherry-pick done, see c893d56...2ecabc1.

@katyhuff
Copy link
Contributor Author

Thanks folks!

On Wed, Jul 16, 2014 at 5:26 PM, Damon McDougall notifications@github.com
wrote:

Cherry-pick done, see c893d56...2ecabc1
c893d56...2ecabc1.


Reply to this email directly or view it on GitHub
#3238 (comment)
.

http://katyhuff.github.com

@pelson
Copy link
Member

pelson commented Jul 17, 2014

👍 good job @katyhuff - I learned something about my mac today. 😄

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

8 participants