GNU bug report logs - #76769
31.0.50; marking inconsistency between VC-Dir and dired-vc-next-action

Previous Next

Package: emacs;

Reported by: Sean Whitton <spwhitton <at> spwhitton.name>

Date: Thu, 6 Mar 2025 03:58:02 UTC

Severity: normal

Found in version 31.0.50

Done: Sean Whitton <spwhitton <at> spwhitton.name>

Bug is archived. No further changes may be made.

Full log


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

From: Sean Whitton <spwhitton <at> spwhitton.name>
To: Dmitry Gutov <dmitry <at> gutov.dev>
Cc: sbaugh <at> janestreet.com, 76769 <at> debbugs.gnu.org, juri <at> linkov.net
Subject: Re: bug#76769: 31.0.50; marking inconsistency between VC-Dir and
 dired-vc-next-action
Date: Sat, 08 Mar 2025 10:40:09 +0800
Hello,

On Fri 07 Mar 2025 at 05:13am +02, Dmitry Gutov wrote:

> Hi!
>
> On 06/03/2025 05:56, Sean Whitton wrote:
>>    --8<---------------cut here---------------start------------->8---
>>                             ./
>>      *                      src/
>>      *  edited              src/emacs.c
>>    --8<---------------cut here---------------end--------------->8---
>>    Currently vc-dir-deduce-fileset returns ("src/" "src/emacs.c"),
>>    it should return just ("src/").
>
> Could you give an example of when this is a problem in practice? If you just
> pass this list to 'git diff' or 'git log', either command would behave as if
> the fileset contained only a directory.
>
> I suppose the list of files in log-edit would be more awkward. Is that all?

No, my thought here is that the VC-Dir abstraction is leaking through.

Modulo possible cases which I'm hoping to hear more about from Eli, if
the user has marked a directory, then the operation is on that directory
and everything in it, and passing through the information that the user
happened to have src/emacs.c marked as well is strange.

> Speaking of interface changes, I wonder if marking (or unmarking) a directory
> should select/unselect all of its files instead (those displayed in the
> buffer). But that's a more drastic change.

I hope that we can figure out something more subtle, but having the
ability to opt into that would be an improvement over what we have now,
I think.

-- 
Sean Whitton




This bug report was last modified 49 days ago.

Previous Next


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