[PATCH v2] emacs: add function to toggle display of all multipart/alternative parts

2012-06-22 Thread David Bremner
Jani Nikula  writes:

>
> A small wrinkle is that in order to make the variable survive
> notmuch-show-refresh-view (which is required for expanding/collapsing
> the parts, but ends up calling kill-all-local-variables through
> notmuch-show-mode) it is necessary to give it the permanent-local
> property.

The code looks simple enough; should we apply this patch while we wait
for something fancier? I don't really know how to evaluate the
permanent-local bit.

d


adding emacs-snapshot as alternative to emacs

2012-06-22 Thread Dmitrijs Ledkovs
On 22/06/12 11:23, David Bremner wrote:
> Rainer M Krug  writes:
> 
>> OK - I accept the reasoning behind not adding emacs-snapshot as an
>> alternative to the defaults.  But as I guess as quite a few are using
>> emacs-snapshot and notmuch, it would make sense to just add a patch to
>> the source which one could execute to add emacs-snapshot to the list
>> of alternative dependencies.
> 

Similar request was declined by Debian maintainer as well.

As emacs-snapshot is a moving target, w.r.t. features, maybe we should
ask emacs-snaphost packaging to add

Provides: emacs22, emacs23, emacs24

And update those as necessary? Then no changes to all the external
package would be needed.

-- 
Regards,
Dmitrijs.


[PATCH] emacs: add pipe attachment command

2012-06-22 Thread David Bremner
Jameson Graef Rollins  writes:

> From: Mark Walters 
>
> Allow the user to pipe the attachment somewhere. Bound to '|' on the
> attachment button.
>

pushed

d


adding emacs-snapshot as alternative to emacs

2012-06-22 Thread David Bremner
Rainer M Krug  writes:

> OK - I accept the reasoning behind not adding emacs-snapshot as an
> alternative to the defaults.  But as I guess as quite a few are using
> emacs-snapshot and notmuch, it would make sense to just add a patch to
> the source which one could execute to add emacs-snapshot to the list
> of alternative dependencies.

It seems to me the wiki is the appropriate place for such information.

   See http://notmuchmail.org/wikiwriteaccess/ 

for more information.

David

-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 315 bytes
Desc: not available
URL: 
<http://notmuchmail.org/pipermail/notmuch/attachments/20120622/1762a659/attachment.pgp>


Re: adding emacs-snapshot as alternative to emacs

2012-06-22 Thread Rainer M Krug
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 21/06/12 20:47, David Bremner wrote:
 Rainer M Krug r.m.k...@gmail.com writes:
 
 
 Here's what I'm using to get it to build right.
 
 Thanks - installs nicely.
 
 Would it be possible to include this in the source?
 
 Rainer
 
 See
 
 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=677900
 
 And yes, I'm the grinch.

OK - I accept the reasoning behind not adding emacs-snapshot as an alternative 
to the defaults.
But as I guess as quite a few are using emacs-snapshot and notmuch, it would 
make sense to just
add a patch to the source which one could execute to add emacs-snapshot to the 
list of alternative
dependencies. This would make the process more transparent and possibly easier 
to debug if errors
occur.

Cheers,

Rainer

 
 d
 
 
 


-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAk/kG1IACgkQoYgNqgF2egrbOACggXJZxR7kQoQ+LxWZp/0cJLWB
e2cAn3qhB3bHSZQvHvBgChUemSfMUeUH
=r4H/
-END PGP SIGNATURE-

___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch


Re: adding emacs-snapshot as alternative to emacs

2012-06-22 Thread David Bremner
Rainer M Krug r.m.k...@gmail.com writes:

 OK - I accept the reasoning behind not adding emacs-snapshot as an
 alternative to the defaults.  But as I guess as quite a few are using
 emacs-snapshot and notmuch, it would make sense to just add a patch to
 the source which one could execute to add emacs-snapshot to the list
 of alternative dependencies.

It seems to me the wiki is the appropriate place for such information.

   See http://notmuchmail.org/wikiwriteaccess/ 

for more information.

David



pgp7RNL5v3bMF.pgp
Description: PGP signature
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch


Re: adding emacs-snapshot as alternative to emacs

2012-06-22 Thread Dmitrijs Ledkovs
On 22/06/12 11:23, David Bremner wrote:
 Rainer M Krug r.m.k...@gmail.com writes:
 
 OK - I accept the reasoning behind not adding emacs-snapshot as an
 alternative to the defaults.  But as I guess as quite a few are using
 emacs-snapshot and notmuch, it would make sense to just add a patch to
 the source which one could execute to add emacs-snapshot to the list
 of alternative dependencies.
 

Similar request was declined by Debian maintainer as well.

As emacs-snapshot is a moving target, w.r.t. features, maybe we should
ask emacs-snaphost packaging to add

Provides: emacs22, emacs23, emacs24

And update those as necessary? Then no changes to all the external
package would be needed.

-- 
Regards,
Dmitrijs.
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch


Re: [PATCHv2] emacs: derive correct timestamp in FCC unique name

2012-06-22 Thread David Bremner
Jesse Rosenthal jrosent...@jhu.edu writes:


 This change is mostly a question of consistency, since the unique name
 is arbitrary anyway. But since most people use timestamps, and that was
 the original intention here as well, we might as well.

pushed.

d
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch


Re: [PATCH] emacs: add pipe attachment command

2012-06-22 Thread David Bremner
Jameson Graef Rollins jroll...@finestructure.net writes:

 From: Mark Walters markwalters1...@gmail.com

 Allow the user to pipe the attachment somewhere. Bound to '|' on the
 attachment button.


pushed

d
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch


Re: [PATCH v2] emacs: add function to toggle display of all multipart/alternative parts

2012-06-22 Thread David Bremner
Jani Nikula j...@nikula.org writes:


 A small wrinkle is that in order to make the variable survive
 notmuch-show-refresh-view (which is required for expanding/collapsing
 the parts, but ends up calling kill-all-local-variables through
 notmuch-show-mode) it is necessary to give it the permanent-local
 property.

The code looks simple enough; should we apply this patch while we wait
for something fancier? I don't really know how to evaluate the
permanent-local bit.

d
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch


Re: [PATCH 4/8] man: clarify search --exclude documentation

2012-06-22 Thread Peter Wang
On Wed, 20 Jun 2012 21:08:05 +0100, Mark Walters markwalters1...@gmail.com 
wrote:
 
 I have reviewed all the new parts of this series (judged as being
 patches 3-8) and the changes made to my two patches and they are all
 fine (with one small comment below). Patch 1/8 does need a proper review
 though as it ended up more intrusive than I would have liked.
 
  +Messages matching search.tag_exclude are called excluded messages.
 
 My one comment is that this is not quite true if the corresponding tag
 is in the query. Since you are defining the term it would be nice to
 mention that, but I can't see a clean wording.

How about:

A message is called excluded if it matches at least one tag
in search.tag_exclude that does not appear explicitly in the search
terms.

Or less densely:

Let excluded tags be the set of tags listed in search.tag_exclude,
minus any tags which appear explicitly in the search terms.
A message is an excluded message if it matches one or more
excluded tags.

Peter
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch