date_relative in notmuch-search
Patrick Totzke
patricktotzke at googlemail.com
Fri Jul 15 14:04:59 PDT 2011
Hi Jason,
Thanks for the tip with git send-mail. I didn't really intend to
get this "patch" accepted. It was more that I feel bad asking for
features all the time and as I'm not so comfortable with hacking
on the notmuch code I thought I'd ask if one could implement it like this.
Anyhow, attached you'll find a patch for the test and another for
the proposed change. Feel free to discard it :)
Best,
/p
On Fri, Jul 15, 2011 at 08:58:45AM -0700, Jameson Graef Rollins wrote:
> On Thu, 14 Jul 2011 23:22:58 +0100, Patrick Totzke <patricktotzke at googlemail.com> wrote:
> > I wodered why "notmuch search --format=json" doesn't provide the "date_relative" field for
> > results, as the show command does.
> >
> > I'm not expert on notmuch internals but I got it working like this:
>
> Hi, Patrick. I think this is a very reasonable patch. However, there
> are a couple of things you need to do to get this patch accepted.
>
> First of all, you should send the patch to the list in a format that
> will allow it to be applied directly to a git repo. In particular, it
> needs to include a good commit log message. Look at using git send-mail
> to send it to the list, which will format it correctly.
>
> Also, I'm quite sure this patch will require a couple of changes to the
> test suite. You should submit a patch that changes the test suite to
> reflect the desired changes *first*, followed by the patch that fixes
> the issue.
>
> Thanks!
>
> jamie.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-test-date_relative-in-notmuch-search-json-output.patch
Type: text/x-diff
Size: 1230 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20110715/4747ac30/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-json-date_relative-for-threads.patch
Type: text/x-diff
Size: 941 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20110715/4747ac30/attachment-0001.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20110715/4747ac30/attachment.pgp>
More information about the notmuch
mailing list