GNU bug report logs - #78987
30.1; Unable to create new Git repository in Dired mode

Previous Next

Package: emacs;

Reported by: "Zipei Lu" <zipeilu <at> qq.com>

Date: Wed, 9 Jul 2025 21:57:09 UTC

Severity: normal

Found in version 30.1

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

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Stephen Berman <stephen.berman <at> gmx.net>
To: Sean Whitton <spwhitton <at> spwhitton.name>
Cc: zipeilu <at> qq.com, 78987 <at> debbugs.gnu.org
Subject: bug#78987: 30.1; Unable to create new Git repository in Dired mode
Date: Sat, 12 Jul 2025 11:12:35 +0200
On Sat, 12 Jul 2025 11:07:28 +0200 Stephen Berman <stephen.berman <at> gmx.net> wrote:

> On Fri, 11 Jul 2025 19:13:59 +0100 Sean Whitton <spwhitton <at> spwhitton.name> wrote:
>
>> Hello,
>>
>> On Fri 11 Jul 2025 at 11:33am +02, Stephen Berman via "Bug reports for GNU
>> Emacs, the Swiss army knife of text editors" wrote:
>>
>>> When I test with -Q, a Git repository is created and either the file at
>>> point or the marked files in Dired are added to it, although
>>> `vc-deduce-fileset' calls `(dired-vc-deduce-fileset
>>> state-model-only-files not-state-changing)', i.e., without passing
>>> `allow-unregistered'; so why do you think it must be passed (and
>>> non-nil)?
>>
>> It should be passed because the corresponding code path for vc-dir
>> passes allow-unregistered down.  dired-vc-deduce-fileset is meant to be
>> just a dired version of the vc-dir code path in vc-deduce-fileset.
>
> I admit that I find the VC code hard to follow, but I don't see where
> `vc-dir' uses ALLOW-UNREGISTERED.  The docstring of `vc-deduce-fileset'
> says:
>
>   If the current buffer is in `vc-dir' or Dired mode, FILESET is the
>   list of marked files, or the file under point if no files are
>   marked.
>   [...]
>   Otherwise, if ALLOW-UNREGISTERED is non-nil and the visited file
>   is unregistered, FILESET is a single-file list containing the
>   name of the visited file.
>
> This seems to say that the code path using ALLOW-UNREGISTERED bypasses
> the code path using Dired.  Anyway, I'm sure you understand the VC

                     ^ either vc-dir or 

> better than I do, so you should make the change you think is required.
>
> Steve Berman




This bug report was last modified 31 days ago.

Previous Next


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