[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: vc-edit-next-command (was bug#57807: 29.0.50; Make vc-print-branch-l
From: |
Sean Whitton |
Subject: |
Re: vc-edit-next-command (was bug#57807: 29.0.50; Make vc-print-branch-log able to log multiple branches) |
Date: |
Tue, 27 Sep 2022 18:15:01 -0700 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/29.0.50 (gnu/linux) |
Hello,
On Tue 27 Sep 2022 at 09:59PM +03, Juri Linkov wrote:
>>> Thanks, I'm going to try it out.
>>
>> It looks like I need to call (minibuffer-depth) like
>> display-buffer-override-next-command does. Updated patch attached.
>
> I tested your latest patch with many vc commands, and everything
> works nicely, even M-x for vc commands without a keybinding
> and with a prefix arg is supported like in
>
> C-x v ! C-u M-x vc-log-search RET
>
> (C-u is not needed for vc-log-search anymore,
> it's just for testing a prefix arg here).
Thank you for the through testing.
Here is a more formal patch, including manual updates.
>> I realised that we could add an override inside vc-read-revision, and
>> potentially other places, which makes it considerably less annoying.
>> Done in the attached.
>
> Maybe it's fine to disable editing for vc-read-revision,
> but I'm not sure about other commands, such as e.g.
> vc-git-log-incoming that runs two git commands:
>
> vc-git-command fetch
> vc-git-command log
>
> I see no problem to type an extra RET even for a secondary git command.
> But it helps in case when someone really wants to edit it.
Yeah, we should be conservative. I've done it for only vc-read-revision
for now, and perhaps I'll raise some others later.
--
Sean Whitton
v3-0001-Add-vc-edit-next-command.patch
Description: Text Data
- Re: vc-edit-next-command (was bug#57807: 29.0.50; Make vc-print-branch-log able to log multiple branches),
Sean Whitton <=