#10159: matplotlib: avoid race condition when creating config directory
------------------------------+---------------------------------------------
Reporter: jhpalmieri | Owner: tbd
Type: defect | Status: positive_review
Priority: major | Milestone: sage-4.6
Component: packages | Keywords: matplotlib
Author: John Palmieri | Upstream: Reported upstream. Little or no
feedback.
Reviewer: Leif Leonhardy | Merged:
Work_issues: |
------------------------------+---------------------------------------------
Comment(by leif):
Oh, I just noticed the patch does ''not'' (exactly) what the ticket
advertises...
(In fact, the patch addresses the race condition in the creation of the
TeX cache.)
There are still some other instances of {{{if not os.path.exists(foo):
mkdir(foo)}}} and similar, so it's a bit surprising I did not run into any
race condition.
John, are you willing to fix these, too?
Unless you meanwhile have Sun's grep installed, you can find them by e.g.
{{{
#!sh
$ grep -B5 -A2 mkdir `find src -name \*.py`
}}}
(Giving about two false positives; also not all are equally important I
think.)
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10159#comment:5>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
