Re: [PATCH] emacs: Put notmuch-hello-sections in custom group notmuch-hello

2012-04-15 Thread Jani Nikula
Hi Austin, id:"1330811724-30901-1-git-send-email-j...@nikula.org" :) Mark had a valid point about ordering within groups, which I didn't follow up on, but this is a necessary intermediate step in the right direction no matter what. I don't care whose patches get merged, but let's merge the change

some stale patches dropped from the review queue

2012-04-15 Thread Jani Nikula
On Sun, 15 Apr 2012, Jani Nikula wrote: > I have tagged the following patches notmuch::stale, removing them from > the review queue [1], purely based on they not applying to master > anymore (see [2] for tag definitions). Please rebase your patches > against master and resubmit if you think they'r

incrontab?

2012-04-15 Thread David Belohrad
Hi Adam, interestingly, your 'G' short-cut hint works out of the box! What happens is, that when one presses 'G', this invokes notmuch-hello-poll-and-update. This function calls 'notmuch-poll', which does exactly what I want. I.e. it runs 'notmuch new' if no polling script is specified. In my case

[PATCH] emacs: Put notmuch-print-mechanism in custom group notmuch-show

2012-04-15 Thread Austin Clements
--- emacs/notmuch-print.el |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/emacs/notmuch-print.el b/emacs/notmuch-print.el index 6653d97..8c18f4b 100644 --- a/emacs/notmuch-print.el +++ b/emacs/notmuch-print.el @@ -25,7 +25,7 @@ (defcustom notmuch-print-mechanism 'notmu

[PATCH] emacs: Put notmuch-hello-sections in custom group notmuch-hello

2012-04-15 Thread Austin Clements
--- emacs/notmuch-hello.el |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/emacs/notmuch-hello.el b/emacs/notmuch-hello.el index 71d37b8..f10d98d 100644 --- a/emacs/notmuch-hello.el +++ b/emacs/notmuch-hello.el @@ -226,7 +226,7 @@ by an additional filter query. Similarly,

Re: [PATCH 1/3] emacs: modify help message for notmuch-search-line-faces to reflect preferred "deleted" tag name.

2012-04-15 Thread Jameson Graef Rollins
On Sun, Apr 15 2012, Tomi Ollila wrote: > id:"1326826969-23545-1-git-send-email-jroll...@finestructure.net" > > Does just "delete" -> "deleted" change: the only question I have left > is that should that be left as is, this change made, or just drop > the "delete" coloring altogether ? Sorry, yes

[PATCH 1/3] emacs: modify help message for notmuch-search-line-faces to reflect preferred "deleted" tag name.

2012-04-15 Thread Jameson Graef Rollins
-- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120415/d937054f/attachment.pgp>

Re: some stale patches dropped from the review queue

2012-04-15 Thread Jameson Graef Rollins
Sorry, I'm out of touch with nmbug at the moment. I'll try to get reacquainted. In the mean time... > id:1325986015-22510-3-git-send-email-jroll...@finestructure.net > id:1325986015-22510-4-git-send-email-jroll...@finestructure.net > id:1325986015-22510-5-git-send-email-jroll...@finestructure.n

some stale patches dropped from the review queue

2012-04-15 Thread Jameson Graef Rollins
oleted by a later version: id:"1334429574-12918-1-git-send-email-jrollins at finestructure.net" jamie. -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120415/8cc37493/attachment.pgp>

Re: [PATCH 1/3] emacs: modify help message for notmuch-search-line-faces to reflect preferred "deleted" tag name.

2012-04-15 Thread Tomi Ollila
On Sun, Apr 15 2012, Jameson Graef Rollins wrote: > On Sun, Apr 15 2012, Mark Walters wrote: >> This patch is trivially correct regardless of the rest of the >> series. >> >> I think the rest of the series (which provides keybindings for >> adding/removing the delete tag to messages/threads) is

incrontab?

2012-04-15 Thread Austin Clements
Quoth David Belohrad on Apr 15 at 11:39 pm: > There is just one slightly weird thing. This is when 'notmuch' is > opened, the focus goes directly to search button instead of 'inbox' > messages (or something else), which is imho more interesting than going into > search. Because > at first what one

[PATCH 1/3] emacs: modify help message for notmuch-search-line-faces to reflect preferred "deleted" tag name.

2012-04-15 Thread Tomi Ollila
On Sun, Apr 15 2012, Mark Walters wrote: > On Tue, 17 Jan 2012, Jameson Graef Rollins > wrote: >> No functional change here. The help message previously referred to >> the "delete" tag, but "deleted" is now preferred, so hopefully this >> will reduce any potential confusion. > > This patch is t

[PATCH 2/2] emacs: Don't move to the next thread unless the cursor is at the end of the buffer.

2012-04-15 Thread Tomi Ollila
On Sun, Apr 15 2012, Mark Walters wrote: > On Tue, 31 Jan 2012, David Edmondson wrote: >> When using the spacebar to scroll through a thread, hitting 'space' >> when the bottom of the last message is visible should take the cursor >> to the end of the buffer rather than immediately archiving the

[No Subject]

2012-04-15 Thread Tomi Ollila
On Sun, Apr 15 2012, Jameson Graef Rollins wrote: > A previous patch [0] replaced blank subject lines with '[No Subject]' > in search and show mode. Apparently this was needed to circumvent > some bug in the printing code, but there was no need for it search or > show, and it is definitely not

[PATCH] emacs: Put notmuch-print-mechanism in custom group notmuch-show

2012-04-15 Thread Austin Clements
--- emacs/notmuch-print.el |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/emacs/notmuch-print.el b/emacs/notmuch-print.el index 6653d97..8c18f4b 100644 --- a/emacs/notmuch-print.el +++ b/emacs/notmuch-print.el @@ -25,7 +25,7 @@ (defcustom notmuch-print-mechanism 'notm

[PATCH] emacs: Put notmuch-hello-sections in custom group notmuch-hello

2012-04-15 Thread Austin Clements
--- emacs/notmuch-hello.el |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/emacs/notmuch-hello.el b/emacs/notmuch-hello.el index 71d37b8..f10d98d 100644 --- a/emacs/notmuch-hello.el +++ b/emacs/notmuch-hello.el @@ -226,7 +226,7 @@ by an additional filter query. Similarly,

[PATCH] emacs: Add configurable wrapping width for notmuch-wash-wrap-long-lines

2012-04-15 Thread Mark Walters
On Fri, 17 Feb 2012, Daniel Schoepe wrote: > This introduces a variable to control after how many characters a line > is wrapped by notmuch-wash-wrap-long-lines (still wrapping at the > window width if it is lower). Hi This looks ok but I wonder if slightly different behaviour might be prefera

[PATCH 1/3] emacs: modify help message for notmuch-search-line-faces to reflect preferred "deleted" tag name.

2012-04-15 Thread Mark Walters
On Tue, 17 Jan 2012, Jameson Graef Rollins wrote: > No functional change here. The help message previously referred to > the "delete" tag, but "deleted" is now preferred, so hopefully this > will reduce any potential confusion. This patch is trivially correct regardless of the rest of the serie

[PATCH 2/2] emacs: Don't move to the next thread unless the cursor is at the end of the buffer.

2012-04-15 Thread Mark Walters
On Tue, 31 Jan 2012, David Edmondson wrote: > When using the spacebar to scroll through a thread, hitting 'space' > when the bottom of the last message is visible should take the cursor > to the end of the buffer rather than immediately archiving the thread > and moving to the next thread. Hi T

Re: incrontab?

2012-04-15 Thread Austin Clements
Quoth David Belohrad on Apr 15 at 11:39 pm: > There is just one slightly weird thing. This is when 'notmuch' is > opened, the focus goes directly to search button instead of 'inbox' > messages (or something else), which is imho more interesting than going into > search. Because > at first what one

Re: incrontab?

2012-04-15 Thread David Belohrad
Hi Adam, interestingly, your 'G' short-cut hint works out of the box! What happens is, that when one presses 'G', this invokes notmuch-hello-poll-and-update. This function calls 'notmuch-poll', which does exactly what I want. I.e. it runs 'notmuch new' if no polling script is specified. In my case

Re: [PATCH 4/4] Explicitly type void* pointers

2012-04-15 Thread Jani Nikula
I tagged this patch notmuch::obsolete (see [1]) as the template workaround was pushed to master (commit de0557477d908be26615e8fda9f5eb62bed68b65). Jani. [1] http://nmbug.tethera.net/status/ On Mon, 09 Apr 2012, vladimir.ma...@oracle.com wrote: > From: Vladimir Marek > > > Signed-off-by: Vlad

Re: some stale patches dropped from the review queue

2012-04-15 Thread Jani Nikula
On Sun, 15 Apr 2012, Jani Nikula wrote: > I have tagged the following patches notmuch::stale, removing them from > the review queue [1], purely based on they not applying to master > anymore (see [2] for tag definitions). Please rebase your patches > against master and resubmit if you think they'r

Re: [PATCH 1/3] emacs: modify help message for notmuch-search-line-faces to reflect preferred "deleted" tag name.

2012-04-15 Thread Jameson Graef Rollins
On Sun, Apr 15 2012, Mark Walters wrote: > This patch is trivially correct regardless of the rest of the > series. > > I think the rest of the series (which provides keybindings for > adding/removing the delete tag to messages/threads) is worthwhile > particularly now the exclude stuff is fairly

[PATCH 1/3] emacs: modify help message for notmuch-search-line-faces to reflect preferred "deleted" tag name.

2012-04-15 Thread Jameson Graef Rollins
gs you're looking for... jamie. -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120415/237d9eec/attachment.pgp>

[PATCH v3 1/4] emacs: Let the user choose where to compose new mails

2012-04-15 Thread David Bremner
Jameson Graef Rollins writes: > I think the issues that David was experiencing have to do with flakiness > in emacs's dedicated windows, not in this patch itself. Thomas, Did you have a change to investigate this as proposed in id:"87zke0aifa.fsf at thor.loria.fr"? d

Re: [PATCH 1/3] emacs: modify help message for notmuch-search-line-faces to reflect preferred "deleted" tag name.

2012-04-15 Thread Tomi Ollila
On Sun, Apr 15 2012, Mark Walters wrote: > On Tue, 17 Jan 2012, Jameson Graef Rollins wrote: >> No functional change here. The help message previously referred to >> the "delete" tag, but "deleted" is now preferred, so hopefully this >> will reduce any potential confusion. > > This patch is triv

Re: [PATCH] emacs: Add configurable wrapping width for notmuch-wash-wrap-long-lines

2012-04-15 Thread Mark Walters
On Fri, 17 Feb 2012, Daniel Schoepe wrote: > This introduces a variable to control after how many characters a line > is wrapped by notmuch-wash-wrap-long-lines (still wrapping at the > window width if it is lower). Hi This looks ok but I wonder if slightly different behaviour might be prefera

Re: [PATCH 2/2] emacs: Don't move to the next thread unless the cursor is at the end of the buffer.

2012-04-15 Thread Tomi Ollila
On Sun, Apr 15 2012, Mark Walters wrote: > On Tue, 31 Jan 2012, David Edmondson wrote: >> When using the spacebar to scroll through a thread, hitting 'space' >> when the bottom of the last message is visible should take the cursor >> to the end of the buffer rather than immediately archiving the

Re: [PATCH 1/3] emacs: modify help message for notmuch-search-line-faces to reflect preferred "deleted" tag name.

2012-04-15 Thread Mark Walters
On Tue, 17 Jan 2012, Jameson Graef Rollins wrote: > No functional change here. The help message previously referred to > the "delete" tag, but "deleted" is now preferred, so hopefully this > will reduce any potential confusion. This patch is trivially correct regardless of the rest of the series

[No Subject]

2012-04-15 Thread Tomi Ollila
On Sun, Apr 15 2012, Jameson Graef Rollins wrote: > A previous patch [0] replaced blank subject lines with '[No Subject]' > in search and show mode. Apparently this was needed to circumvent > some bug in the printing code, but there was no need for it search or > show, and it is definitely not d

Re: [PATCH 2/2] emacs: Don't move to the next thread unless the cursor is at the end of the buffer.

2012-04-15 Thread Mark Walters
On Tue, 31 Jan 2012, David Edmondson wrote: > When using the spacebar to scroll through a thread, hitting 'space' > when the bottom of the last message is visible should take the cursor > to the end of the buffer rather than immediately archiving the thread > and moving to the next thread. Hi T

[PATCH 0/6] Finish show rewrite

2012-04-15 Thread David Bremner
Austin Clements writes: > The long-awaited and oft-belated conclusion of the show rewrite. All > of the formatters have been converted to the new style, so this series > just rips out unused code and does a little cleanup. > pushed, d

[PATCH v2] Record dependencies during build instead of before

2012-04-15 Thread David Bremner
Austin Clements writes: > This patch eliminates the dependency generation rules and instead > generates dependency files as a side-effect of the regular build rule. pushed, d

[PATCH] lib: work around talloc_steal usage from C++ code

2012-04-15 Thread David Bremner
Jani Nikula writes: > Implicit typecast from 'void *' to 'T *' is okay in C, but not in > C++. In talloc_steal, an explicit cast is provided for type safety in > some GCC versions. Otherwise, a cast is required. Provide a template > function for this to maintain type safety, and redefine talloc_s

[PATCH v2 0/6] batch tagging support: "notmuch tag --stdin"

2012-04-15 Thread David Bremner
Jani Nikula writes: > I'm totally fine with modifying the proposed format (e.g. change "T" to > "tag", make things compatible with a future general batch mode), but to > be absolutely clear: I will not implement a general batch command > mode. I was thinking about the best way of making the in

Re: [PATCH v3 1/4] emacs: Let the user choose where to compose new mails

2012-04-15 Thread David Bremner
Jameson Graef Rollins writes: > I think the issues that David was experiencing have to do with flakiness > in emacs's dedicated windows, not in this patch itself. Thomas, Did you have a change to investigate this as proposed in id:"87zke0aifa@thor.loria.fr"? d ___

Re: [PATCH 0/6] Finish show rewrite

2012-04-15 Thread David Bremner
Austin Clements writes: > The long-awaited and oft-belated conclusion of the show rewrite. All > of the formatters have been converted to the new style, so this series > just rips out unused code and does a little cleanup. > pushed, d ___ notmuch mai

Re: [PATCH v2] Record dependencies during build instead of before

2012-04-15 Thread David Bremner
Austin Clements writes: > This patch eliminates the dependency generation rules and instead > generates dependency files as a side-effect of the regular build rule. pushed, d ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mail

Re: [PATCH] lib: work around talloc_steal usage from C++ code

2012-04-15 Thread David Bremner
Jani Nikula writes: > Implicit typecast from 'void *' to 'T *' is okay in C, but not in > C++. In talloc_steal, an explicit cast is provided for type safety in > some GCC versions. Otherwise, a cast is required. Provide a template > function for this to maintain type safety, and redefine talloc_s

Re: [PATCH v2 0/6] batch tagging support: "notmuch tag --stdin"

2012-04-15 Thread David Bremner
Jani Nikula writes: > I'm totally fine with modifying the proposed format (e.g. change "T" to > "tag", make things compatible with a future general batch mode), but to > be absolutely clear: I will not implement a general batch command > mode. I was thinking about the best way of making the in

[PATCH 2/2] emacs: new mua mailto: URI handler

2012-04-15 Thread Jani Nikula
Jameson Graef Rollins writes: > On Sat, Apr 14 2012, Jani Nikula wrote: >> I was doing this purely based on whether the patches apply or not. > > I'm so sorry, Jani. You are correct that this patch requires a small > rebase fix to apply to master. I'm not sure how I missed it previously. No p

some stale patches dropped from the review queue

2012-04-15 Thread Jani Nikula
I have tagged the following patches notmuch::stale, removing them from the review queue [1], purely based on they not applying to master anymore (see [2] for tag definitions). Please rebase your patches against master and resubmit if you think they're still relevant. If you think this is in error,

[PATCH 2/2] emacs: new mua mailto: URI handler

2012-04-15 Thread Jani Nikula
Jameson Graef Rollins writes: > On Sat, Apr 14 2012, Jani Nikula wrote: >> I'm going through the review queue right now, tagging notmuch::stale any >> patches that don't apply on master. I was just about to nmbug push stale >> on this one (2/2). Is that in error? Care to verify? > > Yes, it is a

using bbdb with autocompletion

2012-04-15 Thread Tomi Ollila
On Thu, Apr 12 2012, David Belohrad wrote: > Dear All, > before I was using gnus to read my emails. This was setup together with > bbdb such, that every email address I got an email delivered got stored > into the bbdb database. > > The config was following: > > --- > ;;

[PATCH v2 0/6] batch tagging support: "notmuch tag --stdin"

2012-04-15 Thread Jani Nikula
Jameson Graef Rollins writes: > On Sat, Apr 14 2012, Jani Nikula wrote: >> This series adds support for batch tagging through stdin to "notmuch >> tag". This should be useful and efficient for e.g. initial tagging >> scripts. Also, this adds locking around a batch of tag changes, which is >> als