Add workaround for failures due to newer miktex #6744

Merged
merged 1 commit into from Jul 15, 2016

Conversation

Projects
None yet
4 participants
Contributor

janschulz commented Jul 15, 2016

This should stop the current failures on appveyor, e.g., #6741

I will investigate what the real problem is...

@janschulz janschulz Add workaround for failures due to newer miktex
9ff8535

mdboom added the needs_review label Jul 15, 2016

Owner

jenshnielsen commented Jul 15, 2016

Thanks @janschulz

Owner

jenshnielsen commented Jul 15, 2016 edited

For the record I am happy for you to self merge this (I.e. count this as a review)

Contributor

janschulz commented Jul 15, 2016

Oh fun, can't reproduce on my local system (which also has updated local latex packages on top of the conda installed miktex conda-package provided ones...):

[matplotlib_build] λ nosetests.exe -v matplotlib.tests.test_backend_svg
matplotlib.tests.test_backend_svg.test_bold_font_output.test ... SKIP: Don't know how to convert svg files to png
matplotlib.tests.test_backend_svg.test_bold_font_output_with_none_fonttype.test ... SKIP: Don't know how to convert svg files to png
matplotlib.tests.test_backend_svg.test_fill_black_with_alpha.test ... SKIP: Don't know how to convert svg files to png
matplotlib.tests.test_backend_svg.test_noscale.test ... ok
matplotlib.tests.test_backend_svg.test_noscale.test ... ok
matplotlib.tests.test_backend_svg.test_noscale.test ... SKIP: Don't know how to convert svg files to png
matplotlib.tests.test_backend_svg.test_composite_images ... ok
matplotlib.tests.test_backend_svg.test_determinism_notex ... ok
matplotlib.tests.test_backend_svg.test_text_urls ... ok
matplotlib.tests.test_backend_svg.test_visibility ... ok
matplotlib.tests.test_backend_svg.test_determinism_tex ... ok

----------------------------------------------------------------------
Ran 11 tests in 86.486s

OK (SKIP=4)
Contributor

janschulz commented Jul 15, 2016

Ok, I can replicate when I start with no local latex packages -> it seems that this happens when you use a newly installed miktex, so the problem is in the miktex package.

Contributor

janschulz commented Jul 15, 2016

Ok, tests pass, lets merge this and then I will hopefully find the problem in miktex and fix it there...

@janschulz janschulz merged commit cf24e05 into matplotlib:master Jul 15, 2016

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.002%) to 70.366%
Details

mdboom removed the needs_review label Jul 15, 2016

QuLogic added this to the 2.1 (next point release) milestone Jul 15, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment