Re: [sage-devel] Temporary files problems

2022-09-27 Thread John H Palmieri
I created https://trac.sagemath.org/ticket/34593: we should document how to change the location of the temporary directory, and maybe we should create a `.tmpreaper` file. On Tuesday, September 27, 2022 at 11:06:01 AM UTC-7 Michael Orlitzky wrote: > On Tue, 2022-09-27 at 18:10 +0100, Dima

Re: [sage-devel] Temporary files problems

2022-09-27 Thread Michael Orlitzky
On Tue, 2022-09-27 at 18:10 +0100, Dima Pasechnik wrote: > Basically, we should deprecate and remove tmp_dir() and tmp_filename() > from Sage. > Does Sagecell use them? It should not, Python3 has perfectly good > replacements... > That was always the plan. From #33213: > Afterward, the custom

Re: [sage-devel] Temporary files problems

2022-09-27 Thread Dima Pasechnik
On Tue, 27 Sep 2022, 18:38 Andrey Novoseltsev, wrote: > On Tuesday, 27 September 2022 at 11:11:00 UTC-6 dim...@gmail.com wrote: > >> Basically, we should deprecate and remove tmp_dir() and tmp_filename() >> from Sage. >> Does Sagecell use them? It should not, Python3 has perfectly good >>

Re: [sage-devel] Temporary files problems

2022-09-27 Thread Matthias Koeppe
On Tuesday, September 27, 2022 at 10:21:15 AM UTC-7 John H Palmieri wrote: > On Tuesday, September 27, 2022 at 8:33:01 AM UTC-7 dim...@gmail.com wrote: > >> >> tempfile.TemporaryDirectory(*suffix=None*, *prefix=None*, *dir=None*, >> *ignore_cleanup_errors=False*) >> >> dir= may be used to set a

Re: [sage-devel] Temporary files problems

2022-09-27 Thread Andrey Novoseltsev
On Tuesday, 27 September 2022 at 11:11:00 UTC-6 dim...@gmail.com wrote: > Basically, we should deprecate and remove tmp_dir() and tmp_filename() > from Sage. > Does Sagecell use them? It should not, Python3 has perfectly good > replacements... > The problems that surfaces so far that I

Re: [sage-devel] Temporary files problems

2022-09-27 Thread John H Palmieri
On Tuesday, September 27, 2022 at 10:11:00 AM UTC-7 dim...@gmail.com wrote: > Basically, we should deprecate and remove tmp_dir() and tmp_filename() > from Sage. > Does Sagecell use them? It should not, Python3 has perfectly good > replacements... > As long as we test everything on a

Re: [sage-devel] Temporary files problems

2022-09-27 Thread John H Palmieri
On Tuesday, September 27, 2022 at 8:33:01 AM UTC-7 dim...@gmail.com wrote: > > > On Tue, 27 Sep 2022, 15:45 John H Palmieri, wrote: > >> One of the issues is that code in Sage is using this temporary directory >> in the background; the user is not doing it and has no control over it. >>

Re: [sage-devel] Temporary files problems

2022-09-27 Thread Dima Pasechnik
Basically, we should deprecate and remove tmp_dir() and tmp_filename() from Sage. Does Sagecell use them? It should not, Python3 has perfectly good replacements... On Tue, Sep 27, 2022 at 6:06 PM Matthias Koeppe wrote: > > Previous discussion -- in the ticket that made these changes to SAGE_TMP

Re: [sage-devel] Temporary files problems

2022-09-27 Thread Matthias Koeppe
Previous discussion -- in the ticket that made these changes to SAGE_TMP --- https://trac.sagemath.org/ticket/33213#comment:11 On Tuesday, September 27, 2022 at 9:10:28 AM UTC-7 novo...@gmail.com wrote: > On Tuesday, 27 September 2022 at 03:22:47 UTC-6 dim...@gmail.com wrote: > >> On Tue, Sep

Re: [sage-devel] Temporary files problems

2022-09-27 Thread Andrey Novoseltsev
On Tuesday, 27 September 2022 at 03:22:47 UTC-6 dim...@gmail.com wrote: > On Tue, Sep 27, 2022 at 3:58 AM Andrey Novoseltsev > wrote: > > this temporary directory is created and then used ever after. But what > if it gets deleted? > I think it's meant to be used in a context manager, i.e.

Re: [sage-devel] Temporary files problems

2022-09-27 Thread Dima Pasechnik
On Tue, 27 Sep 2022, 15:45 John H Palmieri, wrote: > One of the issues is that code in Sage is using this temporary directory > in the background; the user is not doing it and has no control over it. > Regardless of the context manager approach, apparently some browsers refuse > to open local

Re: [sage-devel] Temporary files problems

2022-09-27 Thread John H Palmieri
One of the issues is that code in Sage is using this temporary directory in the background; the user is not doing it and has no control over it. Regardless of the context manager approach, apparently some browsers refuse to open local files that are not in the user's home directory, so this

Re: [sage-devel] Temporary files problems

2022-09-27 Thread Dima Pasechnik
On Tue, Sep 27, 2022 at 3:58 AM Andrey Novoseltsev wrote: > > Hello! > > I've run into this while testing upgrades to Ubuntu 22.04 from 20.04, but it > seems that it comes from how temporary files are handled in Sage 9.7: > > TMP_DIR_FILENAME_BASE=tempfile.TemporaryDirectory() > > this temporary

[sage-devel] Temporary files problems

2022-09-26 Thread Andrey Novoseltsev
Hello! I've run into this while testing upgrades to Ubuntu 22.04 from 20.04, but it seems that it comes from how temporary files are handled in Sage 9.7: TMP_DIR_FILENAME_BASE=tempfile.TemporaryDirectory() this temporary directory is created and then used ever after. But what if it gets