#8677: race condition creating dirs during Sage build on RH 5.4
-----------------------------+----------------------------------------------
Reporter: GeorgSWeber | Owner: GeorgSWeber
Type: defect | Status: needs_review
Priority: blocker | Milestone: sage-4.6
Component: build | Keywords:
Author: William Stein | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
-----------------------------+----------------------------------------------
Comment(by mpatel):
Replying to [comment:9 vbraun]:
> Replying to [comment:8 was]:
> > I strongly disagree. I see no point in not including the
> > if not os.path.exists(dirname):
> The point is that the pattern "if (look at filesystem) then (modify
filesystem)" is fundamentally flawed. Its the origin of countless security
problems from temp races etc. The world is a better place without it :-)
And the only way to make it die is to stomp it out wherever we encounter
it...
What if we use William's solution for the first "hunk" and Volker's for
the second? Then we can avoid many unnecessary exceptions in the first.
In the second, we only call `os.makedirs` if copying the Cython-compiled
file fails.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8677#comment:10>
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.
