From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Karl Fogel Newsgroups: gmane.emacs.bugs Subject: bug#12507: [debbugs-tracker] Processed: severity 12507 wishlist Date: Tue, 29 Sep 2020 00:25:02 -0500 Message-ID: <875z7x889d.fsf@red-bean.com> References: <87bogubqjy.fsf@gnu.org> <873925ebpd.fsf@gnu.org> <87363ff7qp.fsf@gnus.org> <11ee80e5-6d98-4a1f-93c8-babb6da003f3@default> <87r1qxrgs1.fsf@gnus.org> <87v9g52dvz.fsf@red-bean.com> Reply-To: Karl Fogel Mime-Version: 1.0 Content-Type: text/plain Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="36985"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux) Cc: Lars Ingebrigtsen , Chong Yidong , 12507@debbugs.gnu.org To: Drew Adams Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Tue Sep 29 07:26:10 2020 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1kN89R-0009UZ-Pp for geb-bug-gnu-emacs@m.gmane-mx.org; Tue, 29 Sep 2020 07:26:10 +0200 Original-Received: from localhost ([::1]:58634 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1kN89Q-000364-Rx for geb-bug-gnu-emacs@m.gmane-mx.org; Tue, 29 Sep 2020 01:26:08 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:55976) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1kN89K-00035q-OT for bug-gnu-emacs@gnu.org; Tue, 29 Sep 2020 01:26:02 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]:43057) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1kN89K-0001VC-Dm for bug-gnu-emacs@gnu.org; Tue, 29 Sep 2020 01:26:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1kN89K-000899-B8 for bug-gnu-emacs@gnu.org; Tue, 29 Sep 2020 01:26:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Karl Fogel Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Tue, 29 Sep 2020 05:26:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 12507 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: notabug Original-Received: via spool by 12507-submit@debbugs.gnu.org id=B12507.160135710731223 (code B ref 12507); Tue, 29 Sep 2020 05:26:02 +0000 Original-Received: (at 12507) by debbugs.gnu.org; 29 Sep 2020 05:25:07 +0000 Original-Received: from localhost ([127.0.0.1]:54598 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1kN88R-00087X-02 for submit@debbugs.gnu.org; Tue, 29 Sep 2020 01:25:07 -0400 Original-Received: from newsp.red-bean.com ([45.79.25.59]:38244 helo=sanpietro.red-bean.com) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1kN88O-00087N-M1 for 12507@debbugs.gnu.org; Tue, 29 Sep 2020 01:25:06 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=red-bean.com; s=202005newsp; h=Content-Type:MIME-Version:Message-ID: In-Reply-To:Date:Reply-To:References:Subject:Cc:To:From:Sender: Content-Transfer-Encoding:Content-ID:Content-Description; bh=Qn5oB0wZo91efAB3OwnFA/V1Jegh9qPwjXL+Y3jYU+U=; t=1601357104; x=1602566704; b=HVSCBb8+Ax/tb9/p4nmfTt/fXohaYgSXg1XP8VrpiHMG0b90aWrOousnTKOyq13cGmEpul4poF M/55vLRPAbOEM/zYdLpBXLKDCZ1MDI7+kYT/Cq37xpeaahQxDanwTZHYAn07FGsFe6+UhhbcIJI4c /1lDdb9GgkevffpXXGDOOYaZVeZpVlhrHD0YLlU2ZNSobHisLnRRFr9OLrbOyaPSqyO3Zn+ZBe6ua n6eH7t6N/oD4hZzMay4P+KZZ41P3EHusgeSpQKZJzON5z326UZB/ZsQ3QxrX8vIfNxgxgUy9kwTfY 54YIEmw/Pe1jnKs0askHpkfEcL57k+7nunOLg==; Original-Received: from 99-112-125-163.lightspeed.cicril.sbcglobal.net ([99.112.125.163]:54994 helo=floss) by sanpietro.red-bean.com with esmtpsa (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1kN88N-0006bj-LA; Tue, 29 Sep 2020 05:25:03 +0000 In-Reply-To: (Drew Adams's message of "Wed, 23 Sep 2020 11:14:16 -0700 (PDT)") X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.io gmane.emacs.bugs:189235 Archived-At: On 23 Sep 2020, Drew Adams wrote: >At this point I don't have anything special to say >about your general proposal regarding backups. My >comments regard the bookmarks code. > >1. I don't see why this proposal is in bug #12507. > It seems like a general proposal, which should > probably go to emacs-devel. When the first motivation for a change that has potentially general use is for fixing a specific bug, I normally initiate the discussion of that change in the ticket for the bug, since there isn't anything else directly motivating the change (even if we suspect it has more general use beyond just being part of the bugfix). The more general change may be worth mentioning on emacs-devel once the bug-specific thread has consensed, of course; it's just that I wouldn't start out on emacs-devel, because the discussion may go elsewhere and the change might not happen after all. >2. I don't see why the bookmark code shouldn't visit > the file. That hasn't been explained. You cite > a commit for save-place, not for bookmark.el. > And even for that commit there's no explanation > of _why_ the particular file shouldn't be visited. > >I switched to using `write-file' for bookmark files >long ago, and I haven't seen any problem. When using >a bookmark file, you're anyway visiting it, in some >sense. I see no inconvenience in having the bookmark >file be added to a list of visited files. You might >not be reading or editing it directly (as text), but >you're definitely reading it or editing it indirectly. > >Until we hear a reason for why `write-region' should >be used for bookmark files, using it seems like a >solution (for save-place?) looking for a problem. > >What's wrong with the obvious, simple solution to the >real, recognized problem of no backups for bookmark >files: use `write-file'? That's the first question >to ask and answer, no? I can't think of any reason now. I have a vague memory that there *was* a specific reason, but if there was, I failed to document adequately at the time and have been unable to recover it now. Eli helpfully linked to Richard's message (https://lists.gnu.org/archive/html/emacs-devel/2005-05/msg01376.html), in which Richard points out that if bookmark visits the file anyway when reading, then there's nothing to be gained (from the perspective of saveplace.el interference) from avoiding visiting it when writing. And `bookmark-load' *does* visit the file when reading, so I'm leaning toward just taking your suggestion and reverting to `write-file' -- but this time leaving better historical breadcrumbs (in comments and/or log messages) in case my vaguely-remembered bug turns out to be real and reappears. I will do this as soon as I have time to focus on it and make sure I don't make a silly blunder; that may be several days. Best regards, -Karl