Do path: searches handle spaces correctly?

2014-09-20 Thread Keith Amidon
I'm trying to update an archiving tool that used the old folder: search
terms to use the newer folder: or path: terms.  From playing around with
it, I think that the path: term is what I want to use.  However, I am
getting some behavior I don't understand unless path: searches don't
properly handle spaces or I don't know how to quote them properly.

If I do:

notmuch search --output=files path:'dir/INBOX/**'

I get:

/dir/INBOX/cur/1411244319_3.18990.kea-tablet,U=24,FMD5=7e33429f656f1e6e9d79b29c3f82c57e:2,S
/dir/INBOX/cur/1411244319_2.18990.kea-tablet,U=23,FMD5=7e33429f656f1e6e9d79b29c3f82c57e:2,S
/dir/INBOX/INBOX/Sent 
Items/cur/1411244324_2.18990.kea-tablet,U=3,FMD5=e99e6ef6cc1489bb6d8b47a4c49bb989:2,S
/dir/INBOX/cur/1411244319_1.18990.kea-tablet,U=22,FMD5=7e33429f656f1e6e9d79b29c3f82c57e:2,RS
/dir/INBOX/cur/1411244319_0.18990.kea-tablet,U=21,FMD5=7e33429f656f1e6e9d79b29c3f82c57e:2,S
/dir/INBOX/cur/1411244318_3.18990.kea-tablet,U=20,FMD5=7e33429f656f1e6e9d79b29c3f82c57e:2,S
/dir/INBOX/cur/1411244318_2.18990.kea-tablet,U=19,FMD5=7e33429f656f1e6e9d79b29c3f82c57e:2,S
/dir/INBOX/cur/1411244318_1.18990.kea-tablet,U=18,FMD5=7e33429f656f1e6e9d79b29c3f82c57e:2,S
/dir/INBOX/INBOX/Sent 
Items/cur/1411244324_1.18990.kea-tablet,U=2,FMD5=e99e6ef6cc1489bb6d8b47a4c49bb989:2,S
/dir/INBOX/cur/1411244318_0.18990.kea-tablet,U=17,FMD5=7e33429f656f1e6e9d79b29c3f82c57e:2,S

Notice that this contains results for "dir/INBOX/INBOX/Sent Items",
which has a space in it.  If I do:

notmuch search --output=files path:'dir/INBOX/INBOX/Sent Items'

I don't get any results, but it seems like the two results above should
be shown, right?  Am I doing something wrong here?  If it looks like I'm
doing it correctly, what can I do to help troubleshoot the issue?

 Thanks, Keith


[PATCH] cli: add --output=address-{from, to, all} to notmuch search

2014-09-20 Thread Michal Sojka
On Fri, Sep 19 2014, David Bremner wrote:
> Jani Nikula  writes:
>
>> +if (name && *name)
>> +full_address = talloc_asprintf (NULL, "%s <%s>", name, addr);
>> +else
>> +full_address = talloc_asprintf (NULL, "<%s>", addr
>
> Is there some reason not to use sprinter as a talloc context here?
>
>> +
>> +if (!full_address)
>> +break;
>
> Is the error here out of memory? Maybe an error message would be a good
> idea.
>
>
> Obviously the docs need to be updated as well, and ideally the tests.  I
> think Jani was hinting that he didn't want to be the person to do
> that. Any volunteers?

Yes, I'll look at that. It will probably take me a few days.

-Michal


[Vagrant Cascadian] Bug#759302: notmuch-emacs: citation should use datestring from the replied to message

2014-09-20 Thread David Bremner

I thought I forwarded this already, but maybe not.

-- next part --
An embedded message was scrubbed...
From: Vagrant Cascadian 
Subject: Bug#759302: notmuch-emacs: citation should use datestring from the 
replied to message
Date: Mon, 25 Aug 2014 17:11:57 -0700
Size: 6240
URL: 
<http://notmuchmail.org/pipermail/notmuch/attachments/20140920/659864ab/attachment.mht>


[PATCH] test/emacs: globally force the html renderer to html2text

2014-09-20 Thread David Bremner
Previously we did this for a single test, but some other proposed
tests ( id:1398105468-14317-3-git-send-email-amdragon at mit.edu ) show
similar breakage when switching renderers.
---
 test/T310-emacs.sh | 2 +-
 test/test-lib.el   | 5 +
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/test/T310-emacs.sh b/test/T310-emacs.sh
index ac966e5..c302402 100755
--- a/test/T310-emacs.sh
+++ b/test/T310-emacs.sh
@@ -483,7 +483,7 @@ test_expect_equal_file OUTPUT EXPECTED
 test_begin_subtest "Reply within emacs to an html-only message"
 add_message '[content-type]="text/html"' \
'[body]="Hi,This is an HTML test message.OK?"'
-test_emacs "(let ((message-hidden-headers '()) (mm-text-html-renderer 
'html2text))
+test_emacs "(let ((message-hidden-headers '()))
(notmuch-show \"id:${gen_msg_id}\")
(notmuch-show-reply)
(test-output))"
diff --git a/test/test-lib.el b/test/test-lib.el
index 36afe63..bbc03cb 100644
--- a/test/test-lib.el
+++ b/test/test-lib.el
@@ -181,3 +181,8 @@ nothing."
 ;; suite
 (setq notmuch-tag-deleted-formats
   '((".*" nil)))
+
+;; force a common html renderer, to avoid test variations between
+;; environments
+
+(setq mm-text-html-renderer 'html2text)
-- 
2.1.0



[PATCH] python: Add binding for notmuch_query_add_tag_exclude

2014-09-20 Thread David Bremner
Thibaut Horel  writes:

> Implemented as the method `exclude_tag` of the class `Query`. This method 
> takes
> one argument, a string containing the name of the tag to exclude

This tag looks pretty trivial.  I'll merge it unless I hear some
objections over the weekend.

d




Re: [PATCH] cli: add --output=address-{from, to, all} to notmuch search

2014-09-20 Thread Michal Sojka
On Fri, Sep 19 2014, David Bremner wrote:
> Jani Nikula  writes:
>
>> +if (name && *name)
>> +full_address = talloc_asprintf (NULL, "%s <%s>", name, addr);
>> +else
>> +full_address = talloc_asprintf (NULL, "<%s>", addr
>
> Is there some reason not to use sprinter as a talloc context here?
>
>> +
>> +if (!full_address)
>> +break;
>
> Is the error here out of memory? Maybe an error message would be a good
> idea.
>
>
> Obviously the docs need to be updated as well, and ideally the tests.  I
> think Jani was hinting that he didn't want to be the person to do
> that. Any volunteers?

Yes, I'll look at that. It will probably take me a few days.

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


[Vagrant Cascadian] Bug#759302: notmuch-emacs: citation should use datestring from the replied to message

2014-09-20 Thread David Bremner

I thought I forwarded this already, but maybe not.

--- Begin Message ---
Package: notmuch-emacs
Version: 0.18.1-1
Severity: normal

So, I've struggled with coming up with a citation line format:

 '(message-citation-line-format "On %Y-%m-%d %T %z, %N wrote:")
 '(message-citation-line-function (quote 
message-insert-formatted-citation-line))

But it localizes the timezone of datestring, rather than simply copying
the datestring from the replied to message. Several other mail clients,
including "notmuch reply", seem to simply copy the datestring including
the timezone data, it quotes using:

  On Sat, 23 Aug 2014 12:36:53 +0600, Example User  
wrote:

But notmuch-emacs does changes the datestring to my locally configured
timezone. As far as I can tell, it can either be configured to use UTC,
or my local timezone only.

David Bremner suggests this may need to be fixed upstream.


live well,
  vagrant

-- System Information:
Debian Release: jessie/sid
  APT prefers testing
  APT policy: (500, 'testing'), (120, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386
armhf

Kernel: Linux 3.14-2-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages notmuch-emacs depends on:
ii  emacs2323.4+1-4.1+b1
ii  emacs24-nox [emacs24]  24.3+1-4+b1
ii  emacsen-common 2.0.8
ii  notmuch0.18.1-1

notmuch-emacs recommends no packages.

notmuch-emacs suggests no packages.

-- no debconf information


pgpbLqfw5WTwz.pgp
Description: PGP signature
--- End Message ---
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch