-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
PGF: Replace \pgfimage by \includegraphics to fix \import regression #10973
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
Conversation
|
Ah yes, my current system uses TeX Live 2017, where |
Attempt to upgrade TeX Live used by Travis to 2015 by adding PPA |
Well xelatex claims to be the 2015 version: https://travis-ci.org/matplotlib/matplotlib/jobs/363726207#L2738 |
I think that's a lie though. The PPA doesn't provide anything significant beyond |
Does this need to wait for more distros to have a more modern texlive? I'll mark needs upstream fix, though I know thats not quite the situation here... |
In the real world I'd say the typically distributed TeX live version should be recent enough, it's just that Travis is stuck at the extremely old Ubuntu 14.04. Reading their comments on this they don't seem to consider upgrading to a newer version anytime soon and suggest using the Docker option of defining your own base image. Is moving from Travis stock images to a Docker based setup something that is being considered for matplotlib? |
@pwuertz no, but in principle open to one or more of the builds in the test matrix moving over. |
@pwuertz I bumped to 3.2, but if you think it'll work on modern CI, feel free to bumpy back... |
power-cycled to re-trigger CI, looks like travis and azure are both too old... |
We probably should document #10963 (comment) ? |
Superseded by #15150. |
The suggested method for importing PGF figures from other directories using the import package does not work anymore. Tex does not find the images referenced by the PGF file. Replacing
\pgfimage
by\includegraphics
fixes this problem (fixes issue #10963).