Remove sage.libs.readline

[mkoeppe]

Unused, should remove.

It was added 8 years ago, in #14523, but the Python world has moved away from using readline: IPython is using different libraries for terminal interaction now.

CC: @dimpase @tscrim

Component: interfaces

Author: Matthias Koeppe, Travis Scrimshaw

Branch/Commit: ed14848

Reviewer: Travis Scrimshaw, Matthias Koeppe

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

[mkoeppe]

comment:2

I think it is already defunct because in normal Sage terminal interaction, readline is not used, so whatever this interface is offering (with interleaved_output()) can't work any more.

How about we just remove it immediately, without deprecation?

[dimpase]

comment:3

It's broken, no need for deprecation.

[mkoeppe]

Branch: u/mkoeppe/deprecate_sage_libs_readline

[mkoeppe]

New commits:

c9e42ec

src/sage/libs/readline.pyx: Unused, remove; remove dependency of sagelib on readline

[mkoeppe]

Author: Matthias Koeppe

[mkoeppe]

Commit: c9e42ec

[tscrim]

comment:6

This won't break again #14289, correct? I don't have an OSX machine to verify this.

[mkoeppe]

comment:7

This module is unused. Try a search for imports of it

[tscrim]

comment:8

Thanks. Then I will now apply fire and holy water to it.

[tscrim]

Reviewer: Travis Scrimshaw

[mkoeppe]

comment:9

Thanks!

[vbraun]

comment:10

Merge conflict

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

8feca2a

Merge tag '9.4.beta5' into t/32073/deprecate_sage_libs_readline

[sagetrac-git]

Changed commit from c9e42ec to 8feca2a

[vbraun]

comment:13

Documentation fails to build

[tscrim]

Changed commit from 8feca2a to ed14848

[tscrim]

Changed branch from u/mkoeppe/deprecate_sage_libs_readline to u/tscrim/deprecate_readline-32073

[tscrim]

comment:14

Removed the lib file from the reference manual. Doc builds for me.

---

New commits:

ed14848

Remove readline lib file from the documentation.

[mkoeppe]

Changed author from Matthias Koeppe to Matthias Koeppe, Travis Scrimshaw

[mkoeppe]

Changed reviewer from Travis Scrimshaw to Travis Scrimshaw, Matthias Koeppe

[vbraun]

Changed branch from u/tscrim/deprecate_readline-32073 to ed14848