GNU bug report logs - #77140
[PATCH] Strip text properties on recentf entries

Previous Next

Package: emacs;

Reported by: Ship Mints <shipmints <at> gmail.com>

Date: Thu, 20 Mar 2025 16:57:01 UTC

Severity: normal

Tags: patch

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


Message #49 received at 77140-done <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Ship Mints <shipmints <at> gmail.com>
Cc: 77140-done <at> debbugs.gnu.org, stefankangas <at> gmail.com
Subject: Re: bug#77140: [PATCH] Strip text properties on recentf entries
Date: Sat, 19 Apr 2025 17:10:55 +0300
> From: Ship Mints <shipmints <at> gmail.com>
> Date: Wed, 9 Apr 2025 16:14:04 -0400
> Cc: stefankangas <at> gmail.com, 77140 <at> debbugs.gnu.org
> 
> On Fri, Mar 21, 2025 at 11:13 AM Ship Mints <shipmints <at> gmail.com> wrote:
> 
>  On Fri, Mar 21, 2025 at 11:07 AM Eli Zaretskii <eliz <at> gnu.org> wrote:
> 
>  > From: Ship Mints <shipmints <at> gmail.com>
>  > Date: Fri, 21 Mar 2025 10:02:56 -0400
>  > Cc: stefankangas <at> gmail.com, 77140 <at> debbugs.gnu.org
>  > 
>  >  Thanks, but please make the lines in the log message shorter (see
>  >  CONTRIBUTE for guidance).  This includes both the heading and the rest
>  >  of the log message.
>  > 
>  > This commit log entry summary is 64 chars (less than the 79 max target) and the rest of the log
>  is filled to 72
>  > (.dir-locals.el is in effect), so I'm missing what you are suggesting.
> 
>  The log message is indented with a TAB when we generate the ChangeLog
>  file from the Git log, so the 72-character lines will be 81, more than
>  we'd like to see in ChangeLog.  CONTRIBUTE says "Lines in ChangeLog
>  entries should preferably be not longer than 63 characters" for that
>  reason.  Please try not to exceed this limit, unless you really must
>  (like if you need to quote a long URL there).
> 
>  Got it.  I've adjusted my 'git-commit-mode' 'fill-column' accordingly.  Revised patch attached.
> 
> Possible to install this patch?  I've attached the same copy of the patch from the above so it's easy to find.

Thanks, now installed, and closing the bug.




This bug report was last modified 85 days ago.

Previous Next


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