As a side-effect, this also fixes #6748. Term::ReadLine would prompt and read from terminal even if stdin/stdout are redirected, the new helper will use the redirected channels. It also would underline the prompt, which the new helper does not. And Term::ReadLine also has advanced editing and history shortcuts.
For the prompts in pve-manager, not having the advanced features is perfectly fine. For qm monitor, (advanced) users might be unhappy with the change. Other packages need a dependency bump for pve-common. common: Fiona Ebner (1): developer readme: remove libterm-readline-gnu-perl dependency Stefan Hrdlicka (1): pty: introduce read_line() function README.dev | 2 +- src/PVE/PTY.pm | 17 +++++++++++++++++ 2 files changed, 18 insertions(+), 1 deletion(-) manager: Fiona Ebner (1): partially fix #2077: remove dependency on Term::ReadLine PVE/CLI/pvenode.pm | 19 +++++++------------ debian/control | 1 - 2 files changed, 7 insertions(+), 13 deletions(-) qemu-server: Fiona Ebner (1): partially fix #2077: remove dependency on Term::ReadLine debian/control | 1 - src/PVE/CLI/qm.pm | 6 ++---- 2 files changed, 2 insertions(+), 5 deletions(-) Summary over all repositories: 6 files changed, 27 insertions(+), 19 deletions(-) -- Generated by git-murpp 0.5.0 _______________________________________________ pve-devel mailing list [email protected] https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
