bug#68847: 29.1; Comint read-only mode breaks kill ring

bug#68847: 29.1; Comint read-only mode breaks kill ring

[Elliott R]

[-- Attachment #1: Type: text/plain, Size: 4447 bytes --]

Saving read-only text to the kill ring breaks the usual cycling

behavior; once you hit a piece of read-only text in the kill ring, you

can't cycle further back (or forward).



To reproduce this issue, run emacs -Q and do the following:

- Select any word (say from the scratch buffer) and run M-x

kill-ring-save.

- Run M-x eval-expression and (setq comint-prompt-read-only t). This will

make the text in the prompt of the inferior shell read-only.

- Run M-x shell, select any text from the prompt, and run M-x

kill-ring-save.

- Run M-x yank-from-kill-ring (say in the scratch buffer), and press C-p

repeatedly to cycle backward through the kill ring.

- Note how we cannot cycle past the read-only text, in particular we

have lost access to the text we saved in the first step.



Thanks!





In GNU Emacs 29.1 (build 1, x86_64-pc-linux-gnu, X toolkit, cairo

version 1.16.0, Xaw3d scroll bars)

Windowing system distributor 'The Cygwin/X Project', version 11.0.12000000

System Description: Red Hat Enterprise Linux 8.9 (Ootpa)



Configured using:

'configure

--prefix=/nix/store/5m64iwpqw2m978r2hsp1823dk9w9bbxd-emacs-29.1

--disable-build-details --with-modules --with-x-toolkit=lucid

--with-xft --with-cairo --with-native-compilation --with-tree-sitter

--with-xinput2'



Configured features:

CAIRO DBUS FREETYPE GIF GLIB GMP GNUTLS GPM GSETTINGS HARFBUZZ JPEG JSON

LIBOTF LIBSELINUX LIBSYSTEMD LIBXML2 M17N_FLT MODULES NATIVE_COMP NOTIFY

INOTIFY PDUMPER PNG RSVG SECCOMP SOUND SQLITE3 THREADS TIFF

TOOLKIT_SCROLL_BARS TREE_SITTER WEBP X11 XAW3D XDBE XIM XINPUT2 XPM

LUCID ZLIB



Important settings:

  value of $EMACSLOADPATH:
/nix/store/l3kshi53cdl4w8dy2vziacl8sg63df4b-emacs-packages-deps/share/emacs/site-lisp:

  value of $EMACSNATIVELOADPATH:
/nix/store/l3kshi53cdl4w8dy2vziacl8sg63df4b-emacs-packages-deps/share/emacs/native-lisp:

  value of $LC_COLLATE: C

  value of $LANG: en_US.UTF-8

  locale-coding-system: utf-8-unix



Major mode: Shell



Minor modes in effect:

  shell-dirtrack-mode: t

  comint-fontify-input-mode: t

  tooltip-mode: t

  global-eldoc-mode: t

  show-paren-mode: t

  electric-indent-mode: t

  mouse-wheel-mode: t

  tool-bar-mode: t

  menu-bar-mode: t

  file-name-shadow-mode: t

  global-font-lock-mode: t

  font-lock-mode: t

  blink-cursor-mode: t

  line-number-mode: t

  indent-tabs-mode: t

  transient-mark-mode: t

  auto-composition-mode: t

  auto-encryption-mode: t

  auto-compression-mode: t



Load-path shadows:

None found.



Features:

(shadow sort mail-extr emacsbug message mailcap yank-media puny dired

dired-loaddefs rfc822 mml mml-sec password-cache epa derived epg rfc6068

epg-config gnus-util text-property-search time-date mm-decode mm-bodies

mm-encode mail-parse rfc2231 mailabbrev gmm-utils mailheader sendmail

rfc2047 rfc2045 ietf-drums mm-util mail-prsvr mail-utils sh-script smie

treesit cl-loaddefs comp comp-cstr warnings icons rx cl-seq cl-macs gv

cl-extra help-mode bytecomp byte-compile cl-lib executable files-x shell

subr-x pcomplete comint ansi-osc ansi-color ring rmc iso-transl tooltip

cconv eldoc paren electric uniquify ediff-hook vc-hooks lisp-float-type

elisp-mode mwheel term/x-win x-win term/common-win x-dnd tool-bar dnd

fontset image regexp-opt fringe tabulated-list replace newcomment

text-mode lisp-mode prog-mode register page tab-bar menu-bar rfn-eshadow

isearch easymenu timer select scroll-bar mouse jit-lock font-lock syntax

font-core term/tty-colors frame minibuffer nadvice seq simple cl-generic

indonesian philippine cham georgian utf-8-lang misc-lang vietnamese

tibetan thai tai-viet lao korean japanese eucjp-ms cp51932 hebrew greek

romanian slovak czech european ethiopic indian cyrillic chinese

composite emoji-zwj charscript charprop case-table epa-hook

jka-cmpr-hook help abbrev obarray oclosure cl-preloaded button loaddefs

theme-loaddefs faces cus-face macroexp files window text-properties

overlay sha1 md5 base64 format env code-pages mule custom widget keymap

hashtable-print-readable backquote threads dbusbind inotify

dynamic-setting system-font-setting font-render-setting cairo x-toolkit

xinput2 x multi-tty make-network-process native-compile emacs)



Memory information:

((conses 16 97113 11443)

(symbols 48 8350 0)

(strings 32 26050 1168)

(string-bytes 1 862799)

(vectors 16 18221)

(vector-slots 8 377924 14082)

(floats 8 47 42)

(intervals 56 445 0)

(buffers 984 14))

[-- Attachment #2: Type: text/html, Size: 8554 bytes --]

bug#68847: 29.1; Comint read-only mode breaks kill ring

[Juri Linkov]

close 68847 30.0.50
thanks

> Saving read-only text to the kill ring breaks the usual cycling
> behavior; once you hit a piece of read-only text in the kill ring, you
> can't cycle further back (or forward).

Thanks for the reproducible bug report.
This is fixed now on master.

bug#68847: 29.1; Comint read-only mode breaks kill ring

[Richard Stallman]

[[[ To any NSA and FBI agents reading my email: please consider    ]]]
[[[ whether defending the US Constitution against all enemies,     ]]]
[[[ foreign or domestic, requires you to follow Snowden's example. ]]]

  > Saving read-only text to the kill ring breaks the usual cycling
  > behavior; once you hit a piece of read-only text in the kill ring, you
  > can't cycle further back (or forward).

The joke is on us.  It should have been obvious that this would happen
-- but it seems that none of us thought of it.

-- 
Dr Richard Stallman (https://stallman.org)
Chief GNUisance of the GNU Project (https://gnu.org)
Founder, Free Software Foundation (https://fsf.org)
Internet Hall-of-Famer (https://internethalloffame.org)