From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.bugs Subject: bug#63627: Improve plstore.el and fix various issues of it Date: Fri, 08 Sep 2023 08:40:18 +0300 Message-ID: <83sf7pxlx9.fsf@gnu.org> References: <2063d5d2-ae9e-020a-3c19-54508ddbabab@vodafonemail.de> <37949bc5-c572-ff98-ebe6-9217ab7067cc@vodafonemail.de> <83cz2gfypg.fsf@gnu.org> <0101e7f1-970b-297c-836f-e9c0aa5e38a0@vodafonemail.de> <83cyz3g6oc.fsf@gnu.org> <8b00a564-c0d2-3072-50bc-530c50fa25b2@vodafonemail.de> <83pm319g3i.fsf@gnu.org> <24a502cc-9151-31bf-e0c6-078a400c6761@vodafonemail.de> <83r0naz6s1.fsf@gnu.org> <82b540c7-403b-3904-a16d-bda3c084f78a@vodafonemail.de> Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="16847"; mail-complaints-to="usenet@ciao.gmane.io" Cc: 63627@debbugs.gnu.org To: Jens Schmidt Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Fri Sep 08 07:41:43 2023 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 1qeUFS-0004ED-Ua for geb-bug-gnu-emacs@m.gmane-mx.org; Fri, 08 Sep 2023 07:41:42 +0200 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1qeUF8-0002er-AR; Fri, 08 Sep 2023 01:41:22 -0400 Original-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 ) id 1qeUEp-0002ck-Dm for bug-gnu-emacs@gnu.org; Fri, 08 Sep 2023 01:41:07 -0400 Original-Received: from debbugs.gnu.org ([2001:470:142:5::43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1qeUEm-0006E0-Gi for bug-gnu-emacs@gnu.org; Fri, 08 Sep 2023 01:41:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1qeUEo-0001bQ-5l for bug-gnu-emacs@gnu.org; Fri, 08 Sep 2023 01:41:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Eli Zaretskii Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Fri, 08 Sep 2023 05:41:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 63627 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 63627-submit@debbugs.gnu.org id=B63627.16941516416083 (code B ref 63627); Fri, 08 Sep 2023 05:41:02 +0000 Original-Received: (at 63627) by debbugs.gnu.org; 8 Sep 2023 05:40:41 +0000 Original-Received: from localhost ([127.0.0.1]:41951 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1qeUES-0001a3-IW for submit@debbugs.gnu.org; Fri, 08 Sep 2023 01:40:40 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:38344) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1qeUEN-0001ZX-Q9 for 63627@debbugs.gnu.org; Fri, 08 Sep 2023 01:40:39 -0400 Original-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 ) id 1qeUEG-00066H-69; Fri, 08 Sep 2023 01:40:28 -0400 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=kEZ1gYQROnTlFaBynNQ0elOGfspdiZa+KECWyCYCIfw=; b=Mg/uHKBe/ZrD KoqdI+nV/J2TxGnoQZsbhhuVVI8ODCwt55Bh7z83F64bO6EZvEp57B8PC1/2eJl2WAgCFmHoMDHih hEGdx9OfWB3SwvNM6ju8y07zbIOaosqHo+KOCBoTPUElMVOCSKfxZ/RdW0XDSCwfRbNrx6/wFyiI8 mC/nd3n/v1jBfxXDcwbhZV/ab8rKIYvDX1lMKkrwVF8Jj4uZ+YhWfz+jGFdcIfWG1DBdu3Evjr+ZV 2uNeyuJuYpK3+lXEuq7XkLNwyCtYp1mEY4erb7pGknlK5KWOuhOATAp+eBGWjPWLEBugeVvhM7Kl/ X956SugL5TRPe//93OGzfQ==; In-Reply-To: <82b540c7-403b-3904-a16d-bda3c084f78a@vodafonemail.de> (message from Jens Schmidt on Thu, 7 Sep 2023 21:27:13 +0200) 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-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.bugs:269721 Archived-At: > Date: Thu, 7 Sep 2023 21:27:13 +0200 > From: Jens Schmidt > Cc: 63627@debbugs.gnu.org > > > This seems to imply that you are still working on this changeset? > > I plan for plstore.el: > > 1. Two bug fixes on emacs-29 that I would like to add tests for on > master once the bug fixes have made their way to master. > > 2. More tests and new features on emacs-master. > > All of the above will still take some time to implement due to time > constraints. > > For all of the above it would be convenient to have the infrastructure > of the plstore tests committed on master or at least agreed upon so > that I can build on that. > > Not sure how you prefer to get such long-running projects presented - > as a series of minor (but self-contained!) patches over time or at > once as one big set of patches. It is preferable to install separate patches only if each one of them has merit on its own. E.g., imagine that some "force majeure" will prevent you from working on further changes, and ask yourself whether what you already have would make an improvement if installed. Changes that only make sense together are better submitted and installed as a single patch. But if you personally prefer to submit a series of patches even when they should be installed together, that's fine, it will just make them slightly harder to review, especially if they touch the same code.