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

Install texlive and other dependencies when building docs. #3559

Merged
merged 1 commit into from Sep 24, 2014

Conversation

jenshnielsen
Copy link
Member

This is a version of #3550 that actually works.

@jenshnielsen
Copy link
Member Author

There are still parts not build due to missing backends and scikit-image.

The backends should probably be mocked http://read-the-docs.readthedocs.org/en/latest/faq.html
since they aren't actually needed beyond import:

Unfortunately the part of scikit-image that we need also depends on scipy. I am not sure that the system scipy will work since it is compiled against system numpy which matplotlib is not. We could change that and use system numpy when building the docs since building scipy is slow. I guess that in the long term using miniconda or bin wheels for the dependencies on travis is the right solution.

@tacaswell tacaswell added this to the v1.4.x milestone Sep 24, 2014
@tacaswell
Copy link
Member

👍 on both mocking and conda

@jenshnielsen
Copy link
Member Author

Just to be clear. That is orthogonal to the changes in this PR since the system texlive and graphviz that this installs works fine.

tacaswell added a commit that referenced this pull request Sep 24, 2014
TST : Install texlive and other dependencies when building docs.
@tacaswell tacaswell merged commit ee2dde7 into matplotlib:v1.4.x Sep 24, 2014
tacaswell added a commit that referenced this pull request Sep 24, 2014
@jenshnielsen jenshnielsen deleted the texlive_travis branch October 17, 2014 22:12
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

2 participants