GNU bug report logs - #72341
VC: CVS template lines not stripped when committing

Previous Next

Package: emacs;

Reported by: Christoph Badura <bad <at> bsd.de>

Date: Sun, 28 Jul 2024 16:36:02 UTC

Severity: normal

Tags: patch

Fixed in version 31.1

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

Bug is archived. No further changes may be made.

Full log


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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Dmitry Gutov <dmitry <at> gutov.dev>
Cc: bad <at> bsd.de, 72341 <at> debbugs.gnu.org
Subject: Re: bug#72341: VC: CVS template lines not stripped when committing
Date: Fri, 02 Aug 2024 10:19:30 +0300
> Date: Tue, 30 Jul 2024 16:35:29 +0300
> Cc: 72341 <at> debbugs.gnu.org
> From: Dmitry Gutov <dmitry <at> gutov.dev>
> 
> On 29/07/2024 15:34, Eli Zaretskii wrote:
> > I'm unsure how to proceed with this.  My bother is that this is a
> > definite change in behavior wrt what VC did until now.  Users of VC
> > might be unaware of this removal, and could start some log lines with
> > the prefix, which will mysteriously disappear from the log message.
> > 
> > Dmitry, WDYT?  Maybe we should add this removal guarded by a user
> > option, by default off?  Or maybe we can identify where the template
> > starts and ends, and only remove in that region?
> 
> If we do this, we'd only remove the lines starting with "CVS:". We could 
> also add special syntax highlighting for them. That would probably be 
> enough.

Agreed.

> Then vc-cvs-checkin would additionally process the commit message string 
> returned by log-edit-extract-headers.
> 
> Note that it seems we only handle "templates" for CVS and RCS, so there 
> is no prior art for implementing this feature.

Right.  Patches welcome to implement this.




This bug report was last modified 73 days ago.

Previous Next


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