sage.libs.ecl, sage.interfaces.maxima*: Do not use ECL_TMP

[mkoeppe]

(split out from #33213)

We have historically massaged TMPDIR and TMP when interfacing with ECL to ensure that SAGE_TMP was used consistently for temporary
files. Simply removing this code seems to do the trick, since ECL
wants to use the system location by default.

Depends on #33829

CC: @orlitzky @spaghettisalat @nbruin

Component: interfaces

Author: Michael Orlitzky

Branch/Commit: 4610b1c

Reviewer: Dima Pasechnik

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

[orlitzky]

comment:1

The fallback within ECL is a little naive, to /tmp if TMPDIR is unset. But I think that should work on all of our supported platforms, and the GH actions didn't turn up any problems.

If not we could pass in tempfile.gettempdir() from python instead.

[mkoeppe]

Branch: u/mkoeppe/sage_libs_ecl__sage_interfaces_maxima___do_not_use_ecl_tmp

[mkoeppe]

Commit: 1a2bea0

[mkoeppe]

New commits:

1a2bea0

Trac #33213: replace ECL_TMP (based on SAGE_TMP) in maxima/ecl interfaces.

[dimpase]

Changed branch from u/mkoeppe/sage_libs_ecl__sage_interfaces_maxima___do_not_use_ecl_tmp to u/dimpase/sage_libs_ecl__sage_interfaces_maxima___do_not_use_ecl_tmp

[dimpase]

Reviewer: Dima Pasechnik

[dimpase]

comment:5

rebased over #33829 (and Sage 9.7.beta2)

---

New commits:

818410f	Trac #33213: replace SAGE_TMP in the isgci database download routine.
71c9325	Trac #33213: replace SAGE_TMP in GLPK graph backend doctests.
08213fa	Trac #33213: replace SAGE_TMP in graph doctests.
976f0d6	remove crud after except, make it pass
4610b1c	Trac #33213: replace ECL_TMP (based on SAGE_TMP) in maxima/ecl interfaces.

[dimpase]

Changed commit from 1a2bea0 to 4610b1c

[mkoeppe]

comment:6

I think you got the dependencies the wrong way around

[dimpase]

comment:7

should be like this, sorry.

[dimpase]

Dependencies: #33829

[vbraun]

Changed branch from u/dimpase/sage_libs_ecl__sage_interfaces_maxima___do_not_use_ecl_tmp to 4610b1c