* Re: master d20acfe: Fix Gnus registry pruning and sorting, and rename file [not found] ` <E1Y1ZAB-0003ek-Vw@vcs.savannah.gnu.org> @ 2014-12-18 14:33 ` Dmitry Gutov 2014-12-18 15:26 ` Eric Abrahamsen 2014-12-18 23:11 ` Katsumi Yamaoka 2014-12-19 7:39 ` [Emacs-diffs] " Glenn Morris 1 sibling, 2 replies; 17+ messages in thread From: Dmitry Gutov @ 2014-12-18 14:33 UTC (permalink / raw) To: emacs-devel; +Cc: Katsumi Yamaoka Hi! > branch: master > commit d20acfe06340953dce443795d28ac224a2223414 > Author: Eric Abrahamsen <eric@ericabrahamsen.net> > Commit: Katsumi Yamaoka <yamaoka@jpl.org> This commit breaks compilation of Emacs master: Compiling gnus/gnus-registry.el Eager macro-expansion failure: (invalid-slot-type max-size integer most-positive-fixnum) Eager macro-expansion failure: (invalid-slot-type max-size integer most-positive-fixnum) In toplevel form: gnus/gnus-registry.el:88:1:Error: Invalid slot type: max-size, integer, most-positive-fixnum make[3]: *** [gnus/gnus-registry.elc] Error 1 make[3]: Leaving directory `/home/gutov/vc/emacs/lisp' make[2]: *** [compile-main] Error 2 make[2]: Leaving directory `/home/gutov/vc/emacs/lisp' make[1]: *** [lisp] Error 2 make[1]: Leaving directory `/home/gutov/vc/emacs' make: *** [bootstrap] Error 2 ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: master d20acfe: Fix Gnus registry pruning and sorting, and rename file 2014-12-18 14:33 ` master d20acfe: Fix Gnus registry pruning and sorting, and rename file Dmitry Gutov @ 2014-12-18 15:26 ` Eric Abrahamsen 2014-12-18 15:35 ` Andreas Schwab 2014-12-18 23:11 ` Katsumi Yamaoka 1 sibling, 1 reply; 17+ messages in thread From: Eric Abrahamsen @ 2014-12-18 15:26 UTC (permalink / raw) To: emacs-devel [-- Attachment #1: Type: text/plain, Size: 556 bytes --] Dmitry Gutov <dgutov@yandex.ru> writes: > Hi! > >> branch: master >> commit d20acfe06340953dce443795d28ac224a2223414 >> Author: Eric Abrahamsen <eric@ericabrahamsen.net> >> Commit: Katsumi Yamaoka <yamaoka@jpl.org> > > This commit breaks compilation of Emacs master: I just got a bunch of mean emails from the buildbot... Sorry about that. I didn't know enough about the build process to anticipate this failure -- I guess most-positive-fixnum isn't available when batch-compiling? I've attached a patch that puts things back the way they were. Eric [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #2: 0001-Revert-gnus-registry-slot-initform-to-previous-value.patch --] [-- Type: text/x-diff, Size: 921 bytes --] From c556f9b91e9a1d2d0fefb7440d49d09ba375b0fa Mon Sep 17 00:00:00 2001 From: Eric Abrahamsen <eric@ericabrahamsen.net> Date: Thu, 18 Dec 2014 23:22:43 +0800 Subject: [PATCH] Revert gnus registry slot initform to previous value * lisp/gnus/registry.el (registry-db): Hardcode an integer; using most-positive-fixnum breaks compilation. --- lisp/gnus/registry.el | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lisp/gnus/registry.el b/lisp/gnus/registry.el index d086d64..bd711e7 100644 --- a/lisp/gnus/registry.el +++ b/lisp/gnus/registry.el @@ -98,7 +98,7 @@ :type (or null float) :documentation "The registry version.") (max-size :initarg :max-size - :initform most-positive-fixnum + :initform 5000000 :type integer :custom integer :documentation "The maximum number of registry entries.") -- 2.2.0 [-- Attachment #3: Type: text/plain, Size: 649 bytes --] > Compiling gnus/gnus-registry.el > Eager macro-expansion failure: (invalid-slot-type max-size integer most-positive-fixnum) > Eager macro-expansion failure: (invalid-slot-type max-size integer most-positive-fixnum) > > In toplevel form: > gnus/gnus-registry.el:88:1:Error: Invalid slot type: max-size, integer, most-positive-fixnum > make[3]: *** [gnus/gnus-registry.elc] Error 1 > make[3]: Leaving directory `/home/gutov/vc/emacs/lisp' > make[2]: *** [compile-main] Error 2 > make[2]: Leaving directory `/home/gutov/vc/emacs/lisp' > make[1]: *** [lisp] Error 2 > make[1]: Leaving directory `/home/gutov/vc/emacs' > make: *** [bootstrap] Error 2 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: master d20acfe: Fix Gnus registry pruning and sorting, and rename file 2014-12-18 15:26 ` Eric Abrahamsen @ 2014-12-18 15:35 ` Andreas Schwab 2014-12-18 21:03 ` Paul Eggert 0 siblings, 1 reply; 17+ messages in thread From: Andreas Schwab @ 2014-12-18 15:35 UTC (permalink / raw) To: Eric Abrahamsen; +Cc: emacs-devel Eric Abrahamsen <eric@ericabrahamsen.net> writes: > Sorry about that. I didn't know enough about the build process to > anticipate this failure -- I guess most-positive-fixnum isn't available > when batch-compiling? No, the problem is that 'most-positive-fixnum is a symbol, not an integer. Andreas. -- Andreas Schwab, SUSE Labs, schwab@suse.de GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7 "And now for something completely different." ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: master d20acfe: Fix Gnus registry pruning and sorting, and rename file 2014-12-18 15:35 ` Andreas Schwab @ 2014-12-18 21:03 ` Paul Eggert 2014-12-19 1:41 ` Ted Zlatanov 2014-12-19 1:55 ` Eric Abrahamsen 0 siblings, 2 replies; 17+ messages in thread From: Paul Eggert @ 2014-12-18 21:03 UTC (permalink / raw) To: Eric Abrahamsen; +Cc: Andreas Schwab, emacs-devel [-- Attachment #1: Type: text/plain, Size: 220 bytes --] On 12/18/2014 07:35 AM, Andreas Schwab wrote: > No, the problem is that 'most-positive-fixnum is a symbol I attempted to work around the problem by installing the attached patch. Please let me know if it's not right. [-- Attachment #2: 0001-registry.el-registry-db-Set-default-slot-later.patch --] [-- Type: text/x-patch, Size: 1746 bytes --] From 22e70c0440941ff8f77f990292bd20d79da3f99e Mon Sep 17 00:00:00 2001 From: Paul Eggert <eggert@cs.ucla.edu> Date: Thu, 18 Dec 2014 13:00:15 -0800 Subject: [PATCH] * registry.el (registry-db): Set default slot later. This is because its value is not a literal integer. --- lisp/gnus/ChangeLog | 5 +++++ lisp/gnus/registry.el | 4 +++- 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/lisp/gnus/ChangeLog b/lisp/gnus/ChangeLog index 34dd919..6f0ea0f 100644 --- a/lisp/gnus/ChangeLog +++ b/lisp/gnus/ChangeLog @@ -1,3 +1,8 @@ +2014-12-18 Paul Eggert <eggert@cs.ucla.edu> + + * registry.el (registry-db): Set default slot later. + This is because its value is not a literal integer. + 2014-12-18 Stefan Monnier <monnier@iro.umontreal.ca> * mm-util.el (mm-with-unibyte-current-buffer): Mark obsolete and diff --git a/lisp/gnus/registry.el b/lisp/gnus/registry.el index d086d64..cc18407 100644 --- a/lisp/gnus/registry.el +++ b/lisp/gnus/registry.el @@ -98,7 +98,7 @@ :type (or null float) :documentation "The registry version.") (max-size :initarg :max-size - :initform most-positive-fixnum + ;; :initform most-positive-fixnum ;; see below :type integer :custom integer :documentation "The maximum number of registry entries.") @@ -123,6 +123,8 @@ (data :initarg :data :type hash-table :documentation "The data hashtable."))) +;; Do this separately, since defclass doesn't allow expressions in :initform. +(oset-default registry-db max-size most-positive-fixnum) (defmethod initialize-instance :BEFORE ((this registry-db) slots) "Check whether a registry object needs to be upgraded." -- 1.9.3 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: master d20acfe: Fix Gnus registry pruning and sorting, and rename file 2014-12-18 21:03 ` Paul Eggert @ 2014-12-19 1:41 ` Ted Zlatanov 2014-12-19 1:55 ` Eric Abrahamsen 1 sibling, 0 replies; 17+ messages in thread From: Ted Zlatanov @ 2014-12-19 1:41 UTC (permalink / raw) To: emacs-devel On Thu, 18 Dec 2014 13:03:09 -0800 Paul Eggert <eggert@cs.ucla.edu> wrote: PE> On 12/18/2014 07:35 AM, Andreas Schwab wrote: >> No, the problem is that 'most-positive-fixnum is a symbol PE> I attempted to work around the problem by installing the attached PE> patch. Please let me know if it's not right. It was perfect, thanks. Ted ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: master d20acfe: Fix Gnus registry pruning and sorting, and rename file 2014-12-18 21:03 ` Paul Eggert 2014-12-19 1:41 ` Ted Zlatanov @ 2014-12-19 1:55 ` Eric Abrahamsen 1 sibling, 0 replies; 17+ messages in thread From: Eric Abrahamsen @ 2014-12-19 1:55 UTC (permalink / raw) To: emacs-devel Paul Eggert <eggert@cs.ucla.edu> writes: > On 12/18/2014 07:35 AM, Andreas Schwab wrote: >> No, the problem is that 'most-positive-fixnum is a symbol > > I attempted to work around the problem by installing the attached > patch. Please let me know if it's not right. Thank you for the fix! > From 22e70c0440941ff8f77f990292bd20d79da3f99e Mon Sep 17 00:00:00 2001 > From: Paul Eggert <eggert@cs.ucla.edu> > Date: Thu, 18 Dec 2014 13:00:15 -0800 > Subject: [PATCH] * registry.el (registry-db): Set default slot later. > > This is because its value is not a literal integer. > --- > lisp/gnus/ChangeLog | 5 +++++ > lisp/gnus/registry.el | 4 +++- > 2 files changed, 8 insertions(+), 1 deletion(-) > > diff --git a/lisp/gnus/ChangeLog b/lisp/gnus/ChangeLog > index 34dd919..6f0ea0f 100644 > --- a/lisp/gnus/ChangeLog > +++ b/lisp/gnus/ChangeLog > @@ -1,3 +1,8 @@ > +2014-12-18 Paul Eggert <eggert@cs.ucla.edu> > + > + * registry.el (registry-db): Set default slot later. > + This is because its value is not a literal integer. > + > 2014-12-18 Stefan Monnier <monnier@iro.umontreal.ca> > > * mm-util.el (mm-with-unibyte-current-buffer): Mark obsolete and > diff --git a/lisp/gnus/registry.el b/lisp/gnus/registry.el > index d086d64..cc18407 100644 > --- a/lisp/gnus/registry.el > +++ b/lisp/gnus/registry.el > @@ -98,7 +98,7 @@ > :type (or null float) > :documentation "The registry version.") > (max-size :initarg :max-size > - :initform most-positive-fixnum > + ;; :initform most-positive-fixnum ;; see below > :type integer > :custom integer > :documentation "The maximum number of registry entries.") > @@ -123,6 +123,8 @@ > (data :initarg :data > :type hash-table > :documentation "The data hashtable."))) > +;; Do this separately, since defclass doesn't allow expressions in :initform. > +(oset-default registry-db max-size most-positive-fixnum) > > (defmethod initialize-instance :BEFORE ((this registry-db) slots) > "Check whether a registry object needs to be upgraded." ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: master d20acfe: Fix Gnus registry pruning and sorting, and rename file 2014-12-18 14:33 ` master d20acfe: Fix Gnus registry pruning and sorting, and rename file Dmitry Gutov 2014-12-18 15:26 ` Eric Abrahamsen @ 2014-12-18 23:11 ` Katsumi Yamaoka 1 sibling, 0 replies; 17+ messages in thread From: Katsumi Yamaoka @ 2014-12-18 23:11 UTC (permalink / raw) To: Dmitry Gutov; +Cc: emacs-devel On Thu, 18 Dec 2014 16:33:18 +0200, Dmitry Gutov wrote: > Hi! >> branch: master >> commit d20acfe06340953dce443795d28ac224a2223414 >> Author: Eric Abrahamsen <eric@ericabrahamsen.net> >> Commit: Katsumi Yamaoka <yamaoka@jpl.org> > This commit breaks compilation of Emacs master: [...] This should have been fixed now, thanks to Paul Eggert. ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Emacs-diffs] master d20acfe: Fix Gnus registry pruning and sorting, and rename file [not found] ` <E1Y1ZAB-0003ek-Vw@vcs.savannah.gnu.org> 2014-12-18 14:33 ` master d20acfe: Fix Gnus registry pruning and sorting, and rename file Dmitry Gutov @ 2014-12-19 7:39 ` Glenn Morris 2014-12-19 12:18 ` Eric Abrahamsen 2014-12-20 11:19 ` Ted Zlatanov 1 sibling, 2 replies; 17+ messages in thread From: Glenn Morris @ 2014-12-19 7:39 UTC (permalink / raw) To: emacs-devel; +Cc: Eric Abrahamsen, Katsumi Yamaoka Some small comments on this change: > +(make-obsolete-variable 'gnus-registry-max-pruned-entries nil "24.4") I think you meant 25.1. Also, you haven't made it obsolete, you've removed it entirely: > -(defcustom gnus-registry-max-pruned-entries nil > - "Maximum number of pruned entries in the registry, nil for unlimited." > - :version "24.1" > +(defcustom gnus-registry-prune-factor 0.1 [...] > + :version "24.4" Again, 25.1. > +(defcustom gnus-registry-default-sort-function > + #'gnus-registry-sort-by-creation-time > + "Sort function to use when pruning the registry. > + > +Entries which sort to the front of the list will be pruned > +first. > + > +This can slow pruning down. Set to nil to perform no sorting." > + :version "24.4" > + :group 'gnus-registry > + :type 'symbol) Again, 25.1. Also, a better :type would be: (choice (const :tag "No sorting" nil) function) Also, as a general comment, Gnus changes almost never seem to get NEWS (or GNUS-NEWS) entries. ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Emacs-diffs] master d20acfe: Fix Gnus registry pruning and sorting, and rename file 2014-12-19 7:39 ` [Emacs-diffs] " Glenn Morris @ 2014-12-19 12:18 ` Eric Abrahamsen 2014-12-20 11:19 ` Ted Zlatanov 1 sibling, 0 replies; 17+ messages in thread From: Eric Abrahamsen @ 2014-12-19 12:18 UTC (permalink / raw) To: Glenn Morris; +Cc: Katsumi Yamaoka, emacs-devel On 12/19/14 02:39 AM, Glenn Morris wrote: > Some small comments on this change: > >> +(make-obsolete-variable 'gnus-registry-max-pruned-entries nil "24.4") > > I think you meant 25.1. > Also, you haven't made it obsolete, you've removed it entirely: > >> -(defcustom gnus-registry-max-pruned-entries nil >> - "Maximum number of pruned entries in the registry, nil for unlimited." >> - :version "24.1" > >> +(defcustom gnus-registry-prune-factor 0.1 > [...] >> + :version "24.4" > > Again, 25.1. > >> +(defcustom gnus-registry-default-sort-function >> + #'gnus-registry-sort-by-creation-time >> + "Sort function to use when pruning the registry. >> + >> +Entries which sort to the front of the list will be pruned >> +first. >> + >> +This can slow pruning down. Set to nil to perform no sorting." >> + :version "24.4" >> + :group 'gnus-registry >> + :type 'symbol) > > Again, 25.1. Also, a better :type would be: > > (choice (const :tag "No sorting" nil) function) > > > Also, as a general comment, Gnus changes almost never seem to get NEWS > (or GNUS-NEWS) entries. Thanks for these comments! To be honest, I don't really understand the relationship between the gnus repository on gnus.org, and the gnus codebase as it lives inside the emacs repo. I wrote those patches against the gnus repo, and at the time (this was a while ago) emacs 25 wasn't out yet. Dunno how these things are meant to related. Other points well taken, and I can provide a further patch to fix them. Eric ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Emacs-diffs] master d20acfe: Fix Gnus registry pruning and sorting, and rename file 2014-12-19 7:39 ` [Emacs-diffs] " Glenn Morris 2014-12-19 12:18 ` Eric Abrahamsen @ 2014-12-20 11:19 ` Ted Zlatanov 2014-12-21 4:39 ` Eric Abrahamsen 1 sibling, 1 reply; 17+ messages in thread From: Ted Zlatanov @ 2014-12-20 11:19 UTC (permalink / raw) To: emacs-devel On Fri, 19 Dec 2014 02:39:23 -0500 Glenn Morris <rgm@gnu.org> wrote: GM> Also, as a general comment, Gnus changes almost never seem to get NEWS GM> (or GNUS-NEWS) entries. This one didn't seem to merit it, it was mostly internal (except for the default gnus-registry filename). Eric, do you want to write something? On Fri, 19 Dec 2014 20:18:40 +0800 Eric Abrahamsen <eric@ericabrahamsen.net> wrote: EA> Thanks for these comments! To be honest, I don't really understand the EA> relationship between the gnus repository on gnus.org, and the gnus EA> codebase as it lives inside the emacs repo. I wrote those patches EA> against the gnus repo, and at the time (this was a while ago) emacs 25 EA> wasn't out yet. Dunno how these things are meant to related. The patch reviewer (me) should have adjusted those, so it was my fault. EA> Other points well taken, and I can provide a further patch to fix them. Please do and I'll apply it, thanks. If you could add the 25.1 version fixes Glenn noted as well, it would be really helpful. Ted ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Emacs-diffs] master d20acfe: Fix Gnus registry pruning and sorting, and rename file 2014-12-20 11:19 ` Ted Zlatanov @ 2014-12-21 4:39 ` Eric Abrahamsen 2014-12-21 14:30 ` Stefan Monnier 2014-12-24 12:46 ` Ted Zlatanov 0 siblings, 2 replies; 17+ messages in thread From: Eric Abrahamsen @ 2014-12-21 4:39 UTC (permalink / raw) To: emacs-devel [-- Attachment #1: Type: text/plain, Size: 1379 bytes --] Ted Zlatanov <tzz@lifelogs.com> writes: > On Fri, 19 Dec 2014 02:39:23 -0500 Glenn Morris <rgm@gnu.org> wrote: > > GM> Also, as a general comment, Gnus changes almost never seem to get NEWS > GM> (or GNUS-NEWS) entries. > > This one didn't seem to merit it, it was mostly internal (except for the > default gnus-registry filename). Eric, do you want to write something? > > On Fri, 19 Dec 2014 20:18:40 +0800 Eric Abrahamsen <eric@ericabrahamsen.net> wrote: > > EA> Thanks for these comments! To be honest, I don't really understand the > EA> relationship between the gnus repository on gnus.org, and the gnus > EA> codebase as it lives inside the emacs repo. I wrote those patches > EA> against the gnus repo, and at the time (this was a while ago) emacs 25 > EA> wasn't out yet. Dunno how these things are meant to related. > > The patch reviewer (me) should have adjusted those, so it was my fault. > > EA> Other points well taken, and I can provide a further patch to fix them. > > Please do and I'll apply it, thanks. If you could add the 25.1 version > fixes Glenn noted as well, it would be really helpful. > > Ted Here goes! I think this is everything. I wasn't aware that obsolete variables still need to be defined, and am not sure what I've done is correct. The other `make-obsolete-variable' statements in here refer to otherwise-undefined variables.... ? Eric [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #2: 0001-Fix-version-numbers-for-registry-var-updates.patch --] [-- Type: text/x-diff, Size: 2695 bytes --] From e3a354642abfd18c08c9110a1bfbc7c9a50cf8e1 Mon Sep 17 00:00:00 2001 From: Eric Abrahamsen <eric@ericabrahamsen.net> Date: Sun, 21 Dec 2014 12:32:07 +0800 Subject: [PATCH] Fix version numbers for registry var updates * lisp/gnus-registry.el: Emacs version number for altered variables and options should have been 25.1. Obsolete variables still need to be defined. Improve custom type of `gnus-registry-default-sort-function'. * GNUS-NEWS: Note earlier change of default registry filename --- GNUS-NEWS | 3 +++ lisp/gnus-registry.el | 10 ++++++---- 2 files changed, 9 insertions(+), 4 deletions(-) diff --git a/GNUS-NEWS b/GNUS-NEWS index 16808af..777bc4c 100644 --- a/GNUS-NEWS +++ b/GNUS-NEWS @@ -9,6 +9,9 @@ For older news, see Gnus info node "New Features". \f * New features +** Default Gnus registry filename changed from "eioio" extension to + "eieio". + ** New package `gnus-notifications.el' can send notifications when you receive new messages. diff --git a/lisp/gnus-registry.el b/lisp/gnus-registry.el index 92f8f04..098248b 100644 --- a/lisp/gnus-registry.el +++ b/lisp/gnus-registry.el @@ -171,12 +171,14 @@ nnmairix groups are specifically excluded because they are ephemeral." (defvar gnus-registry-misc-menus nil) ; ugly way to keep the menus +(defvar gnus-registry-max-pruned-entries) + (make-obsolete-variable 'gnus-registry-clean-empty nil "23.4") (make-obsolete-variable 'gnus-registry-use-long-group-names nil "23.4") (make-obsolete-variable 'gnus-registry-max-track-groups nil "23.4") (make-obsolete-variable 'gnus-registry-entry-caching nil "23.4") (make-obsolete-variable 'gnus-registry-trim-articles-without-groups nil "23.4") -(make-obsolete-variable 'gnus-registry-max-pruned-entries nil "24.4") +(make-obsolete-variable 'gnus-registry-max-pruned-entries nil "25.1") (defcustom gnus-registry-track-extra '(subject sender recipient) "Whether the registry should track extra data about a message. @@ -253,7 +255,7 @@ exactly how much less. For example, given a maximum size of cut the registry back to \(- 50000 \(* 50000 0.1\)\) -> 45000 entries. The pruning process is constrained by the presence of \"precious\" entries." - :version "24.4" + :version "25.1" :group 'gnus-registry :type 'float) @@ -265,9 +267,9 @@ Entries which sort to the front of the list will be pruned first. This can slow pruning down. Set to nil to perform no sorting." - :version "24.4" + :version "25.1" :group 'gnus-registry - :type 'symbol) + :type (choice (const :tag "No sorting" nil) function)) (defun gnus-registry-sort-by-creation-time (l r) "Sort older entries to front of list." -- 2.2.1 ^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [Emacs-diffs] master d20acfe: Fix Gnus registry pruning and sorting, and rename file 2014-12-21 4:39 ` Eric Abrahamsen @ 2014-12-21 14:30 ` Stefan Monnier 2014-12-24 12:44 ` Ted Zlatanov 2014-12-24 12:46 ` Ted Zlatanov 1 sibling, 1 reply; 17+ messages in thread From: Stefan Monnier @ 2014-12-21 14:30 UTC (permalink / raw) To: Eric Abrahamsen; +Cc: emacs-devel > Here goes! I think this is everything. I wasn't aware that obsolete > variables still need to be defined, The purpose of marking something obsolete is to try and make sure people don't keep using it without realizing that this functionality is about to disappear. If the variable/function is already gone, then there's no need for any make-obsolete(-variable). Stefan ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Emacs-diffs] master d20acfe: Fix Gnus registry pruning and sorting, and rename file 2014-12-21 14:30 ` Stefan Monnier @ 2014-12-24 12:44 ` Ted Zlatanov 0 siblings, 0 replies; 17+ messages in thread From: Ted Zlatanov @ 2014-12-24 12:44 UTC (permalink / raw) To: emacs-devel On Sun, 21 Dec 2014 09:30:30 -0500 Stefan Monnier <monnier@iro.umontreal.ca> wrote: >> Here goes! I think this is everything. I wasn't aware that obsolete >> variables still need to be defined, SM> The purpose of marking something obsolete is to try and make sure people SM> don't keep using it without realizing that this functionality is about SM> to disappear. SM> If the variable/function is already gone, then there's no need for any SM> make-obsolete(-variable). Yup. And specifically here, I thought it was reasonable because the change is not highly visible to users and will show up in 25.1 only. Ted ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Emacs-diffs] master d20acfe: Fix Gnus registry pruning and sorting, and rename file 2014-12-21 4:39 ` Eric Abrahamsen 2014-12-21 14:30 ` Stefan Monnier @ 2014-12-24 12:46 ` Ted Zlatanov 2014-12-24 14:35 ` Eric Abrahamsen 1 sibling, 1 reply; 17+ messages in thread From: Ted Zlatanov @ 2014-12-24 12:46 UTC (permalink / raw) To: emacs-devel On Sun, 21 Dec 2014 12:39:49 +0800 Eric Abrahamsen <eric@ericabrahamsen.net> wrote: EA> Here goes! I think this is everything. I wasn't aware that obsolete EA> variables still need to be defined, and am not sure what I've done is EA> correct. The other `make-obsolete-variable' statements in here refer to EA> otherwise-undefined variables.... ? The patch looks good, minus the obsolete variables. Please also add ChangeLog entries, then it can go in straight away. Ted ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Emacs-diffs] master d20acfe: Fix Gnus registry pruning and sorting, and rename file 2014-12-24 12:46 ` Ted Zlatanov @ 2014-12-24 14:35 ` Eric Abrahamsen 2014-12-24 15:32 ` Ted Zlatanov 0 siblings, 1 reply; 17+ messages in thread From: Eric Abrahamsen @ 2014-12-24 14:35 UTC (permalink / raw) To: emacs-devel Ted Zlatanov <tzz@lifelogs.com> writes: > On Sun, 21 Dec 2014 12:39:49 +0800 Eric Abrahamsen <eric@ericabrahamsen.net> wrote: > > EA> Here goes! I think this is everything. I wasn't aware that obsolete > EA> variables still need to be defined, and am not sure what I've done is > EA> correct. The other `make-obsolete-variable' statements in here refer to > EA> otherwise-undefined variables.... ? > > The patch looks good, minus the obsolete variables. > > Please also add ChangeLog entries, then it can go in straight away. > > Ted Okay, just to continue being deliberately dumb about this: the change is already in lisp/ChangeLog, but not in the top-level ChangeLog, and the patch I sent in already adds a brief mention in GNUS_NEWS. So do I also want to put it in top-level ChangeLog? Sorry, I'd just like to get this right! Eric ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Emacs-diffs] master d20acfe: Fix Gnus registry pruning and sorting, and rename file 2014-12-24 14:35 ` Eric Abrahamsen @ 2014-12-24 15:32 ` Ted Zlatanov 2014-12-25 4:27 ` Eric Abrahamsen 0 siblings, 1 reply; 17+ messages in thread From: Ted Zlatanov @ 2014-12-24 15:32 UTC (permalink / raw) To: emacs-devel On Wed, 24 Dec 2014 22:35:16 +0800 Eric Abrahamsen <eric@ericabrahamsen.net> wrote: EA> Okay, just to continue being deliberately dumb about this: the change is EA> already in lisp/ChangeLog, but not in the top-level ChangeLog, and the EA> patch I sent in already adds a brief mention in GNUS_NEWS. So do I also EA> want to put it in top-level ChangeLog? Think of the ChangeLog as a restatement of the commit. So in this case, add a new entry to the ChangeLog even though you're not adding new features. Just say what you did ("Add version" etc.) and I can adjust if needed, but it should be part of the patch to make life easier for committers. I think you also want to update the GNUS-NEWS ChangeLog, likewise. Ted ^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [Emacs-diffs] master d20acfe: Fix Gnus registry pruning and sorting, and rename file 2014-12-24 15:32 ` Ted Zlatanov @ 2014-12-25 4:27 ` Eric Abrahamsen 0 siblings, 0 replies; 17+ messages in thread From: Eric Abrahamsen @ 2014-12-25 4:27 UTC (permalink / raw) To: emacs-devel [-- Attachment #1: Type: text/plain, Size: 845 bytes --] Ted Zlatanov <tzz@lifelogs.com> writes: > On Wed, 24 Dec 2014 22:35:16 +0800 Eric Abrahamsen <eric@ericabrahamsen.net> wrote: > > EA> Okay, just to continue being deliberately dumb about this: the change is > EA> already in lisp/ChangeLog, but not in the top-level ChangeLog, and the > EA> patch I sent in already adds a brief mention in GNUS_NEWS. So do I also > EA> want to put it in top-level ChangeLog? > > Think of the ChangeLog as a restatement of the commit. So in this case, > add a new entry to the ChangeLog even though you're not adding new > features. Just say what you did ("Add version" etc.) and I can adjust if > needed, but it should be part of the patch to make life easier for > committers. > > I think you also want to update the GNUS-NEWS ChangeLog, likewise. Okay, this should be it. Sorry for all the questions! Eric [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #2: 0001-Fix-version-numbers-for-registry-var-updates.patch --] [-- Type: text/x-diff, Size: 3306 bytes --] From c52b8af5e0c577e73a90911ab0a5622a9f3a669a Mon Sep 17 00:00:00 2001 From: Eric Abrahamsen <eric@ericabrahamsen.net> Date: Sun, 21 Dec 2014 12:32:07 +0800 Subject: [PATCH] Fix version numbers for registry var updates * lisp/gnus-registry.el: Emacs version number for altered variables and options should have been 25.1. There's no need to keep `make-obsolete-variable' statements around for nonexistent variables. Improve custom type of `gnus-registry-default-sort-function'. * GNUS-NEWS: Note earlier change of default registry filename * ChangeLog: Note change of default registry filename, and squashing of pruning bug in af0fe49. --- ChangeLog | 6 ++++++ GNUS-NEWS | 3 +++ lisp/gnus-registry.el | 13 +++---------- 3 files changed, 12 insertions(+), 10 deletions(-) diff --git a/ChangeLog b/ChangeLog index 5f41b90..76bc47e 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2014-12-25 Eric Abrahamsen <eric@ericabrahamsen.net> + + * lisp/gnus-registry.el: Change default extension of registry data file + from "eioio" to "eieio". Fix bug leading to potential deletion of + "precious" registry entries. + 2014-06-11 Katsumi Yamaoka <yamaoka@jpl.org> * configure.in: Add --without-compress-install option; check for gzip. diff --git a/GNUS-NEWS b/GNUS-NEWS index 16808af..777bc4c 100644 --- a/GNUS-NEWS +++ b/GNUS-NEWS @@ -9,6 +9,9 @@ For older news, see Gnus info node "New Features". \f * New features +** Default Gnus registry filename changed from "eioio" extension to + "eieio". + ** New package `gnus-notifications.el' can send notifications when you receive new messages. diff --git a/lisp/gnus-registry.el b/lisp/gnus-registry.el index 92f8f04..5038781 100644 --- a/lisp/gnus-registry.el +++ b/lisp/gnus-registry.el @@ -171,13 +171,6 @@ nnmairix groups are specifically excluded because they are ephemeral." (defvar gnus-registry-misc-menus nil) ; ugly way to keep the menus -(make-obsolete-variable 'gnus-registry-clean-empty nil "23.4") -(make-obsolete-variable 'gnus-registry-use-long-group-names nil "23.4") -(make-obsolete-variable 'gnus-registry-max-track-groups nil "23.4") -(make-obsolete-variable 'gnus-registry-entry-caching nil "23.4") -(make-obsolete-variable 'gnus-registry-trim-articles-without-groups nil "23.4") -(make-obsolete-variable 'gnus-registry-max-pruned-entries nil "24.4") - (defcustom gnus-registry-track-extra '(subject sender recipient) "Whether the registry should track extra data about a message. The subject, recipients (To: and Cc:), and Sender (From:) headers @@ -253,7 +246,7 @@ exactly how much less. For example, given a maximum size of cut the registry back to \(- 50000 \(* 50000 0.1\)\) -> 45000 entries. The pruning process is constrained by the presence of \"precious\" entries." - :version "24.4" + :version "25.1" :group 'gnus-registry :type 'float) @@ -265,9 +258,9 @@ Entries which sort to the front of the list will be pruned first. This can slow pruning down. Set to nil to perform no sorting." - :version "24.4" + :version "25.1" :group 'gnus-registry - :type 'symbol) + :type '(choice (const :tag "No sorting" nil) function)) (defun gnus-registry-sort-by-creation-time (l r) "Sort older entries to front of list." -- 2.2.1 ^ permalink raw reply related [flat|nested] 17+ messages in thread
end of thread, other threads:[~2014-12-25 4:27 UTC | newest] Thread overview: 17+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- [not found] <20141218112235.14020.81034@vcs.savannah.gnu.org> [not found] ` <E1Y1ZAB-0003ek-Vw@vcs.savannah.gnu.org> 2014-12-18 14:33 ` master d20acfe: Fix Gnus registry pruning and sorting, and rename file Dmitry Gutov 2014-12-18 15:26 ` Eric Abrahamsen 2014-12-18 15:35 ` Andreas Schwab 2014-12-18 21:03 ` Paul Eggert 2014-12-19 1:41 ` Ted Zlatanov 2014-12-19 1:55 ` Eric Abrahamsen 2014-12-18 23:11 ` Katsumi Yamaoka 2014-12-19 7:39 ` [Emacs-diffs] " Glenn Morris 2014-12-19 12:18 ` Eric Abrahamsen 2014-12-20 11:19 ` Ted Zlatanov 2014-12-21 4:39 ` Eric Abrahamsen 2014-12-21 14:30 ` Stefan Monnier 2014-12-24 12:44 ` Ted Zlatanov 2014-12-24 12:46 ` Ted Zlatanov 2014-12-24 14:35 ` Eric Abrahamsen 2014-12-24 15:32 ` Ted Zlatanov 2014-12-25 4:27 ` Eric Abrahamsen
Code repositories for project(s) associated with this external index https://git.savannah.gnu.org/cgit/emacs.git https://git.savannah.gnu.org/cgit/emacs/org-mode.git This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.