From: Fiona Ebner <f.ebner@proxmox.com>
To: pve-devel@lists.proxmox.com
Subject: [pve-devel] [PATCH-SERIES common/manager/qemu-server 0/4] fix #2077: remove dependency on Term::ReadLine
Date: Thu, 18 Sep 2025 15:50:52 +0200 [thread overview]
Message-ID: <20250918135215.95188-1-f.ebner@proxmox.com> (raw)
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
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
next reply other threads:[~2025-09-18 13:52 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-18 13:50 Fiona Ebner [this message]
2025-09-18 13:50 ` [pve-devel] [PATCH common 1/4] pty: introduce read_line() function Fiona Ebner
2025-09-22 9:45 ` Stoiko Ivanov
2025-09-18 13:50 ` [pve-devel] [RFC common 2/4] developer readme: remove libterm-readline-gnu-perl dependency Fiona Ebner
2025-09-18 14:12 ` Fiona Ebner
2025-09-18 13:50 ` [pve-devel] [PATCH manager 3/4] partially fix #2077: remove dependency on Term::ReadLine Fiona Ebner
2025-09-18 13:50 ` [pve-devel] [RFC qemu-server 4/4] " Fiona Ebner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250918135215.95188-1-f.ebner@proxmox.com \
--to=f.ebner@proxmox.com \
--cc=pve-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.