GNU bug report logs - #3860
vc-svn calls "svn resolved" without permission

Previous Next

Package: emacs;

Reported by: James Y Knight <jknight <at> fuhm.net>

Date: Wed, 15 Jul 2009 20:30:03 UTC

Severity: normal

Fixed in version 31.1

Done: Mauro Aranda <maurooaranda <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


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

From: Mauro Aranda <maurooaranda <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 3860 <at> debbugs.gnu.org, larsi <at> gnus.org, jknight <at> fuhm.net, spiegel <at> gnu.org,
 dgutov <at> yandex.ru
Subject: Re: bug#3860: vc-svn calls "svn resolved" without permission
Date: Thu, 13 Mar 2025 12:11:21 -0300
[Message part 1 (text/plain, inline)]
Eli Zaretskii <eliz <at> gnu.org> writes:

>> Date: Thu, 13 Mar 2025 09:04:52 -0300
>> Cc: Eli Zaretskii <eliz <at> gnu.org>, Dmitry Gutov <dgutov <at> yandex.ru>,
>>  Lars Ingebrigtsen <larsi <at> gnus.org>, spiegel <at> gnu.org, jknight <at> fuhm.net
>> From: Mauro Aranda <maurooaranda <at> gmail.com>
>>
>> Eli Zaretskii <eliz <at> gnu.org> writes:
>>
>>  >> Cc: 3860 <at> debbugs.gnu.org, jknight <at> fuhm.net, spiegel <at> gnu.org
>>  >> From: Dmitry Gutov <dgutov <at> yandex.ru>
>>  >> Date: Fri, 22 Jan 2021 00:35:07 +0200
>>  >>
>>  >> On 21.01.2021 21:33, Eli Zaretskii wrote:
>>  >> > I think if we are going to make this optional with SVN, we 
should make
>>  >> > it optional with the other back-ends as well.  VC always 
presented a
>>  >> > uniform UI and user experience with all back-ends, as far as 
possible
>>  >> > and reasonable.  In this case, I think automatically marking the
>>  >> > conflicts as resolved makes a lot of sense, but I won't object 
to make
>>  >> > it opt-out, as long as it's done for every VCS that can 
support it.
>>  >>
>>  >> So someone might want to generalize the Git-specific variable:
>>  >> vc-git-resolve-conflicts.
>>  >
>>  > Yes, I think so.
>>  >
>>  >> But they'd have to consider what to do with the special value
>>  >> 'unstage-maybe', which doesn't really make sense for other backends.
>>  >
>>  > Right.
>>
>> I took a shot at it.  Please review the attached patch.
>
> Thanks.
>
>> +(defcustom vc-bzr-resolve-conflicts 'default
>> +  "When non-nil, mark conflicted file as resolved upon saving.
>
> "If non-nil" is better, IMO.

I was going to change it to that, but I realized the text was
misleading.  So I rephrased it.

>> +If this is non-nil and there are no more conflict markers in the file,
>> +VC will mark the saved file as resolved.
>    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> "VC will mark the conflicts in the saved file as resolved."
>
> Same comments for doc strings of all the other options, since this
> text is repeated in all of them.

Thanks.  Revised patch attached.
[0001-Make-marking-conflicted-files-as-resolved-upon-savin.patch (text/x-patch, attachment)]

This bug report was last modified 53 days ago.

Previous Next


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