#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: |
------------------------------+---------------------------------------------
Changes (by leif):
* status: needs_review => positive_review
* reviewer: => Leif Leonhardy
Comment:
Apart from the typo in the commit message, and I'm not sure if I like
{{{assert}}} there, I'm ok with the patch, and the spkg is clean.
Works for me as advertised, so positive review.
(Tested on Ubuntu 10.04 x86_64, Core2 Quad. I also deleted the MPL config
dir and then ran
{{{
#!sh
$ ./sage -tp N -long devel/sage/sage/plot/ # N={32,64} (though only 49
files)
}}}
Each time all doctests passed. Perhaps others should stress-[doc]test it
on other platforms like MacOS X, too.)
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10159#comment:3>
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.
