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: Sean Whitton <spwhitton <at> spwhitton.name>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: stephen.berman <at> gmx.net, 78987 <at> debbugs.gnu.org, Juri Linkov <juri <at> linkov.net>
Subject: bug#78987: 30.1; Unable to create new Git repository in Dired mode
Date: Sun, 13 Jul 2025 12:12:20 +0100
Hello,

On Sun 13 Jul 2025 at 10:54am +03, Eli Zaretskii wrote:

>> Cc: 78987 <at> debbugs.gnu.org
>> From: Juri Linkov <juri <at> linkov.net>
>> Date: Sun, 13 Jul 2025 09:58:59 +0300
>>
>> >  (defun dired-vc-deduce-fileset
>> >      (&optional state-model-only-files not-state-changing)
>> > -  (let ((backend (vc-responsible-backend default-directory))
>> > -        (files (dired-get-marked-files nil nil nil nil t)))
>> > +  (let* ((files (dired-get-marked-files nil nil nil nil t))
>> > +         (backend (or (vc-responsible-backend default-directory t)
>> > +                      (vc-backend-for-registration (car files)))))
>>
>> The original recipe still fails with the same error.
>> Shouldn't 'vc-responsible-backend' be wrapped with
>> 'ignore-errors' in 'dired-vc-deduce-fileset'?
>
> There's no need for ignore-errors; instead, vc-responsible-backend
> should be called with the second argument NO-ERROR non-nil.
>
> The original patch did precisely that, so it looks like some snafu
> when applying the patch; now fixed.

Oh, my fault.  Thank you for the fix.

-- 
Sean Whitton




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.