remove quit_sage() command from all.py top level

[was]

It is stupid that it is this easy to accidentally destabilize and segfault Sage. Also, having a function "quit_sage()" available at the sage: prompt by default that does not quit sage, is dumb.

wstein@boxen:~/build/sage-4.4$ ./sage
----------------------------------------------------------------------
| Sage Version 4.4, Release Date: 2010-04-24                         |
| Type notebook() for the GUI, and license() for information.        |
----------------------------------------------------------------------
sage: quit_sage()
Exiting Sage (CPU time 0m0.04s, Wall time 0m3.16s).
sage: quit
Exiting Sage (CPU time 0m0.07s, Wall time 0m4.80s).
/virtual/scratch/wstein/build/sage-4.4/local/bin/sage-sage: line 206: 11559 Segmentation fault      sage-ipython "$@" -i
wstein@boxen:~/build/sage-4.4$            

The fix is to rename quit_sage() somehow and change *all* code that calls it.

[jason]

Maybe rename it to "sage_library_cleanup"?

[mjo]

Now all of the cleanup happens automatically and quit_sage() is a no-op:

sage: quit_sage()
<ipython-input-1-ce1781e96a1f>:1: DeprecationWarning: quit_sage is deprecated and now does nothing; please simply delete it
See http://trac.sagemath.org/8784 for details.
  quit_sage()
sage:

---

New commits:

​94dcf5c	Trac #8784: clean up after symmetrica when sage exits.
​7890d1b	Trac #8784: clean up after m4ri when sage exits.
​0cd365d	Trac #8784: free the flint stack when sage terminates.
​ebeb316	Trac #8784: free the integer pool when sage exits.
​b7ae30f	Trac #8784: clear quaternion algebra globals when sage exits.
​15182f2	Trac #8784: cdef two internal quaternion functions.
​eaf3978	Trac #8784: quit pexpect processes when sage terminates.
​b2d1032	Trac #8784: deprecate quit_sage().

[dimpase]

Does this mean that sage-cleaner may go, too?

[mjo]

Replying to dimpase:

Does this mean that sage-cleaner may go, too?

Not yet, although that's another long-term goal of mine. sage-cleaner also cleans up "temporary" files, which it wouldn't have to do if we used the OS's built-in tempfile functions instead of our home-grown SAGE_TMP. The first and easiest part of that cleanup is #33213.

[mjo]

Replying to dimpase:

Does this mean that sage-cleaner may go, too?

Ok, I actually did this in #33213, which now depends on this ticket.

[dimpase]

OK, looks and tests good.

[dimpase]

perhaps you want to rebase on the latest beta

[git]

Branch pushed to git repo; I updated commit sha1 and set ticket back to needs_review. This was a forced push. New commits:

​7abf04d	Trac #8784: clean up after symmetrica when sage exits.
​3424071	Trac #8784: clean up after m4ri when sage exits.
​a053757	Trac #8784: free the flint stack when sage terminates.
​4f5a56d	Trac #8784: free the integer pool when sage exits.
​1bf07ff	Trac #8784: clear quaternion algebra globals when sage exits.
​1f86192	Trac #8784: cdef two internal quaternion functions.
​75e7ccb	Trac #8784: quit pexpect processes when sage terminates.
​ef89f43	Trac #8784: deprecate quit_sage().

[dimpase]

oops, overlooked this, sorry. Looks good.

[mkoeppe]

Follow up in #33706.

[dimpase]

We have a complaint about ipython processes staying alive in a patchbot. ​https://groups.google.com/d/msgid/sage-devel/20220717160807.GA13671%40metelu.net

Could it be due to this ticket, perhaps the change for src/sage/repl/ipython_extension.py was an overkill?