GNU bug report logs - #18475
24.4.50; Wdired: cannot use C-k to delete a dir name if -F switch used

Previous Next

Package: emacs;

Reported by: Drew Adams <drew.adams <at> oracle.com>

Date: Sun, 14 Sep 2014 18:03:01 UTC

Severity: minor

Tags: fixed, patch

Found in version 24.4.50

Fixed in version 28.1

Done: Lars Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 18475 in the body.
You can then email your comments to 18475 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-gnu-emacs <at> gnu.org:
bug#18475; Package emacs. (Sun, 14 Sep 2014 18:03:01 GMT) Full text and rfc822 format available.

Acknowledgement sent to Drew Adams <drew.adams <at> oracle.com>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Sun, 14 Sep 2014 18:03:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Drew Adams <drew.adams <at> oracle.com>
To: bug-gnu-emacs <at> gnu.org
Subject: 24.4.50; Wdired: cannot use C-k to delete a dir name if -F switch used
Date: Sun, 14 Sep 2014 11:01:29 -0700 (PDT)
The `wdired-mode' doc string says: "If you delete the filename of a
file, it is flagged for deletion in the Dired buffer."  And (emacs)
`Wdired' says: "To mark a file for deletion, delete the entire file
name."

And this applies to directory names also.

No problem, except if you use `ls' switch `-F', which appends `/' to
directory names.  In that case, if you try to use `C-k' anywhere on the
dir name text, you get the error "Text is read only".

(Yes, you can use other deletion keys besides `C-k' to delete the dir
name but not delete the `/'.  That looks weird and is not easily
guessable by users, but it works.  `C-k' should work also, as one would
expect.)

In GNU Emacs 24.4.50.1 (i686-pc-mingw32)
 of 2014-08-15 on LEG570
Bzr revision: 117706 rgm <at> gnu.org-20140815043406-p5hbu97cbm7pulcn
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --enable-checking 'CFLAGS=-O0 -g3' CPPFLAGS=-DGLYPH_DEBUG=1'




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#18475; Package emacs. (Sat, 27 Apr 2019 00:10:02 GMT) Full text and rfc822 format available.

Message #8 received at 18475 <at> debbugs.gnu.org (full text, mbox):

From: "Basil L. Contovounesios" <contovob <at> tcd.ie>
To: Stephen Berman <stephen.berman <at> gmx.net>
Cc: 18475 <at> debbugs.gnu.org, Drew Adams <drew.adams <at> oracle.com>
Subject: Re: bug#18475: 24.4.50;
 Wdired: cannot use C-k to delete a dir name if -F switch used
Date: Sat, 27 Apr 2019 01:09:16 +0100
Drew Adams <drew.adams <at> oracle.com> writes:

> The `wdired-mode' doc string says: "If you delete the filename of a
> file, it is flagged for deletion in the Dired buffer."  And (emacs)
> `Wdired' says: "To mark a file for deletion, delete the entire file
> name."
>
> And this applies to directory names also.
>
> No problem, except if you use `ls' switch `-F', which appends `/' to
> directory names.  In that case, if you try to use `C-k' anywhere on the
> dir name text, you get the error "Text is read only".
>
> (Yes, you can use other deletion keys besides `C-k' to delete the dir
> name but not delete the `/'.  That looks weird and is not easily
> guessable by users, but it works.  `C-k' should work also, as one would
> expect.)
>
> In GNU Emacs 24.4.50.1 (i686-pc-mingw32)
>  of 2014-08-15 on LEG570
> Bzr revision: 117706 rgm <at> gnu.org-20140815043406-p5hbu97cbm7pulcn
> Windowing system distributor `Microsoft Corp.', version 6.1.7601
> Configured using:
>  `configure --enable-checking 'CFLAGS=-O0 -g3' CPPFLAGS=-DGLYPH_DEBUG=1'

I can still reproduce this on Emacs 26 and latest master.

Stephen, any chance you could take a look at this given your recent work
in the area?

Thanks,

-- 
Basil




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#18475; Package emacs. (Sun, 28 Apr 2019 08:15:02 GMT) Full text and rfc822 format available.

Message #11 received at 18475 <at> debbugs.gnu.org (full text, mbox):

From: Stephen Berman <stephen.berman <at> gmx.net>
To: "Basil L. Contovounesios" <contovob <at> tcd.ie>
Cc: 18475 <at> debbugs.gnu.org, Drew Adams <drew.adams <at> oracle.com>
Subject: Re: bug#18475: 24.4.50;
 Wdired: cannot use C-k to delete a dir name if -F switch used
Date: Sun, 28 Apr 2019 10:13:54 +0200
On Sat, 27 Apr 2019 01:09:16 +0100 "Basil L. Contovounesios" <contovob <at> tcd.ie> wrote:

> Drew Adams <drew.adams <at> oracle.com> writes:
>
>> The `wdired-mode' doc string says: "If you delete the filename of a
>> file, it is flagged for deletion in the Dired buffer."  And (emacs)
>> `Wdired' says: "To mark a file for deletion, delete the entire file
>> name."
>>
>> And this applies to directory names also.
>>
>> No problem, except if you use `ls' switch `-F', which appends `/' to
>> directory names.  In that case, if you try to use `C-k' anywhere on the
>> dir name text, you get the error "Text is read only".
>>
>> (Yes, you can use other deletion keys besides `C-k' to delete the dir
>> name but not delete the `/'.  That looks weird and is not easily
>> guessable by users, but it works.  `C-k' should work also, as one would
>> expect.)
>>
>> In GNU Emacs 24.4.50.1 (i686-pc-mingw32)
>>  of 2014-08-15 on LEG570
>> Bzr revision: 117706 rgm <at> gnu.org-20140815043406-p5hbu97cbm7pulcn
>> Windowing system distributor `Microsoft Corp.', version 6.1.7601
>> Configured using:
>>  `configure --enable-checking 'CFLAGS=-O0 -g3' CPPFLAGS=-DGLYPH_DEBUG=1'
>
> I can still reproduce this on Emacs 26 and latest master.
>
> Stephen, any chance you could take a look at this given your recent work
> in the area?

Thanks for bringing this to my attention.  I've started looking at it
and have an idea I'll try to work out.

Steve Berman




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#18475; Package emacs. (Sun, 28 Apr 2019 13:08:01 GMT) Full text and rfc822 format available.

Message #14 received at 18475 <at> debbugs.gnu.org (full text, mbox):

From: "Basil L. Contovounesios" <contovob <at> tcd.ie>
To: Stephen Berman <stephen.berman <at> gmx.net>
Cc: 18475 <at> debbugs.gnu.org, Drew Adams <drew.adams <at> oracle.com>
Subject: Re: bug#18475: 24.4.50;
 Wdired: cannot use C-k to delete a dir name if -F switch used
Date: Sun, 28 Apr 2019 14:07:37 +0100
Stephen Berman <stephen.berman <at> gmx.net> writes:

> On Sat, 27 Apr 2019 01:09:16 +0100 "Basil L. Contovounesios" <contovob <at> tcd.ie> wrote:
>
>> Stephen, any chance you could take a look at this given your recent work
>> in the area?
>
> Thanks for bringing this to my attention.  I've started looking at it
> and have an idea I'll try to work out.

Thanks!

-- 
Basil




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#18475; Package emacs. (Tue, 30 Apr 2019 21:51:02 GMT) Full text and rfc822 format available.

Message #17 received at 18475 <at> debbugs.gnu.org (full text, mbox):

From: Stephen Berman <stephen.berman <at> gmx.net>
To: "Basil L. Contovounesios" <contovob <at> tcd.ie>
Cc: 18475 <at> debbugs.gnu.org, Drew Adams <drew.adams <at> oracle.com>
Subject: Re: bug#18475: 24.4.50;
 Wdired: cannot use C-k to delete a dir name if -F switch used
Date: Tue, 30 Apr 2019 23:50:27 +0200
[Message part 1 (text/plain, inline)]
On Sat, 27 Apr 2019 01:09:16 +0100 "Basil L. Contovounesios" <contovob <at> tcd.ie> wrote:

> Drew Adams <drew.adams <at> oracle.com> writes:
>
>> The `wdired-mode' doc string says: "If you delete the filename of a
>> file, it is flagged for deletion in the Dired buffer."  And (emacs)
>> `Wdired' says: "To mark a file for deletion, delete the entire file
>> name."
>>
>> And this applies to directory names also.
>>
>> No problem, except if you use `ls' switch `-F', which appends `/' to
>> directory names.  In that case, if you try to use `C-k' anywhere on the
>> dir name text, you get the error "Text is read only".
>>
>> (Yes, you can use other deletion keys besides `C-k' to delete the dir
>> name but not delete the `/'.  That looks weird and is not easily
>> guessable by users, but it works.  `C-k' should work also, as one would
>> expect.)
>>
>> In GNU Emacs 24.4.50.1 (i686-pc-mingw32)
>>  of 2014-08-15 on LEG570
>> Bzr revision: 117706 rgm <at> gnu.org-20140815043406-p5hbu97cbm7pulcn
>> Windowing system distributor `Microsoft Corp.', version 6.1.7601
>> Configured using:
>>  `configure --enable-checking 'CFLAGS=-O0 -g3' CPPFLAGS=-DGLYPH_DEBUG=1'
>
> I can still reproduce this on Emacs 26 and latest master.

This happens not only with `/' but also with the other ls file indicator
characters appended when using the -F switch.  Making C-k work as
expected for these cases is a small fix.  However, the same issue also
arises with symlinks, whether or not -F is used, and it does not seem as
straightforward to deal with this case.  The attached patch (against
master) tries to ensure the following behavior:

- Typing `C-k' with point just before the first character of a file name
  ending with an indicator character (using -F), or of link name,
  deletes the file name/symlink in WDired, but it is restored but marked
  for deletion on returning to Dired.  The same also happens when just
  deleting the link name in WDired.

- Typing `C-k' with point on such a file or link name, but after the
  first character, deletes the rest, resulting in renaming on returning
  to Dired.

- Deleting the indicator character is possible in WDired but a noop: the
  character is restored on returning to Dired.  (In the current code
  without the patch, the indicator characters are read-only, but I had
  to change that to make C-k work.)

With symlinks:

- The patch preserves the current behavior that an edit of the target
  name (possible when wdired-allow-to-redirect-links is non-nil, as it
  is by default) is saved on returning to Dired, and if the target name
  is deleted, then the new target on returning to Dired is "/dev/null".

- Editing (changing or deleting) the string " -> " between the link and
  target names is possible in WDired but a noop: the symlink is
  unaltered on returning to Dired.  (In the current code without the
  patch, " -> " is read-only, but I had to change that to make C-k work.)

I've tested these cases, but it is quite possible that I overlooked some
variants or other cases, so I'd appreciate testing and feedback from
others.  (Also, the code still needs more commenting and probably
cleaning up.)

Steve Berman

[Message part 2 (text/x-patch, attachment)]

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#18475; Package emacs. (Sun, 12 May 2019 12:37:02 GMT) Full text and rfc822 format available.

Message #20 received at 18475 <at> debbugs.gnu.org (full text, mbox):

From: "Basil L. Contovounesios" <contovob <at> tcd.ie>
To: Stephen Berman <stephen.berman <at> gmx.net>
Cc: 18475 <at> debbugs.gnu.org, Drew Adams <drew.adams <at> oracle.com>
Subject: Re: bug#18475: 24.4.50;
 Wdired: cannot use C-k to delete a dir name if -F switch used
Date: Sun, 12 May 2019 13:36:04 +0100
Stephen Berman <stephen.berman <at> gmx.net> writes:

> This happens not only with `/' but also with the other ls file indicator
> characters appended when using the -F switch.  Making C-k work as
> expected for these cases is a small fix.  However, the same issue also
> arises with symlinks, whether or not -F is used, and it does not seem as
> straightforward to deal with this case.  The attached patch (against
> master) tries to ensure the following behavior:
>
> - Typing `C-k' with point just before the first character of a file name
>   ending with an indicator character (using -F), or of link name,
>   deletes the file name/symlink in WDired, but it is restored but marked
>   for deletion on returning to Dired.  The same also happens when just
>   deleting the link name in WDired.
>
> - Typing `C-k' with point on such a file or link name, but after the
>   first character, deletes the rest, resulting in renaming on returning
>   to Dired.
>
> - Deleting the indicator character is possible in WDired but a noop: the
>   character is restored on returning to Dired.  (In the current code
>   without the patch, the indicator characters are read-only, but I had
>   to change that to make C-k work.)
>
> With symlinks:
>
> - The patch preserves the current behavior that an edit of the target
>   name (possible when wdired-allow-to-redirect-links is non-nil, as it
>   is by default) is saved on returning to Dired, and if the target name
>   is deleted, then the new target on returning to Dired is "/dev/null".
>
> - Editing (changing or deleting) the string " -> " between the link and
>   target names is possible in WDired but a noop: the symlink is
>   unaltered on returning to Dired.  (In the current code without the
>   patch, " -> " is read-only, but I had to change that to make C-k work.)
>
> I've tested these cases, but it is quite possible that I overlooked some
> variants or other cases, so I'd appreciate testing and feedback from
> others.  (Also, the code still needs more commenting and probably
> cleaning up.)

I can confirm your patch fixes the issue in the OP, but I haven't tested
it extensively.

Thanks for working on this,

-- 
Basil




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#18475; Package emacs. (Tue, 25 Aug 2020 10:12:02 GMT) Full text and rfc822 format available.

Message #23 received at 18475 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: "Basil L. Contovounesios" <contovob <at> tcd.ie>
Cc: 18475 <at> debbugs.gnu.org, Stephen Berman <stephen.berman <at> gmx.net>,
 Drew Adams <drew.adams <at> oracle.com>
Subject: Re: bug#18475: 24.4.50; Wdired: cannot use C-k to delete a dir name
 if -F switch used
Date: Tue, 25 Aug 2020 12:11:22 +0200
"Basil L. Contovounesios" <contovob <at> tcd.ie> writes:

>> I've tested these cases, but it is quite possible that I overlooked some
>> variants or other cases, so I'd appreciate testing and feedback from
>> others.  (Also, the code still needs more commenting and probably
>> cleaning up.)
>
> I can confirm your patch fixes the issue in the OP, but I haven't tested
> it extensively.

The patch no longer applies to Emacs 28, so I've respun the patch
(included below).

I've given it some light testing, and it seems to fix the problem for
me, but there's been other changes in this area in the meantime...
Could people who use wdired a lot give this patch a spin and see whether
it works for them?

diff --git a/lisp/wdired.el b/lisp/wdired.el
index b98becfafe..2c2d3cec25 100644
--- a/lisp/wdired.el
+++ b/lisp/wdired.el
@@ -255,7 +255,7 @@ wdired-change-to-wdired-mode
   (setq buffer-read-only nil)
   (dired-unadvertise default-directory)
   (add-hook 'kill-buffer-hook 'wdired-check-kill-buffer nil t)
-  (add-hook 'after-change-functions 'wdired--restore-dired-filename-prop nil t)
+  (add-hook 'after-change-functions 'wdired--restore-properties nil t)
   (setq major-mode 'wdired-mode)
   (setq mode-name "Editable Dired")
   (setq revert-buffer-function 'wdired-revert)
@@ -266,7 +266,7 @@ wdired-change-to-wdired-mode
   (wdired-preprocess-files)
   (if wdired-allow-to-change-permissions
       (wdired-preprocess-perms))
-  (if (and wdired-allow-to-redirect-links (fboundp 'make-symbolic-link))
+  (if (fboundp 'make-symbolic-link)
       (wdired-preprocess-symlinks))
   (buffer-enable-undo) ; Performance hack. See above.
   (set-buffer-modified-p nil)
@@ -288,6 +288,7 @@ wdired-preprocess-files
   (save-excursion
     (goto-char (point-min))
     (let ((b-protection (point))
+          (used-F (dired-check-switches dired-actual-switches "F" "classify"))
 	  filename)
       (while (not (eobp))
 	(setq filename (dired-get-filename nil t))
@@ -299,8 +300,16 @@ wdired-preprocess-files
 	  (add-text-properties
 	   (1- (point)) (point) `(old-name ,filename rear-nonsticky (read-only)))
 	  (put-text-property b-protection (point) 'read-only t)
-	  (setq b-protection (dired-move-to-end-of-filename t))
+          (dired-move-to-end-of-filename t)
 	  (put-text-property (point) (1+ (point)) 'end-name t))
+          (when (and used-F (looking-at "[*/@|=>]$")) (forward-char))
+          (when (save-excursion
+                  (and (re-search-backward
+                        dired-permission-flags-regexp nil t)
+                       (looking-at "l")
+                       (search-forward " -> " (line-end-position) t)))
+            (goto-char (line-end-position)))
+	  (setq b-protection (point))
         (forward-line))
       (put-text-property b-protection (point-max) 'read-only t))))
 
@@ -327,7 +336,8 @@ wdired-get-filename
 non-nil means don't include directory.  Optional arg OLD with value
 non-nil means return old filename."
   ;; FIXME: Use dired-get-filename's new properties.
-  (let (beg end file)
+  (let ((used-F (dired-check-switches dired-actual-switches "F" "classify"))
+        beg end file)
     (save-excursion
       (setq end (line-end-position))
       (beginning-of-line)
@@ -339,7 +349,20 @@ wdired-get-filename
 	  ;; the filename end is found even when the filename is empty.
 	  ;; Fixes error and spurious newlines when marking files for
 	  ;; deletion.
-	  (setq end (next-single-property-change beg 'end-name))
+	  (setq end (next-single-property-change beg 'end-name nil end))
+          (when (save-excursion
+                  (and (re-search-forward
+                        dired-permission-flags-regexp nil t)
+                       (goto-char (match-beginning 0))
+                       (looking-at "l")
+                       (search-forward " -> " (line-end-position) t)))
+            (goto-char (match-beginning 0))
+            (setq end (point)))
+          (when (and used-F
+                     (save-excursion
+                       (goto-char end)
+                       (looking-back "[*/@|=>]$" (1- (point)))))
+              (setq end (1- end)))
 	  (setq file (buffer-substring-no-properties (1+ beg) end)))
 	;; Don't unquote the old name, it wasn't quoted in the first place
         (and file (setq file (wdired-normalize-filename file (not old)))))
@@ -366,7 +389,7 @@ wdired-change-to-dired-mode
   (setq mode-name "Dired")
   (dired-advertise)
   (remove-hook 'kill-buffer-hook 'wdired-check-kill-buffer t)
-  (remove-hook 'after-change-functions 'wdired--restore-dired-filename-prop t)
+  (remove-hook 'after-change-functions 'wdired--restore-properties t)
   (set (make-local-variable 'revert-buffer-function) 'dired-revert))
 
 
@@ -427,9 +450,9 @@ wdired-finish-edit
     (when files-renamed
       (setq errors (+ errors (wdired-do-renames files-renamed))))
     ;; We have to be in wdired-mode when wdired-do-renames is executed
-    ;; so that wdired--restore-dired-filename-prop runs, but we have
-    ;; to change back to dired-mode before reverting the buffer to
-    ;; avoid using wdired-revert, which changes back to wdired-mode.
+    ;; so that wdired--restore-properties runs, but we have to change
+    ;; back to dired-mode before reverting the buffer to avoid using
+    ;; wdired-revert, which changes back to wdired-mode.
     (wdired-change-to-dired-mode)
     (if changes
 	(progn
@@ -451,7 +474,11 @@ wdired-finish-edit
 				'(old-name nil end-name nil old-link nil
 					   end-link nil end-perm nil
 					   old-perm nil perm-changed nil))
-	(message "(No changes to be performed)")))
+	(message "(No changes to be performed)")
+        ;; Deleting file indicator characters or editing the symlink
+        ;; arrow in WDired are noops, so redisplay them immediately on
+        ;; returning to Dired.
+        (revert-buffer)))
     (when files-deleted
       (wdired-flag-for-deletion files-deleted))
     (when (> errors 0)
@@ -605,14 +632,24 @@ wdired-check-kill-buffer
 ;; dired-filename text property, which allows functions that look for
 ;; this property (e.g. dired-isearch-filenames) to work in wdired-mode
 ;; and also avoids an error with non-nil wdired-use-interactive-rename
-;; (bug#32173).
-(defun wdired--restore-dired-filename-prop (beg end _len)
+;; (bug#32173).  Also prevents editing the symlink arrow (which is a
+;; noop) from corrupting the link name (see bug#18475 for elaboration).
+(defun wdired--restore-properties (beg end _len)
   (save-match-data
     (save-excursion
       (let ((lep (line-end-position))
             (used-F (dired-check-switches
                      dired-actual-switches
                      "F" "classify")))
+        ;; Deleting the space between the link name and the arrow (a
+        ;; noop) also deletes the end-name property, so restore it.
+        (when (and (save-excursion
+                     (re-search-backward dired-permission-flags-regexp nil t)
+                     (looking-at "l"))
+                   (get-text-property (1- (point)) 'dired-filename)
+                   (not (get-text-property (point) 'dired-filename))
+                   (not (get-text-property (point) 'end-name)))
+            (put-text-property (point) (1+ (point)) 'end-name t))
         (beginning-of-line)
         (when (re-search-forward
                directory-listing-before-filename-regexp lep t)
@@ -676,33 +713,36 @@ wdired-preprocess-symlinks
     (save-excursion
       (goto-char (point-min))
       (while (not (eobp))
-        (if (looking-at dired-re-sym)
-            (progn
-              (re-search-forward " -> \\(.*\\)$")
-	      (put-text-property (- (match-beginning 1) 2)
-				 (1- (match-beginning 1)) 'old-link
-				 (match-string-no-properties 1))
-              (put-text-property (match-end 1) (1+ (match-end 1)) 'end-link t)
-              (put-text-property (1- (match-beginning 1))
-				 (match-beginning 1)
-				 'rear-nonsticky '(read-only))
-	      (put-text-property (match-beginning 1)
-				 (match-end 1) 'read-only nil)))
+        (when (looking-at dired-re-sym)
+          (re-search-forward " -> \\(.*\\)$")
+	  (put-text-property (1- (match-beginning 1))
+			     (match-beginning 1) 'old-link
+			     (match-string-no-properties 1))
+          (put-text-property (match-end 1) (1+ (match-end 1)) 'end-link t)
+          (unless wdired-allow-to-redirect-links
+            (put-text-property (match-beginning 0)
+			       (match-end 1) 'read-only t)))
         (forward-line)))))
 
-
 (defun wdired-get-previous-link (&optional old move)
   "Return the next symlink target.
 If OLD, return the old target.  If MOVE, move point before it."
   (let (beg end target)
     (setq beg (previous-single-property-change (point) 'old-link nil))
-    (if beg
-	(progn
-	  (if old
-	      (setq target (get-text-property (1- beg) 'old-link))
-	    (setq end (next-single-property-change beg 'end-link))
-	    (setq target (buffer-substring-no-properties (1+ beg) end)))
-	  (if move (goto-char (1- beg)))))
+    (when beg
+      (when (save-excursion
+              (goto-char beg)
+              (and (looking-at " ")
+                   (looking-back " ->" (line-beginning-position))))
+        (setq beg (1+ beg)))
+      (if old
+          (setq target (get-text-property (1- beg) 'old-link))
+        (setq end (save-excursion
+                    (goto-char beg)
+                    (next-single-property-change beg 'end-link nil
+                                                 (line-end-position))))
+        (setq target (buffer-substring-no-properties beg end)))
+      (if move (goto-char (1- beg))))
     (and target (wdired-normalize-filename target t))))
 
 (declare-function make-symbolic-link "fileio.c")


-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no




Added tag(s) patch. Request was from Lars Ingebrigtsen <larsi <at> gnus.org> to control <at> debbugs.gnu.org. (Tue, 25 Aug 2020 10:13:01 GMT) Full text and rfc822 format available.

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#18475; Package emacs. (Wed, 26 Aug 2020 10:12:01 GMT) Full text and rfc822 format available.

Message #28 received at 18475 <at> debbugs.gnu.org (full text, mbox):

From: Stephen Berman <stephen.berman <at> gmx.net>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: "Basil L. Contovounesios" <contovob <at> tcd.ie>, 18475 <at> debbugs.gnu.org,
 Drew Adams <drew.adams <at> oracle.com>
Subject: Re: bug#18475: 24.4.50; Wdired: cannot use C-k to delete a dir name
 if -F switch used
Date: Wed, 26 Aug 2020 12:11:10 +0200
On Tue, 25 Aug 2020 12:11:22 +0200 Lars Ingebrigtsen <larsi <at> gnus.org> wrote:

> "Basil L. Contovounesios" <contovob <at> tcd.ie> writes:
>
>>> I've tested these cases, but it is quite possible that I overlooked some
>>> variants or other cases, so I'd appreciate testing and feedback from
>>> others.  (Also, the code still needs more commenting and probably
>>> cleaning up.)
>>
>> I can confirm your patch fixes the issue in the OP, but I haven't tested
>> it extensively.
>
> The patch no longer applies to Emacs 28, so I've respun the patch
> (included below).

Thanks.

> I've given it some light testing, and it seems to fix the problem for
> me, but there's been other changes in this area in the meantime...
> Could people who use wdired a lot give this patch a spin and see whether
> it works for them?

In particular, there were changes to fix problems with using the -F
switch on macOS, right?  So it would be good for people who use that
system (I don't) to test the patch.

Steve Berman




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#18475; Package emacs. (Sun, 11 Oct 2020 04:07:02 GMT) Full text and rfc822 format available.

Message #31 received at 18475 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Stephen Berman <stephen.berman <at> gmx.net>
Cc: "Basil L. Contovounesios" <contovob <at> tcd.ie>, 18475 <at> debbugs.gnu.org,
 Drew Adams <drew.adams <at> oracle.com>
Subject: Re: bug#18475: 24.4.50; Wdired: cannot use C-k to delete a dir name
 if -F switch used
Date: Sun, 11 Oct 2020 06:06:39 +0200
Stephen Berman <stephen.berman <at> gmx.net> writes:

> In particular, there were changes to fix problems with using the -F
> switch on macOS, right?  So it would be good for people who use that
> system (I don't) to test the patch.

Nobody piped up, so I've pushed the patch to the trunk, although we'll
have to be prepared to revert it if it turns out to be problematic on
Macos.  I'll try to do some testing myself...

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no




Added tag(s) fixed. Request was from Lars Ingebrigtsen <larsi <at> gnus.org> to control <at> debbugs.gnu.org. (Sun, 11 Oct 2020 04:07:02 GMT) Full text and rfc822 format available.

bug marked as fixed in version 28.1, send any further explanations to 18475 <at> debbugs.gnu.org and Drew Adams <drew.adams <at> oracle.com> Request was from Lars Ingebrigtsen <larsi <at> gnus.org> to control <at> debbugs.gnu.org. (Sun, 11 Oct 2020 04:07:02 GMT) Full text and rfc822 format available.

bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Sun, 08 Nov 2020 12:24:08 GMT) Full text and rfc822 format available.

This bug report was last modified 3 years and 167 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.