Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Lock calls to latex in texmanager #5976
Conversation
jenshnielsen
added this to the
2.0 (style change major release)
milestone
Feb 8, 2016
|
And I opened against the wrong branch ups |
mdboom
added the
needs_review
label
Feb 8, 2016
tacaswell
added a commit
that referenced
this pull request
Feb 8, 2016
|
|
tacaswell |
cfd95e0
|
tacaswell
merged commit cfd95e0
into matplotlib:master
Feb 8, 2016
tacaswell
removed the
needs_review
label
Feb 8, 2016
tacaswell
added a commit
that referenced
this pull request
Feb 8, 2016
|
|
tacaswell |
bb6978c
|
|
backported to v2.x as bb6978c |
jenshnielsen
deleted the
jenshnielsen:locklatex branch
Feb 16, 2016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
jenshnielsen commentedFeb 8, 2016
These seem to be one of the more common reasons for random failures in the test suite when running in parallel.
This code should really be rewritten using subprocess rather than os.system but I would like this minimal version to go against 2.0. The Locked context manager is not on 1.5.x so this is tricky to backport further.