sagemath/sage

sage.misc.temporary_file: Remove use of SAGE_TMP

mkoeppe opened this issue · 7 comments

(split out from #33213)

Until tmp_filename and tmp_dir are eliminated from the library, we don't want them cluttering up /tmp. In typical usage, this is easy enough to avoid by using one parent temporary directory to contain all the other temporary files and directories, and then removing that one parent as sage exits.

CC: @orlitzky

Component: refactoring

Author: Michael Orlitzky

Branch: bd76501

Reviewer: Matthias Koeppe

Issue created by migration from https://trac.sagemath.org/ticket/33797

Description changed:

--- 
+++ 
@@ -1,2 +1,5 @@
 (split out from #33213)
 
+Until `tmp_filename` and `tmp_dir` are eliminated from the library, we don't want them cluttering up /tmp. In typical usage, this is easy enough to avoid by using one parent temporary directory to contain all the other temporary files and directories, and then removing that one parent as sage exits.
+
+

Commit: bd76501

New commits:

4182565Trac #33213: remove SAGE_TMP from tmp_dir() and tmp_filename().
bd76501Trac #33213: clean up after tmp_filename() and tmp_dir() for now.

Changed commit from bd76501 to none

comment:6

a follow-up is at #33944