Re: [PATCH 0/2] emacs: show: use interactive instead of current-prefix-arg

2013-10-10 Thread Tomi Ollila
On Wed, Oct 09 2013, Mark Walters markwalters1...@gmail.com wrote: This is a tidied up version of the patch at id:1380729013-3942-1-git-send-email-markwalters1...@gmail.com (incorporating suggestions from Tomi in id:m2siwjhapc@guru.guru-group.fi) This helps fix a problem in pick but

Re: [PATCH] notmuch compact support (v4)

2013-10-10 Thread David Bremner
Ben Gamari bgamari.f...@gmail.com writes: Here is yet another iteration of my notmuch compact patchset. It has been rebased on top of master and the interface reworked as suggested by Jani. Sorry, first version of this reply didn't go to the list. I haven't had time to do a review, but I did

Re: [PATCH 1/3] cli: add insert --must-index option

2013-10-10 Thread David Bremner
Peter Wang noval...@gmail.com writes: On Tue, 10 Sep 2013 09:06:00 +0100, Mark Walters markwalters1...@gmail.com wrote: Alternatively maybe add notmuch_database_destroy_with_flush or something which does give a return value. notmuch_database_close is only called 3 times and

Re: [PATCH 7/6] emacs: Improved `notmuch-describe-keymap' documentation

2013-10-10 Thread David Bremner
Austin Clements amdra...@mit.edu writes: +Each string gives a human-readable description of the key and a +one-line description of the bound function. See `notmuch-help' +for an overview of how this documentation is extracted. + +UA-KEYS should be a key sequence bound to

Re: [PATCH 0/3] General fixes

2013-10-10 Thread David Bremner
Felipe Contreras felipe.contre...@gmail.com writes: Felipe Contreras (3): query: bind queries to database objects ruby: allow build with RUNPATH ruby: bind database close()/destroy() properly Hi Felipe; I agree with the discussion on IRC that the change in the first patch makes sense.

Re: [PATCH] notmuch compact support (v4)

2013-10-10 Thread Ben Gamari
David Bremner da...@tethera.net writes: Ben Gamari bgamari.f...@gmail.com writes: Here is yet another iteration of my notmuch compact patchset. It has been rebased on top of master and the interface reworked as suggested by Jani. Sorry, first version of this reply didn't go to the list.

Re: [PATCH 1/3] cli: add insert --must-index option

2013-10-10 Thread Tomi Ollila
On Thu, Oct 10 2013, David Bremner da...@tethera.net wrote: Peter Wang noval...@gmail.com writes: On Tue, 10 Sep 2013 09:06:00 +0100, Mark Walters markwalters1...@gmail.com wrote: Alternatively maybe add notmuch_database_destroy_with_flush or something which does give a return value.

Re: [PATCH 1/3] cli: add insert --must-index option

2013-10-10 Thread David Bremner
Tomi Ollila tomi.oll...@iki.fi writes: I'm not opposed to doing an SONAME bump for 0.17. Are there other ABI breaking changes that we have been holding back on? Can these maybe go through at the same time? Maybe something along these lines... (Quick draft for the API part; to start

Re: [PATCH 1/3] database: Add notmuch_database_compact_close

2013-10-10 Thread Tomi Ollila
On Wed, Oct 02 2013, Ben Gamari bgamari.f...@gmail.com wrote: This function uses Xapian's Compactor machinery to compact the notmuch database. The compacted database is built in a temporary directory and later moved into place while the original uncompacted database is preserved.

possible infinite recursion in notmuch-cli

2013-10-10 Thread Daniel Kahn Gillmor
Hi notmuch folks-- On oss-security recently, there was a discussion about recursive compression and the ability to create infinite loops. id:20131010013106.ga29...@openwall.com After some discussion with amdragon on IRC, i believe that this is only relevant to notmuch when actively decrypting

[PATCH 0/2] emacs: show: use interactive instead of current-prefix-arg

2013-10-10 Thread Tomi Ollila
On Wed, Oct 09 2013, Mark Walters wrote: > This is a tidied up version of the patch at > id:1380729013-3942-1-git-send-email-markwalters1009 at gmail.com > (incorporating suggestions from Tomi in > id:m2siwjhapc.fsf at guru.guru-group.fi) > > This helps fix a problem in pick but there was

[PATCH] notmuch compact support (v4)

2013-10-10 Thread David Bremner
Ben Gamari writes: > Here is yet another iteration of my "notmuch compact" patchset. It has been > rebased on top of master and the interface reworked as suggested by Jani. > Sorry, first version of this reply didn't go to the list. I haven't had time to do a review, but I did try them out,

[PATCH 1/3] cli: add insert --must-index option

2013-10-10 Thread David Bremner
Peter Wang writes: > On Tue, 10 Sep 2013 09:06:00 +0100, Mark Walters gmail.com> wrote: >> >> Alternatively maybe add notmuch_database_destroy_with_flush or something >> which does give a return value. notmuch_database_close is only called 3 >> times and notmuch_database_destroy lots of times

[PATCH 7/6] emacs: Improved `notmuch-describe-keymap' documentation

2013-10-10 Thread David Bremner
Austin Clements writes: > +Each string gives a human-readable description of the key and a > +one-line description of the bound function. See `notmuch-help' > +for an overview of how this documentation is extracted. > + > +UA-KEYS should be a key sequence bound to `universal-argument'. > +It

[PATCH 0/3] General fixes

2013-10-10 Thread David Bremner
Felipe Contreras writes: > Felipe Contreras (3): > query: bind queries to database objects > ruby: allow build with RUNPATH > ruby: bind database close()/destroy() properly Hi Felipe; I agree with the discussion on IRC that the change in the first patch makes sense. It would be nice to

[PATCH 1/3] cli: add insert --must-index option

2013-10-10 Thread Tomi Ollila
On Thu, Oct 10 2013, David Bremner wrote: > Peter Wang writes: > >> On Tue, 10 Sep 2013 09:06:00 +0100, Mark Walters > gmail.com> wrote: >>> >>> Alternatively maybe add notmuch_database_destroy_with_flush or something >>> which does give a return value. notmuch_database_close is only called 3

[PATCH 1/3] cli: add insert --must-index option

2013-10-10 Thread David Bremner
Tomi Ollila writes: >> I'm not opposed to doing an SONAME bump for 0.17. Are there other ABI >> breaking changes that we have been holding back on? Can these maybe go >> through at the same time? > > Maybe something along these lines... > > (Quick draft for the API part; to start discussion

[PATCH 1/3] database: Add notmuch_database_compact_close

2013-10-10 Thread Tomi Ollila
On Wed, Oct 02 2013, Ben Gamari wrote: > This function uses Xapian's Compactor machinery to compact the notmuch > database. The compacted database is built in a temporary directory and > later moved into place while the original uncompacted database is > preserved. > > Signed-off-by: Ben Gamari

possible infinite recursion in notmuch-cli

2013-10-10 Thread Daniel Kahn Gillmor
://www.openwall.com/lists/oss-security/2013/10/10/2 -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 965 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20131010/0cb76