GNU bug report logs - #67946
30.0.50; [RFC PATCH] Add an option to not report "Wrote ..." when saving recentf.

Message received at 67946 <at> debbugs.gnu.org:

Received: (at 67946) by debbugs.gnu.org; 26 Dec 2023 12:42:34 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Tue Dec 26 07:42:34 2023
Received: from localhost ([127.0.0.1]:56074 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1rI6lV-0006pp-Ru
	for submit <at> debbugs.gnu.org; Tue, 26 Dec 2023 07:42:34 -0500
Received: from eggs.gnu.org ([2001:470:142:3::10]:45510)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <eliz@HIDDEN>) id 1rI6lT-0006pb-LA
 for 67946 <at> debbugs.gnu.org; Tue, 26 Dec 2023 07:42:32 -0500
Received: from fencepost.gnu.org ([2001:470:142:3::e])
 by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)
 (Exim 4.90_1) (envelope-from <eliz@HIDDEN>)
 id 1rI6lG-0001NV-VG; Tue, 26 Dec 2023 07:42:18 -0500
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org;
 s=fencepost-gnu-org; h=References:Subject:In-Reply-To:To:From:Date:
 mime-version; bh=Ds3HS3Qo7s/9584XC7+WnQb7FxeCPthu/l7kMpz7dU0=; b=OUkwLrxs0TpU
 2tnmDbY3Y8GqueobAybV+d9TUWKkGawJaQP7ZslDULZw1fc1Hkdl/PYgSxO1xG206DT+um4V3KJ5u
 sZhbN6ZOK0lSLZ9lU/5oWpBGmZnjn2peqx2T8Y/Ej2ta4ikOvGofw42SjnSD36yX5USVm22MnQkZD
 7seFKU/yf5hd3AtsLE/YpRSdvda8+1CQ/Wboz9nVaxLRtSnJLdPFiOZ0SIwdaDv5ZV3tzBmSUzAde
 ThuLEJrcljGdgGeDtcVDGMVPgWSprb1A8+ojP+NAWxz08kbPp0mbp2dyErbjcfSo44Q9psp82r/Sg
 sF3q0xmpteHStysEzWrMrQ==;
Date: Tue, 26 Dec 2023 14:41:56 +0200
Message-Id: <837cl19luz.fsf@HIDDEN>
From: Eli Zaretskii <eliz@HIDDEN>
To: Vladimir Nikishkin <for_emacs-recentf-patch_2023-12-21@HIDDEN>
In-Reply-To: <87plyt4i6k.fsf@HIDDEN> (message from Vladimir
 Nikishkin on Tue, 26 Dec 2023 13:58:45 +0800)
Subject: Re: bug#67946: 30.0.50; [RFC PATCH] Add an option to not report
 "Wrote ..." when saving recentf.
References: <87a5q41azk.fsf@HIDDEN>
 <CADwFkmkJSXe4CWNJtwgjwJuJ4SihwPhgXpM1OnRFgJMZ=8FX1Q@HIDDEN>
 <87wmt2505y.fsf@HIDDEN> <834jg6bhaf.fsf@HIDDEN>
 <CADwFkmk4W1TAFpsGnKv_QB8pu9iO1x4Y4xT2H=apmo3enq=jhQ@HIDDEN>
 <87plyt4i6k.fsf@HIDDEN>
X-Spam-Score: -2.3 (--)
X-Debbugs-Envelope-To: 67946
Cc: 67946 <at> debbugs.gnu.org, stefankangas@HIDDEN
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -3.3 (---)

> From: Vladimir Nikishkin <for_emacs-recentf-patch_2023-12-21@HIDDEN>
> Cc: Eli Zaretskii <eliz@HIDDEN>, 67946 <at> debbugs.gnu.org
> Date: Tue, 26 Dec 2023 13:58:45 +0800
> 
> Stefan Kangas <stefankangas@HIDDEN> writes:
> 
> > Eli Zaretskii <eliz@HIDDEN> writes:
> >
> >> If we decide to turn it off, we will need a knob to revert to the old
> >> behavior, because after so many years of the existing behavior,
> >> someone somewhere might be used to it.
> >
> > I think the existing patch basically has that already, but it seems to
> > be lacking a NEWS entry and a :version tag for the defcustom before it
> > can go in.
> >
> > Vladimir, could you fix that and send a new patch?
> 
> Added an entry to etc/NEWS, and, seems, fixed the actual verbosity
> message printing condition.

Thanks, a few minor comments:

> --- a/etc/NEWS
> +++ b/etc/NEWS
> @@ -1046,7 +1046,11 @@ desktop restoration to continue.
>  Similarly to buffer restoration by Desktop, 'recentf-mode' checking
>  of the accessibility of remote files can now time out if
>  'remote-file-name-access-timeout' is set to a positive number.
> -

That empty separator line should not be deleted.

> +*** New use option 'recentf-verbose'.
           ^^^
"user"

> +By default 'recentf-save-list' prints a message when saving the
> +recentf list. The new option, if set to nil, suppresses this message.
               ^^
Two spaces between sentences, please.

> +If more debugging info suppression switches are to be added, this
> +option might become more useful.

I don't think we need this last sentence in NEWS.  NEWS is about
changes that are actually done, not about potential changes in the
future.

> +(defcustom recentf-verbose 1
> +  "0 means to not show debugging messages related to the recentf machinery.
> +1 means show messages that were printed by default on Emacs <= 29.1."

A doc string should start with a single sentence that summarizes what
the variable or function does.  In this case, I suggest

  Whether to show verbose messages about low-level recentf actions.

Also, if we are going to use an integer here, the variable should be
called something like recentf-verbosity.  Alternatively, we could make
it a simple boolean, since we have no plans to extend it to more than
2 levels.

> +  :group 'recentf

Do you need this :group tag?  It shouldn't be needed in this case, I
think.

Finally, the commit log message is still missing.

Message received at 67946 <at> debbugs.gnu.org:

Received: (at 67946) by debbugs.gnu.org; 26 Dec 2023 06:00:52 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Tue Dec 26 01:00:52 2023
Received: from localhost ([127.0.0.1]:55691 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1rI0Um-00033D-1q
	for submit <at> debbugs.gnu.org; Tue, 26 Dec 2023 01:00:52 -0500
Received: from public.lockywolf.net ([46.23.90.181]:21371)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <for_emacs-recentf-patch_2023-12-21@HIDDEN>)
 id 1rI0Uf-0002qR-Pn
 for 67946 <at> debbugs.gnu.org; Tue, 26 Dec 2023 01:00:50 -0500
DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; s=2021-05-20; bh=SsJMpn4id
 pVSKiPgW0sX58ueOSKnicXJ5BNvfl85elc=;
 h=in-reply-to:date:subject:cc:to:
 from:references; d=lockywolf.net; b=HvR5YLHu709FiLvX9LEO1cKrmUIiR1XEDE
 luNKnhEyePC/E0CFNm+AIP9CCJqE9Sqtrui2KP8UlUSL15BrSKdiIXbDaOVqU9NPiKrywc
 5IPxE3q297WzjDABvya9m5DAw++zaFhdd7ZD/ThF78XDrzH6Pd1iSas6ykA5+0mQJzA=
Received: from laptop.lockywolf.net (<unknown> [2001:470:24:315::102])
 by public.lockywolf.net (OpenSMTPD) with ESMTPSA id 10508e90
 (TLSv1.3:TLS_AES_256_GCM_SHA384:256:NO); 
 Tue, 26 Dec 2023 07:00:32 +0100 (CET)
References: <87a5q41azk.fsf@HIDDEN>
 <CADwFkmkJSXe4CWNJtwgjwJuJ4SihwPhgXpM1OnRFgJMZ=8FX1Q@HIDDEN>
 <87wmt2505y.fsf@HIDDEN> <834jg6bhaf.fsf@HIDDEN>
 <CADwFkmk4W1TAFpsGnKv_QB8pu9iO1x4Y4xT2H=apmo3enq=jhQ@HIDDEN>
User-agent: mu4e 1.10.7; emacs 30.0.50
From: Vladimir Nikishkin <for_emacs-recentf-patch_2023-12-21@HIDDEN>
To: Stefan Kangas <stefankangas@HIDDEN>
Subject: Re: bug#67946: 30.0.50; [RFC PATCH] Add an option to not report
 "Wrote ..." when saving recentf.
Date: Tue, 26 Dec 2023 13:58:45 +0800
In-reply-to: <CADwFkmk4W1TAFpsGnKv_QB8pu9iO1x4Y4xT2H=apmo3enq=jhQ@HIDDEN>
Message-ID: <87plyt4i6k.fsf@HIDDEN>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="=-=-="
X-Spam-Score: -0.0 (/)
X-Debbugs-Envelope-To: 67946
Cc: Eli Zaretskii <eliz@HIDDEN>, 67946 <at> debbugs.gnu.org
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -1.0 (-)

--=-=-=
Content-Type: text/plain


Stefan Kangas <stefankangas@HIDDEN> writes:

> Eli Zaretskii <eliz@HIDDEN> writes:
>
>> If we decide to turn it off, we will need a knob to revert to the old
>> behavior, because after so many years of the existing behavior,
>> someone somewhere might be used to it.
>
> I think the existing patch basically has that already, but it seems to
> be lacking a NEWS entry and a :version tag for the defcustom before it
> can go in.
>
> Vladimir, could you fix that and send a new patch?

Added an entry to etc/NEWS, and, seems, fixed the actual verbosity
message printing condition.

-- 
Your sincerely,
Vladimir Nikishkin (MiEr, lockywolf)
(Laptop)

--=-=-=
Content-Type: text/x-patch
Content-Disposition: attachment; filename=001_patch-recentf-verbose_v2.patch

diff --git a/etc/NEWS b/etc/NEWS
index 1ff2f8a149f..9bc58e790f5 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -1046,7 +1046,11 @@ desktop restoration to continue.
 Similarly to buffer restoration by Desktop, 'recentf-mode' checking
 of the accessibility of remote files can now time out if
 'remote-file-name-access-timeout' is set to a positive number.
-
+*** New use option 'recentf-verbose'.
+By default 'recentf-save-list' prints a message when saving the
+recentf list. The new option, if set to nil, suppresses this message.
+If more debugging info suppression switches are to be added, this
+option might become more useful.
 ** Notifications
 
 +++
diff --git a/lisp/recentf.el b/lisp/recentf.el
index 94ae871763b..616bd373232 100644
--- a/lisp/recentf.el
+++ b/lisp/recentf.el
@@ -319,6 +319,13 @@ recentf-show-file-shortcuts-flag
 used as shortcuts to open the Nth file."
   :group 'recentf
   :type 'boolean)
+
+(defcustom recentf-verbose 1
+  "0 means to not show debugging messages related to the recentf machinery.
+1 means show messages that were printed by default on Emacs <= 29.1."
+  :group 'recentf
+  :type 'integer
+  :version "30.1")
 
 ;;; Utilities
 ;;
@@ -1324,7 +1331,12 @@ recentf-save-list
                 (format ";; coding: %s\n" recentf-save-file-coding-system)
                 ";; End:\n")
         (write-region (point-min) (point-max)
-                      (expand-file-name recentf-save-file))
+                      (expand-file-name recentf-save-file)
+                      nil
+                      (if (= recentf-verbose 1)
+                          nil ;; show "Wrote ..." messages.
+                        1) ;; non-nil, non-t, non-string, do not show.
+                      )
         (when recentf-save-file-modes
           (set-file-modes recentf-save-file recentf-save-file-modes))
         nil)

--=-=-=--

Message received at 67946 <at> debbugs.gnu.org:

Received: (at 67946) by debbugs.gnu.org; 25 Dec 2023 21:50:33 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Mon Dec 25 16:50:33 2023
Received: from localhost ([127.0.0.1]:55487 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1rHsqH-00060l-0I
	for submit <at> debbugs.gnu.org; Mon, 25 Dec 2023 16:50:33 -0500
Received: from mail-ed1-x529.google.com ([2a00:1450:4864:20::529]:47214)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <stefankangas@HIDDEN>) id 1rHsqB-00060S-Rh
 for 67946 <at> debbugs.gnu.org; Mon, 25 Dec 2023 16:50:31 -0500
Received: by mail-ed1-x529.google.com with SMTP id
 4fb4d7f45d1cf-554e9f51522so668339a12.2
 for <67946 <at> debbugs.gnu.org>; Mon, 25 Dec 2023 13:50:20 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=gmail.com; s=20230601; t=1703541015; x=1704145815; darn=debbugs.gnu.org;
 h=cc:to:subject:message-id:date:mime-version:references:in-reply-to
 :from:from:to:cc:subject:date:message-id:reply-to;
 bh=ycJecH0/mIKGG9/EL17m1H/LPW9eoW3GK4s+JxRvzi8=;
 b=iyyzcgR9zodIsUXpu1BL9Rtvxx+Xm6ZiLUHcl5hQEe+tZMHqu3aPQgkBYcreE8a89A
 cW7a53THwJ9IOC1Elq2mFKaJbMrKmWgKTmckgLygL/b9nTDTNvpbnCFCyZLdHM86ZtFx
 UZEx2FLfRzmsrOoXnOD0BWAOrhrRPxQZR1q0N4dX3DTLwASZW0vE4exsECh2scA6rCD5
 ifUbkW2ABdTS78lwcz+xo1k0ShbOFpuubp09RNgyjnTL5eBvbYX8YYbCd7MRWRZO8uyU
 fwYFEkKX/9qn8mgkNqpMIQ9WGg9gMHXWzS9H0HVt6TI+PCCUk5HgmzRtI7hYOmhcRxWz
 GngA==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=1e100.net; s=20230601; t=1703541015; x=1704145815;
 h=cc:to:subject:message-id:date:mime-version:references:in-reply-to
 :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to;
 bh=ycJecH0/mIKGG9/EL17m1H/LPW9eoW3GK4s+JxRvzi8=;
 b=QvkjHL2Y+AVsmPymOvessAHGdMJbC2nwMRJE3sOy90OUlOIaiIQM70vrRBMu2nS/S8
 3wI0iN1ZEIIJSfRZh7cvPuiRVoUyI2hzqp4mTNA6nQ6DVZoEvgot5TD8DBCENswu4EOG
 cSD3QkT1Tn5G4txPxspRNx1MBbkahUIHv399GnMYqDpHeitrwZLftZJfd+e6AXzN3mJ4
 yXX9fufhkxB3QnHsPnIyeUFXloLgahPGOrxGx0SzD1WtxkHvwG6pSjaqB64hidXVJFls
 cCCosfuVKQt8I85FqoJVM86edDMC7An6qEZu6b39rKY6xQefbBksif6XpX8pFpPwEAhh
 d0KA==
X-Gm-Message-State: AOJu0YxWNiXRr2mZnFn2JFEGH1P+R014aC8IuVqlGEdgMyjXzWXahN4F
 NRo4CWIZSnVjhMKRj0K22ElBH+zJ1OWbLsVZTT8=
X-Google-Smtp-Source: AGHT+IHzEjJexOj52iI0w35ohfnFfKT2WOHjnKKz6U2wlJsekDeWBwB9s5TE4vrkCK8IPxIaLJ8EZZkfPduwo+9Vo9A=
X-Received: by 2002:a50:9999:0:b0:552:fcca:ee11 with SMTP id
 m25-20020a509999000000b00552fccaee11mr3640135edb.74.1703541014954; Mon, 25
 Dec 2023 13:50:14 -0800 (PST)
Received: from 753933720722 named unknown by gmailapi.google.com with
 HTTPREST; Mon, 25 Dec 2023 13:50:14 -0800
From: Stefan Kangas <stefankangas@HIDDEN>
In-Reply-To: <834jg6bhaf.fsf@HIDDEN>
References: <87a5q41azk.fsf@HIDDEN>
 <CADwFkmkJSXe4CWNJtwgjwJuJ4SihwPhgXpM1OnRFgJMZ=8FX1Q@HIDDEN>
 <87wmt2505y.fsf@HIDDEN> <834jg6bhaf.fsf@HIDDEN>
MIME-Version: 1.0
Date: Mon, 25 Dec 2023 13:50:14 -0800
Message-ID: <CADwFkmk4W1TAFpsGnKv_QB8pu9iO1x4Y4xT2H=apmo3enq=jhQ@HIDDEN>
Subject: Re: bug#67946: 30.0.50; [RFC PATCH] Add an option to not report
 "Wrote ..." when saving recentf.
To: Eli Zaretskii <eliz@HIDDEN>, 
 Vladimir Nikishkin <for_emacs-recentf-patch_2023-12-21@HIDDEN>
Content-Type: text/plain; charset="UTF-8"
X-Spam-Score: -0.0 (/)
X-Debbugs-Envelope-To: 67946
Cc: 67946 <at> debbugs.gnu.org
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -1.0 (-)

Eli Zaretskii <eliz@HIDDEN> writes:

> If we decide to turn it off, we will need a knob to revert to the old
> behavior, because after so many years of the existing behavior,
> someone somewhere might be used to it.

I think the existing patch basically has that already, but it seems to
be lacking a NEWS entry and a :version tag for the defcustom before it
can go in.

Vladimir, could you fix that and send a new patch?

Message received at 67946 <at> debbugs.gnu.org:

Received: (at 67946) by debbugs.gnu.org; 25 Dec 2023 12:25:46 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Mon Dec 25 07:25:46 2023
Received: from localhost ([127.0.0.1]:54101 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1rHk1i-0000pQ-91
	for submit <at> debbugs.gnu.org; Mon, 25 Dec 2023 07:25:46 -0500
Received: from eggs.gnu.org ([2001:470:142:3::10]:53690)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <eliz@HIDDEN>) id 1rHk1g-0000p6-9t
 for 67946 <at> debbugs.gnu.org; Mon, 25 Dec 2023 07:25:44 -0500
Received: from fencepost.gnu.org ([2001:470:142:3::e])
 by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)
 (Exim 4.90_1) (envelope-from <eliz@HIDDEN>)
 id 1rHk1U-0000Hx-BK; Mon, 25 Dec 2023 07:25:32 -0500
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org;
 s=fencepost-gnu-org; h=References:Subject:In-Reply-To:To:From:Date:
 mime-version; bh=zQ2vv1eLhhRRZWa0IPeSUyGJk1UHYjqR9r68i1Qa1uo=; b=NPcVX9CkphCm
 q/zoR8unhVKvVCkV+7f5lUzXU0MOkaQAy6NKPdaiTNU88EUHerJfLYUzc1okq9uWMUYshX0gcsTFy
 +pn+UZsCLApK1tEluRnf7rG+YAp91q9YXmive1nWs/p4BUh3lEJ1qG+1i8PCxvbNDOuhba7eHsSXX
 ocqwhWwcXX75/u9XZOiY2MdtiA7yzIJeZrhRWYn807zts+IzCWVNOn3+++QM6AezwAa00suB/1Jxw
 mC32APTD2TBScCZZHLmBkP1tQlZZCol8Y/arOD7H2clrAk47pBhbjekD63Sv+eh9gfStY2iLbMjPq
 bzaOehJMWexqQ7Dgq1XLmA==;
Date: Mon, 25 Dec 2023 14:25:28 +0200
Message-Id: <834jg6bhaf.fsf@HIDDEN>
From: Eli Zaretskii <eliz@HIDDEN>
To: Vladimir Nikishkin
	<for_emacs-recentf-patch_2023-12-21@HIDDEN>
In-Reply-To: <87wmt2505y.fsf@HIDDEN> (message from Vladimir
 Nikishkin on Mon, 25 Dec 2023 13:17:37 +0800)
Subject: Re: bug#67946: 30.0.50;
 [RFC PATCH] Add an option to not report "Wrote ..." when saving
 recentf.
References: <87a5q41azk.fsf@HIDDEN>
 <CADwFkmkJSXe4CWNJtwgjwJuJ4SihwPhgXpM1OnRFgJMZ=8FX1Q@HIDDEN>
 <87wmt2505y.fsf@HIDDEN>
X-Spam-Score: -2.3 (--)
X-Debbugs-Envelope-To: 67946
Cc: 67946 <at> debbugs.gnu.org, stefankangas@HIDDEN
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -3.3 (---)

> Cc: 67946 <at> debbugs.gnu.org
> From: Vladimir Nikishkin <for_emacs-recentf-patch_2023-12-21@HIDDEN>
> Date: Mon, 25 Dec 2023 13:17:37 +0800
> 
> 
> Stefan Kangas <stefankangas@HIDDEN> writes:
> 
> > Vladimir Nikishkin <for_emacs-recentf-patch_2023-12-21@HIDDEN>
> > writes:
> >
> >> I have the following setting in my init.el
> >>
> >> (add-hook 'buffer-list-update-hook 'recentf-track-opened-file)
> >> (add-hook 'buffer-list-update 'recentf-save-list)
> >>
> >> The first one is the one recommended by describe-function recentf-mode,
> >> and the second one is to make this switch persistent, and resistant to
> >> emacs being killed.
> >>
> >> However, I am getting a lot of messages like
> >> `Wrote /home/lockywolf/.emacs.d/recentf', which are not very helpful
> >> normally.
> >
> > When are these messages helpful?
> 
> Are you suggesting to turn them off unconditionally?

If we decide to turn it off, we will need a knob to revert to the old
behavior, because after so many years of the existing behavior,
someone somewhere might be used to it.

Message received at 67946 <at> debbugs.gnu.org:

Received: (at 67946) by debbugs.gnu.org; 25 Dec 2023 05:20:03 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Mon Dec 25 00:20:02 2023
Received: from localhost ([127.0.0.1]:53801 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1rHdNh-0000Pj-En
	for submit <at> debbugs.gnu.org; Mon, 25 Dec 2023 00:20:02 -0500
Received: from public.lockywolf.net ([2a03:6000:6f63:638::2]:47062)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <for_emacs-recentf-patch_2023-12-21@HIDDEN>)
 id 1rHdNc-0000PV-Tc
 for 67946 <at> debbugs.gnu.org; Mon, 25 Dec 2023 00:20:00 -0500
DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; s=2021-05-20; bh=EXOfB796D
 NJmrDfYbVjFe/kJUKjDfWQuNWcvqGxpPkw=;
 h=in-reply-to:date:subject:cc:to:
 from:references; d=lockywolf.net; b=X/ZvThBw1kBsmUYJHzVOPMTH40Oz841s+w
 PuaXvUliHAUMBppzTusNK9zV00wjLkyu3sHhwMya1DN+qagZ8Qk72vi0GfYeepZruiSsrL
 MnTidPg6baq7lh1Y8VwqFez73/1msDqqRgU6RNLoiwMuf36dyFCNGrxcA7wSLO0QiUM=
Received: from laptop.lockywolf.net (<unknown> [2001:470:24:315::102])
 by public.lockywolf.net (OpenSMTPD) with ESMTPSA id 67ac0256
 (TLSv1.3:TLS_AES_256_GCM_SHA384:256:NO); 
 Mon, 25 Dec 2023 06:19:45 +0100 (CET)
References: <87a5q41azk.fsf@HIDDEN>
 <CADwFkmkJSXe4CWNJtwgjwJuJ4SihwPhgXpM1OnRFgJMZ=8FX1Q@HIDDEN>
User-agent: mu4e 1.10.7; emacs 30.0.50
From: Vladimir Nikishkin <for_emacs-recentf-patch_2023-12-21@HIDDEN>
To: Stefan Kangas <stefankangas@HIDDEN>
Subject: Re: bug#67946: 30.0.50; [RFC PATCH] Add an option to not report
 "Wrote ..." when saving recentf.
Date: Mon, 25 Dec 2023 13:17:37 +0800
In-reply-to: <CADwFkmkJSXe4CWNJtwgjwJuJ4SihwPhgXpM1OnRFgJMZ=8FX1Q@HIDDEN>
Message-ID: <87wmt2505y.fsf@HIDDEN>
MIME-Version: 1.0
Content-Type: text/plain
X-Spam-Score: -0.0 (/)
X-Debbugs-Envelope-To: 67946
Cc: 67946 <at> debbugs.gnu.org
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -1.0 (-)


Stefan Kangas <stefankangas@HIDDEN> writes:

> Vladimir Nikishkin <for_emacs-recentf-patch_2023-12-21@HIDDEN>
> writes:
>
>> I have the following setting in my init.el
>>
>> (add-hook 'buffer-list-update-hook 'recentf-track-opened-file)
>> (add-hook 'buffer-list-update 'recentf-save-list)
>>
>> The first one is the one recommended by describe-function recentf-mode,
>> and the second one is to make this switch persistent, and resistant to
>> emacs being killed.
>>
>> However, I am getting a lot of messages like
>> `Wrote /home/lockywolf/.emacs.d/recentf', which are not very helpful
>> normally.
>
> When are these messages helpful?

Are you suggesting to turn them off unconditionally?

It would make sense, I guess. It might be useful when debugging "why is
my recentf file not updated", but if such an issue arises, I guess,
instrumenting "recentf-save-list" is more of a straightforward thing?

-- 
Your sincerely,
Vladimir Nikishkin (MiEr, lockywolf)
(Laptop)

Message received at 67946 <at> debbugs.gnu.org:

Received: (at 67946) by debbugs.gnu.org; 24 Dec 2023 14:39:41 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Sun Dec 24 09:39:41 2023
Received: from localhost ([127.0.0.1]:51848 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1rHPdl-0007eK-Bs
	for submit <at> debbugs.gnu.org; Sun, 24 Dec 2023 09:39:41 -0500
Received: from mail-ed1-x531.google.com ([2a00:1450:4864:20::531]:52564)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <stefankangas@HIDDEN>) id 1rHPdj-0007e4-7F
 for 67946 <at> debbugs.gnu.org; Sun, 24 Dec 2023 09:39:39 -0500
Received: by mail-ed1-x531.google.com with SMTP id
 4fb4d7f45d1cf-55333eb0312so2593158a12.1
 for <67946 <at> debbugs.gnu.org>; Sun, 24 Dec 2023 06:39:33 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=gmail.com; s=20230601; t=1703428767; x=1704033567; darn=debbugs.gnu.org;
 h=cc:to:subject:message-id:date:mime-version:references:in-reply-to
 :from:from:to:cc:subject:date:message-id:reply-to;
 bh=jknWbwHkoxG5NcLm+awgcF5bhJbaECD/Tie8JVBoaII=;
 b=d27n+Qpk1ql57uJUfwxAKUjA69wsNKZAT/Lfqk8UO2FgZ33nBxvfs6k18jQ+Ytbcy5
 SVasvDwQUfTAeHYYaXzLffXlodAF1gEax1kdsm2fIEXkZGWzt8holYS4Kj6FvBocMDHO
 /PvPiNrqSPw4AyN14hg8DA9cZbv5NQsvbxoz5ybOfr8BC9guRY8VH4aScFs1YoHmXY3v
 j/GhoP2Jma+Yn4rUg9zw7tOsbxmL9B3A4CU2SlFP7v7Q6v8UYh+DdO0QIYbZ5GizatMb
 hjRBin5IDkckO2XaM9Jpnr9LW7SNnXFt8nfXLTCyNZkrOuT8U8LNZc+AVsEl8RgeM+hd
 t70A==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=1e100.net; s=20230601; t=1703428767; x=1704033567;
 h=cc:to:subject:message-id:date:mime-version:references:in-reply-to
 :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to;
 bh=jknWbwHkoxG5NcLm+awgcF5bhJbaECD/Tie8JVBoaII=;
 b=cq29bw8wld55gXlbZSus+JYuzer/knVG5LwmYd2cHJRtqqR2+otTvr0m3tebC3dCSP
 pwp+r2f/MQlTjUrklWA4CwTyktU+N+wJbjSdLaliVxpm2AtPkpEMr03TCNl6vSV3B8Ir
 GT9z4El4EhyrPa9X+FYwY6jQ1R2sfj4dWTW4D/TKGpz/IHf/jS+2Q07pZon3ZuXMBBdp
 0PMdGYn4nfIDfgAOEt9Vv4ADL+9fL6LbtYaIpzneANYfBOltF+r4he2EpYw+zgzpIL3F
 Vt1BQKrI/H8urV1y48X0sf7Hy5ktaAfjeBVpDwwvGTwpd5TCzmwTPkcmCzwAo5Moisbw
 U68w==
X-Gm-Message-State: AOJu0YzCjQn1LGNud+JHQOvoMANp0Ti9cttsaiL5J2TadpYF+9VcKJb6
 ro3dMqBzdf2uduQEzqCBAYm9h93Se/1A/EMOZCY=
X-Google-Smtp-Source: AGHT+IHEVxqEmg5v6/Wfp9A7a6epOR2sdAzdLTCo2FW+Shdh3DYiQvYeOhv9Zd716j5BpU1OcZB4tuubr4s1sTkIud0=
X-Received: by 2002:a50:8d4e:0:b0:554:53d0:23ea with SMTP id
 t14-20020a508d4e000000b0055453d023eamr3248761edt.60.1703428767452; Sun, 24
 Dec 2023 06:39:27 -0800 (PST)
Received: from 753933720722 named unknown by gmailapi.google.com with
 HTTPREST; Sun, 24 Dec 2023 06:39:27 -0800
From: Stefan Kangas <stefankangas@HIDDEN>
In-Reply-To: <87a5q41azk.fsf@HIDDEN> (Vladimir Nikishkin's
 message of "Thu, 21 Dec 2023 11:26:52 +0800")
References: <87a5q41azk.fsf@HIDDEN>
MIME-Version: 1.0
Date: Sun, 24 Dec 2023 06:39:27 -0800
Message-ID: <CADwFkmkJSXe4CWNJtwgjwJuJ4SihwPhgXpM1OnRFgJMZ=8FX1Q@HIDDEN>
Subject: Re: bug#67946: 30.0.50; [RFC PATCH] Add an option to not report
 "Wrote ..." when saving recentf.
To: Vladimir Nikishkin <for_emacs-recentf-patch_2023-12-21@HIDDEN>
Content-Type: text/plain; charset="UTF-8"
X-Spam-Score: -0.0 (/)
X-Debbugs-Envelope-To: 67946
Cc: 67946 <at> debbugs.gnu.org
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -1.0 (-)

Vladimir Nikishkin <for_emacs-recentf-patch_2023-12-21@HIDDEN>
writes:

> I have the following setting in my init.el
>
> (add-hook 'buffer-list-update-hook 'recentf-track-opened-file)
> (add-hook 'buffer-list-update 'recentf-save-list)
>
> The first one is the one recommended by describe-function recentf-mode,
> and the second one is to make this switch persistent, and resistant to
> emacs being killed.
>
> However, I am getting a lot of messages like
> `Wrote /home/lockywolf/.emacs.d/recentf', which are not very helpful
> normally.

When are these messages helpful?

Message received at 67946 <at> debbugs.gnu.org:

Received: (at 67946) by debbugs.gnu.org; 21 Dec 2023 09:38:50 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Thu Dec 21 04:38:50 2023
Received: from localhost ([127.0.0.1]:42420 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1rGFVx-0005kz-TI
	for submit <at> debbugs.gnu.org; Thu, 21 Dec 2023 04:38:50 -0500
Received: from eggs.gnu.org ([2001:470:142:3::10]:38382)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <eliz@HIDDEN>) id 1rGFVu-0005kl-Sv
 for 67946 <at> debbugs.gnu.org; Thu, 21 Dec 2023 04:38:48 -0500
Received: from fencepost.gnu.org ([2001:470:142:3::e])
 by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)
 (Exim 4.90_1) (envelope-from <eliz@HIDDEN>)
 id 1rGFVl-0007iE-57; Thu, 21 Dec 2023 04:38:37 -0500
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org;
 s=fencepost-gnu-org; h=References:Subject:In-Reply-To:To:From:Date:
 mime-version; bh=kUcwvh5puz4gh4DB8yv6nKI5s/kjoqalzmpMdUCp9mo=; b=lOtdqPx3J9OF
 SffKHA9peTtlmbgrYgJFqOVoS5qtS1n/9R43vpomVnr/f5snH2svTJDjJ924G9VrlS6hnmTrnCVL8
 l5t1LBNu1LfXcZaZ+5kz7j/vRgM7uRITAeapP57+Y7iUrMOf8vI5F/5eskc34Qx/Mkzqjf1hyf8he
 9EWbkK6qC/71kVq6b/3rcROv03xS22Zt+xkat6HN4S/EcGcGnAhuHwuNUD1XBG2M0DnjFCdqyOitA
 tm+VF8WY58/u/YKJ5rmoGrtMK1cnKhYJjVPFFevsXTv3vBprHj/9dMiJj3uQuRrxDZHs17stPypMQ
 1airZ1hnWy5KJSExN0J87g==;
Date: Thu, 21 Dec 2023 11:37:54 +0200
Message-Id: <83zfy3gakt.fsf@HIDDEN>
From: Eli Zaretskii <eliz@HIDDEN>
To: Vladimir Nikishkin <for_emacs-recentf-patch_2023-12-21@HIDDEN>
In-Reply-To: <87a5q41azk.fsf@HIDDEN> (message from Vladimir
 Nikishkin on Thu, 21 Dec 2023 11:26:52 +0800)
Subject: Re: bug#67946: 30.0.50;
 [RFC PATCH] Add an option to not report "Wrote ..." when saving
 recentf.
References: <87a5q41azk.fsf@HIDDEN>
X-Spam-Score: -2.3 (--)
X-Debbugs-Envelope-To: 67946
Cc: 67946 <at> debbugs.gnu.org
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -3.3 (---)

> From: Vladimir Nikishkin <for_emacs-recentf-patch_2023-12-21@HIDDEN>
> Date: Thu, 21 Dec 2023 11:26:52 +0800
> 
> I have the following setting in my init.el
> 
> (add-hook 'buffer-list-update-hook 'recentf-track-opened-file)
> (add-hook 'buffer-list-update 'recentf-save-list)
> 
> The first one is the one recommended by describe-function recentf-mode,
> and the second one is to make this switch persistent, and resistant to
> emacs being killed.
> 
> However, I am getting a lot of messages like
> `Wrote /home/lockywolf/.emacs.d/recentf', which are not very helpful
> normally.
> 
> I am attaching a trivial patch (001_patch-recentf-verbose.patch), which
> adds a `recentf-verbose' defcustom, which, when set to nil, would
> suppress these messages.

Thanks, I have a few comments about the proposed changes:

> +(defcustom recentf-verbose t
> +  "Non-nil means to show debugging messages related to the recentf machinery."
> +  :group 'recentf
> +  :type 'boolean)

This currently only controls the "Wrote" message, so "show debugging
messages" is a bit misleading.

Also, new defcustoms should have a :version tag

> @@ -1324,7 +1329,10 @@ recentf-save-list
>                  (format ";; coding: %s\n" recentf-save-file-coding-system)
>                  ";; End:\n")
>          (write-region (point-min) (point-max)
> -                      (expand-file-name recentf-save-file))
> +                      (expand-file-name recentf-save-file)
> +                      nil
> +                      recentf-verbose
> +                      )

You pass the value of recentf-verbose, either nil or t, to
write-region, but the only way to suppress the "Wrote" message from
write-region is to pass a non-nil, non-t value in the 5th argument;
see the doc string of write-region.  So the above will not suppress
the "Wrote" message, neither if the new defcustom is t nor if it is
nil, and the code needs some simple changes to do what you want.  Am I
missing something?

Finally, this change needs a NEWS entry.  And please accompany the
change with a ChangeLog-style commit log message; see CONTRIBUTE for
how to format the log message, and see "git log" in the repository for
examples of how we do this in practice.

Thanks again for working on this.

Message received at submit <at> debbugs.gnu.org:

Received: (at submit) by debbugs.gnu.org; 21 Dec 2023 04:51:54 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Wed Dec 20 23:51:54 2023
Received: from localhost ([127.0.0.1]:42212 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
	id 1rGB2F-0007nQ-GK
	for submit <at> debbugs.gnu.org; Wed, 20 Dec 2023 23:51:54 -0500
Received: from lists.gnu.org ([2001:470:142::17]:56480)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <for_emacs-recentf-patch_2023-12-21@HIDDEN>)
 id 1rG9tT-0000nY-9p
 for submit <at> debbugs.gnu.org; Wed, 20 Dec 2023 22:38:46 -0500
Received: from eggs.gnu.org ([2001:470:142:3::10])
 by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)
 (Exim 4.90_1)
 (envelope-from <for_emacs-recentf-patch_2023-12-21@HIDDEN>)
 id 1rG9tI-0003Jc-Pw
 for bug-gnu-emacs@HIDDEN; Wed, 20 Dec 2023 22:38:32 -0500
Received: from public.lockywolf.net ([46.23.90.181])
 by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)
 (Exim 4.90_1)
 (envelope-from <for_emacs-recentf-patch_2023-12-21@HIDDEN>)
 id 1rG9tE-0004vK-Tx
 for bug-gnu-emacs@HIDDEN; Wed, 20 Dec 2023 22:38:31 -0500
DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; s=2021-05-20; bh=dtUnK51Qh
 rm6v4/LTBRztXz4bPOIemU9K9HUueF9eWg=; h=date:subject:to:from;
 d=lockywolf.net; b=XxrLM4SqOfKrUWHFHuQpprTzg8sIabYPiKL9IqNcSTuxhN7dhzA
 yYIEWwPfffbXLfV5wA2h2lllWKtgiplAobYz83zZHl8MIVI+a3+01YbrWcE19jQf5Eo/cR
 OX5LNlMts2bxXm5xy9NO8LgFQERyop78VBigxmMbVvSw6YWQYs=
Received: from laptop.lockywolf.net (<unknown> [2001:470:24:315::102])
 by public.lockywolf.net (OpenSMTPD) with ESMTPSA id a38a5f31
 (TLSv1.3:TLS_AES_256_GCM_SHA384:256:NO) for <bug-gnu-emacs@HIDDEN>;
 Thu, 21 Dec 2023 04:38:17 +0100 (CET)
User-agent: mu4e 1.10.7; emacs 30.0.50
From: Vladimir Nikishkin <for_emacs-recentf-patch_2023-12-21@HIDDEN>
To: bug-gnu-emacs@HIDDEN
Subject: 30.0.50; [RFC PATCH] Add an option to not report "Wrote ..." when
 saving recentf.
Date: Thu, 21 Dec 2023 11:26:52 +0800
Message-ID: <87a5q41azk.fsf@HIDDEN>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="=-=-="
Received-SPF: pass client-ip=46.23.90.181;
 envelope-from=for_emacs-recentf-patch_2023-12-21@HIDDEN;
 helo=public.lockywolf.net
X-Spam_score_int: -20
X-Spam_score: -2.1
X-Spam_bar: --
X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1,
 DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, SPF_HELO_PASS=-0.001,
 SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no
X-Spam_action: no action
X-Spam-Score: 0.9 (/)
X-Debbugs-Envelope-To: submit
X-Mailman-Approved-At: Wed, 20 Dec 2023 23:51:45 -0500
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -0.1 (/)

--=-=-=
Content-Type: text/plain

Dear Emacs Developers,

I have the following setting in my init.el

(add-hook 'buffer-list-update-hook 'recentf-track-opened-file)
(add-hook 'buffer-list-update 'recentf-save-list)

The first one is the one recommended by describe-function recentf-mode,
and the second one is to make this switch persistent, and resistant to
emacs being killed.

However, I am getting a lot of messages like
`Wrote /home/lockywolf/.emacs.d/recentf', which are not very helpful
normally.

I am attaching a trivial patch (001_patch-recentf-verbose.patch), which
adds a `recentf-verbose' defcustom, which, when set to nil, would
suppress these messages.

Please, consider including it into Emacs.


In GNU Emacs 30.0.50 (build 1, x86_64-slackware-linux-gnu, GTK+ Version
 3.24.31, cairo version 1.16.0) of 2023-09-22 built on
 laptop.lockywolf.net
Windowing system distributor 'The X.Org Foundation', version 11.0.12014000
System Description: Slackware 15.0 x86_64

Configured using:
 'configure --prefix=/usr --libdir=/usr/lib64 --sysconfdir=/etc
 --localstatedir=/var --program-prefix= --program-suffix=
 --mandir=/usr/man --infodir=/usr/info --without-gconf
 --without-gsettings --with-modules --with-json
 --with-native-compilation=aot --with-imagemagick --with-pdumper
 --with-xwidgets --enable-year2038 --with-tree-sitter --with-cairo-xcb
 --build=x86_64-slackware-linux 'CFLAGS=-O2 -fPIC -g''

Configured features:
ACL CAIRO DBUS FREETYPE GIF GLIB GMP GNUTLS GPM HARFBUZZ IMAGEMAGICK
JPEG JSON LCMS2 LIBXML2 MODULES NATIVE_COMP NOTIFY INOTIFY PDUMPER PNG
RSVG SECCOMP SOUND SQLITE3 THREADS TIFF TOOLKIT_SCROLL_BARS TREE_SITTER
WEBP X11 XDBE XIM XINPUT2 XPM XWIDGETS GTK3 ZLIB

Important settings:
  value of $LC_COLLATE: C
  value of $LC_TIME: en_GB.UTF-8
  value of $LANG: en_US.UTF-8
  value of $XMODIFIERS: @im=ibus
  locale-coding-system: utf-8-unix

Major mode: Dired by name

Minor modes in effect:
  memento-mori-mode: t
  highlight-doxygen-global-mode: t
  hes-mode: t
  global-highlight-parentheses-mode: t
  highlight-parentheses-mode: t
  global-color-identifiers-mode: t
  frecentf-mode: t
  idle-org-agenda-mode: t
  gnus-dired-mode: t
  mu4e-modeline-mode: t
  TeX-PDF-mode: t
  TeX-source-correlate-mode: t
  ido-everywhere: t
  async-bytecomp-package-mode: t
  global-git-commit-mode: t
  magit-auto-revert-mode: t
  server-mode: t
  global-flycheck-mode: t
  global-hl-todo-mode: t
  gdb-many-windows: t
  global-company-mode: t
  company-mode: t
  auto-insert-mode: t
  pdf-occur-dired-minor-mode: t
  pdf-occur-global-minor-mode: t
  shackle-mode: t
  dired-omit-mode: t
  global-auto-revert-mode: t
  electric-pair-mode: t
  which-function-mode: t
  global-hl-line-mode: t
  global-display-fill-column-indicator-mode: t
  display-fill-column-indicator-mode: t
  global-hi-lock-mode: t
  hi-lock-mode: t
  windmove-mode: t
  savehist-mode: t
  recentf-mode: t
  save-place-mode: t
  super-save-mode: t
  xterm-mouse-mode: t
  override-global-mode: t
  tooltip-mode: t
  global-eldoc-mode: t
  show-paren-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  global-prettify-symbols-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  minibuffer-regexp-mode: t
  buffer-read-only: t
  size-indication-mode: t
  column-number-mode: t
  line-number-mode: t
  transient-mark-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  auto-save-visited-mode: t
  abbrev-mode: t

Load-path shadows:
/home/lockywolf/.emacs.d/elpa/company-cmake-0.1/company-cmake hides /home/lockywolf/.emacs.d/elpa/company-20230303.2331/company-cmake
/home/lockywolf/OfficialRepos/org-noter/org-noter hides /home/lockywolf/.emacs.d/elpa/org-noter-20230728.2037/org-noter
/home/lockywolf/OfficialRepos/org-noter/org-noter-core hides /home/lockywolf/.emacs.d/elpa/org-noter-20230728.2037/org-noter-core
/home/lockywolf/.emacs.d/elpa/cmake-mode-20230323.1312/cmake-mode hides /usr/share/emacs/site-lisp/cmake-mode
/home/lockywolf/.emacs.d/elpa/transient-20231112.923/transient hides /usr/share/emacs/30.0.50/lisp/transient
/home/lockywolf/.emacs.d/elpa/bind-key-20230203.2004/bind-key hides /usr/share/emacs/30.0.50/lisp/use-package/bind-key
/home/lockywolf/.emacs.d/elpa/flim-20230205.1423/sasl hides /usr/share/emacs/30.0.50/lisp/net/sasl

Features:
(shadow mail-extr face-remap emacsbug time-stamp shortdoc bug-reference
c++-ts-mode c-ts-mode c-ts-common cl-print ffap misearch multi-isearch
vc-git matlab matlab-scan matlab-syntax matlab-compat pulse dired-aux
company-oddmuse company-keywords make-mode company-etags company-gtags
company-dabbrev-code company-dabbrev company-files company-clang
company-capf company-cmake company-semantic company-template
company-bbdb init rc-local lockywolf-postamble-laptop
display-line-numbers lockywolf-exwm exwm exwm-input xcb-keysyms
exwm-manage exwm-floating xcb-cursor xcb-render exwm-layout
exwm-workspace exwm-core xcb-ewmh xcb-icccm xcb xcb-xkb xcb-xproto
xcb-types xcb-debug lockywolf-erc erc-track erc-match erc erc-backend
erc-networks erc-common erc-compat erc-loaddefs lockywolf-equake equake
lockywolf-gui-x11 hamburger-menu lockywolf-gui-general diary-lib
diary-loaddefs memento-mori highlight-doxygen highlight-escape-sequences
highlight-parentheses rainbow-mode color-identifiers-mode prettify-greek
form-feed tsdh-dark-theme lockywolf-calc calc calc-loaddefs calc-macs
lockywolf-monitoring frecentf persist frecency a dirtrack
idle-org-agenda lockywolf-autocomplete-ido avy
lockywolf-org-scientific-large org-download org-attach org-mind-map
org-sticky-header org-pdftools pdf-annot facemenu org-noter
org-noter-djvu org-noter-nov org-noter-pdf org-noter-core org2blog ox-wp
writegood-mode metaweblog xml-rpc url-http url-auth url-gw nsm hydra lv
htmlize org-journal org-crypt cal-iso org-sidebar org-ql-search
org-ql-view ov org-super-agenda ht org-ql peg ts org-habit org-duration
org-mru-clock ox-beamer ox-org ox-md ox-extra ox-odt rng-loc rng-uri
rng-parse rng-match rng-dt rng-util rng-pttrn nxml-parse nxml-ns
nxml-enc xmltok nxml-util ox-latex ox-icalendar org-agenda ox-html table
ox-ascii ox-publish ox org-clock ob-maxima ob-fortran ob-shell ob-latex
ob-scheme geiser-impl help-fns radix-tree geiser-custom geiser-base
geiser ob-C ob-plantuml org-tempo tempo org-protocol ox-youtube org-ebib
org-bibtex-extras ol-bibtex ox-bibtex org-inlinetask
org-fancy-priorities lockywolf-cryptography pinentry lockywolf-elfeed
lockywolf-email-pim bbdb bbdb-site gnus-dired mu4e-speedbar speedbar
ezimage dframe mu4e mu4e-org mu4e-notification notifications mu4e-main
mu4e-view gnus-art mm-uu mml2015 mm-view mml-smime smime gnutls dig
gnus-sum gnus-group gnus-undo gnus-start gnus-dbus gnus-cloud nnimap
nnmail mail-source utf7 nnoo gnus-spec gnus-int gnus-range gnus-win
mu4e-headers mu4e-compose mu4e-draft mu4e-actions mu4e-search mu4e-lists
mu4e-bookmarks mu4e-mark mu4e-message flow-fill mu4e-contacts
mu4e-update mu4e-folders mu4e-context mu4e-query-items mu4e-server
mu4e-modeline mu4e-vars mu4e-helpers mu4e-config mu4e-window
mu4e-obsolete message-view-patch auth-source-xoauth2 auth-source-pass
smtpmail-multi smtpmail khardel yaml-mode vdirel org-vcard lockywolf-tex
company-auctex vdm-snippets yasnippet bib-cite auctex-latexmk latex
latex-flymake tex-ispell tex-style tex texmathp cdlatex lockywolf-bibtex
ivy-bibtex bibtex-completion org-capture biblio biblio-download
biblio-dissemin biblio-ieee biblio-hal biblio-dblp biblio-crossref
biblio-arxiv timezone biblio-doi biblio-core ido f f-shortdoc s ebib
helm helm-global-bindings helm-easymenu helm-core async-bytecomp
helm-source helm-multi-match helm-lib async ivy delsel ivy-faces
ivy-overlay colir ebib-reading-list ebib-notes org-element org-persist
org-id org-refile org ob ob-tangle ob-ref ob-lob ob-table ob-exp
org-macro org-src ob-comint org-pcomplete org-list org-footnote
org-faces org-entities ob-emacs-lisp ob-core ob-eval org-cycle org-table
org-keys oc org-loaddefs cal-menu calendar cal-loaddefs ol org-fold
org-fold-core org-compat avl-tree org-version org-macs ebib-filters
ebib-keywords ebib-utils ebib-db parsebib mule-util bibretrieve
bibtex-utils reftex reftex-loaddefs reftex-vars bibtex
lockywolf-plantuml flycheck-plantuml plantuml-mode lockywolf-matlab
lockywolf-scheme scheme-complete scheme lockywolf-emacs-lisp
paredit-menu paredit lockywolf-helm lockywolf-vala vala-mode cc-langs
lockywolf-cpp clang-format+ clang-format eldoc-cmake cmake-mode rst
company-c-headers flycheck-clangcheck google-c-style cc-mode cc-fonts
cc-guess cc-menus cc-cmds lockywolf-fortran f90 lockywolf-bash sh-script
smie treesit executable lockywolf-version-control magit-bookmark
magit-submodule magit-blame magit-stash magit-reflog magit-bisect
magit-push magit-pull magit-fetch magit-clone magit-remote magit-commit
magit-sequence magit-notes magit-worktree magit-tag magit-merge
magit-branch magit-reset magit-files magit-refs magit-status magit
magit-repos magit-apply magit-wip magit-log magit-diff smerge-mode
git-commit log-edit message sendmail yank-media rfc822 mml mml-sec epa
epg epg-config mm-decode mm-bodies mm-encode mail-parse rfc2231
mailabbrev gmm-utils mailheader pcvs-util add-log magit-core
magit-autorevert magit-margin magit-transient magit-process with-editor
server magit-mode transient magit-git magit-base crm magit-section vc
vc-dispatcher lockywolf-spellchecking-hard languagetool
languagetool-server languagetool-console languagetool-java
languagetool-issue languagetool-correction languagetool-core flycheck
lockywolf-spellchecking-simple hl-todo compat flyspell ispell
lockywolf-debugging gdb-mi bindat gud lockywolf-advanced-editing eglot
external-completion jsonrpc flymake-proc flymake diff diff-mode ert
debug backtrace markdown-mode color company autoinsert ediff ediff-merg
ediff-mult ediff-wind ediff-diff ediff-help ediff-init ediff-util
lockywolf-ggtags ggtags hippie-exp etags fileloop xref project ewoc
lockywolf-tramp lockywolf-shells esh-help dash em-unix em-smart esh-mode
esh-var eshell esh-cmd generator esh-ext esh-opt esh-proc esh-io esh-arg
esh-module esh-groups esh-util lockywolf-paper-pdf-djvu djvu pdf-occur
ibuf-ext ibuffer ibuffer-loaddefs tablist advice tablist-filter
semantic/wisent/comp semantic/wisent semantic/wisent/wisent
semantic/util-modes semantic/util semantic semantic/tag semantic/lex
semantic/fw mode-local find-func cedet pdf-isearch let-alist pdf-misc
pdf-tools pdf-view derived pdf-cache pdf-info tq pdf-util pdf-macs
doc-view jka-compr image-mode exif ps-print ps-print-loaddefs lpr
lockywolf-downloaded-packages diminish shackle trace mic-paren cl
lockywolf-dired dired-x lockywolf-treesitter lockywolf-standard-packages
artist picture reporter rect eww url-queue thingatpt shr pixel-fill
kinsoku url-file svg dom puny mm-url gnus nnheader gnus-util range woman
man info-colors secrets dbus comp comp-cstr warnings xml locate dired
dired-loaddefs arc-mode archive-mode compile autorevert filenotify
elec-pair which-func imenu hl-line display-fill-column-indicator hi-lock
foldout edmacro kmacro noutline outline windmove bookmark
text-property-search savehist recentf tree-widget saveplace tramp-sh
tramp trampver tramp-integration files-x tramp-message tramp-compat xdg
shell pcomplete comint ansi-osc ring parse-time iso8601 time-date
format-spec ansi-color tramp-loaddefs super-save rmail rfc6068 rfc2047
rfc2045 ietf-drums mm-util mail-prsvr mail-utils image-file
image-converter cl-extra help-mode
lockywolf-handy-own-functions-keybindings lockywolf-basic-init xt-mouse
use-package-ensure-system-package system-packages cus-edit pp cus-load
icons wid-edit use-package use-package-ensure use-package-delight
use-package-diminish use-package-bind-key bind-key easy-mmode
use-package-core init-custom finder-inf 0blayout-autoloads
auctex-latexmk-autoloads auth-source-xoauth2-autoloads
bind-key-autoloads cdlatex-autoloads centered-cursor-mode-autoloads
chatgpt-shell-autoloads cmake-font-lock-autoloads cmake-mode-autoloads
color-identifiers-mode-autoloads anaconda-mode-autoloads
auctex-autoloads tex-site company-emoji-autoloads company-math-autoloads
ac-php-core-autoloads company-quickhelp-autoloads
company-shell-autoloads company-autoloads consult-eglot-autoloads
consult-autoloads csv-mode-autoloads debpaste-autoloads
diminish-autoloads djvu-autoloads ebib-autoloads elisp-depmap-autoloads
emms-info-mediainfo-autoloads emms-autoloads ess-autoloads
exwm-x-autoloads exwm-autoloads flycheck-ocaml-autoloads
flycheck-phpstan-autoloads flymake-easy-autoloads frecentf-autoloads
frecency-autoloads a-autoloads geiser-chez-autoloads
geiser-chibi-autoloads geiser-mit-autoloads geiser-racket-autoloads
geiser-stklos-autoloads geiser-autoloads flycheck-autoloads
gitlab-pipeline-autoloads ghub-autoloads go-mode-autoloads
graphviz-dot-mode-autoloads haskell-mode-autoloads
highlight-doxygen-autoloads highlight-escape-sequences-autoloads
highlight-indent-guides-autoloads highlight-parentheses-autoloads
hy-mode-autoloads i3wm-autoloads i3wm-config-mode-autoloads
imenu-list-autoloads info-colors-autoloads ivy-bibtex-autoloads
ivy-autoloads jabber-autoloads fsm-autoloads jupyter-autoloads
khardel-autoloads kotlin-mode-autoloads languagetool-autoloads
lua-mode-autoloads magit-todos-autoloads hl-todo-autoloads
markdown-preview-mode-autoloads markdown-mode-autoloads
math-symbol-lists-autoloads matlab-mode-autoloads merlin-autoloads
mermaid-mode-autoloads mermaid-ts-mode-autoloads
message-view-patch-autoloads magit-autoloads pcase git-commit-autoloads
mic-paren-autoloads mu4e-conversation-autoloads
mu4e-jump-to-list-autoloads nginx-mode-autoloads ob-hy-autoloads
ob-mermaid-autoloads ocaml-ts-mode-autoloads org-bib-mode-autoloads
org-contrib-autoloads org-download-autoloads org-imenu-autoloads
org-journal-autoloads org-mru-clock-autoloads org-pdftools-autoloads
org-noter-autoloads org-ref-autoloads citeproc-autoloads
bibtex-completion-autoloads biblio-autoloads biblio-core-autoloads
org-roam-autoloads magit-section-autoloads emacsql-autoloads
org-sidebar-autoloads org-ql-autoloads org-sticky-header-autoloads
org-super-agenda-autoloads org-xlatex-autoloads org2blog-autoloads
metaweblog-autoloads ov-autoloads ox-pandoc-autoloads paredit-autoloads
parsebib-autoloads pcre2el-autoloads pcsv-autoloads
pdf-drop-mode-autoloads pdf-view-pagemark-autoloads pdf-tools-autoloads
peg-autoloads persist-autoloads phpstan-autoloads php-mode-autoloads
pomm-autoloads alert-autoloads log4e-autoloads gntp-autoloads
pos-tip-autoloads pythonic-autoloads f-autoloads qml-mode-autoloads
qt-pro-mode-autoloads queue-autoloads rainbow-mode-autoloads
semi-autoloads flim-autoloads apel-autoloads shell-maker-autoloads
sml-mode-autoloads smog-autoloads spell-fu-autoloads
sr-speedbar-autoloads srv-autoloads string-inflection-autoloads
sudo-edit-autoloads tablist-autoloads telega-autoloads
rainbow-identifiers-autoloads transient-autoloads treemacs-autoloads
cfrs-autoloads posframe-autoloads ht-autoloads hydra-autoloads
pfuture-autoloads ace-window-autoloads avy-autoloads treepy-autoloads
ts-autoloads dash-autoloads tuareg-autoloads rx caml-autoloads cc-styles
cc-align cc-engine cc-vars cc-defs vdirel-autoloads helm-autoloads
popup-autoloads helm-core-autoloads async-autoloads org-vcard-autoloads
visual-fill-column-autoloads websocket-autoloads with-editor-autoloads
info compat-autoloads writegood-mode-autoloads xml-rpc-autoloads
yaml-mode-autoloads s-autoloads zmq-autoloads package browse-url url
url-proxy url-privacy url-expand url-methods url-history url-cookie
generate-lisp-file url-domsuf url-util mailcap url-handlers url-parse
auth-source cl-seq eieio eieio-core cl-macs password-cache json subr-x
map byte-opt gv bytecomp byte-compile url-vars cl-loaddefs cl-lib rmc
iso-transl tooltip cconv eldoc paren electric uniquify ediff-hook
vc-hooks lisp-float-type elisp-mode mwheel term/x-win x-win
term/common-win x-dnd touch-screen tool-bar dnd fontset image regexp-opt
fringe tabulated-list replace newcomment text-mode lisp-mode prog-mode
register page tab-bar menu-bar rfn-eshadow isearch easymenu timer select
scroll-bar mouse jit-lock font-lock syntax font-core term/tty-colors
frame minibuffer nadvice seq simple cl-generic indonesian philippine
cham georgian utf-8-lang misc-lang vietnamese tibetan thai tai-viet lao
korean japanese eucjp-ms cp51932 hebrew greek romanian slovak czech
european ethiopic indian cyrillic chinese composite emoji-zwj charscript
charprop case-table epa-hook jka-cmpr-hook help abbrev obarray oclosure
cl-preloaded button loaddefs theme-loaddefs faces cus-face macroexp
files window text-properties overlay sha1 md5 base64 format env
code-pages mule custom widget keymap hashtable-print-readable backquote
threads xwidget-internal dbusbind inotify lcms2 dynamic-setting
font-render-setting cairo gtk x-toolkit xinput2 x multi-tty move-toolbar
make-network-process native-compile emacs)

Memory information:
((conses 16 1523764 420437) (symbols 48 99499 0) (strings 32 382852 18970)
 (string-bytes 1 12879328) (vectors 16 147918) (vector-slots 8 3842385 201399)
 (floats 8 1392 9690) (intervals 56 11091 4667) (buffers 992 41))

-- 
Your sincerely,
Vladimir Nikishkin (MiEr, lockywolf)
(Laptop)

--=-=-=
Content-Type: text/x-patch
Content-Disposition: attachment; filename=001_patch-recentf-verbose.patch

diff --git a/lisp/recentf.el b/lisp/recentf.el
index 94ae871763b..b9272cb672e 100644
--- a/lisp/recentf.el
+++ b/lisp/recentf.el
@@ -319,6 +319,11 @@ recentf-show-file-shortcuts-flag
 used as shortcuts to open the Nth file."
   :group 'recentf
   :type 'boolean)
+
+(defcustom recentf-verbose t
+  "Non-nil means to show debugging messages related to the recentf machinery."
+  :group 'recentf
+  :type 'boolean)
 
 ;;; Utilities
 ;;
@@ -1324,7 +1329,10 @@ recentf-save-list
                 (format ";; coding: %s\n" recentf-save-file-coding-system)
                 ";; End:\n")
         (write-region (point-min) (point-max)
-                      (expand-file-name recentf-save-file))
+                      (expand-file-name recentf-save-file)
+                      nil
+                      recentf-verbose
+                      )
         (when recentf-save-file-modes
           (set-file-modes recentf-save-file recentf-save-file-modes))
         nil)

--=-=-=--