Re: Inconsistent output from "notmuch search --output=<foo>"

Subject: Re: Inconsistent output from "notmuch search --output=<foo>"

Date: Wed, 24 Nov 2010 10:25:53 +0100

To: Carl Worth, notmuch@notmuchmail.org

Cc:

From: Michal Sojka

On Wed, 24 Nov 2010, Carl Worth wrote:
> Michal, in trying to add a test for that bug, I found that the current
> test suite infrastructure can't catch it because the shell's $()
> construct doesn't distinguish whether that final newline is present or
> not. I don't see an easy way to fix this, (other than making all tests
> put results into files and making test_expect_equal accept those
> filenames). Do you see any easy fix?

Unfortunately no. I did the same in
id:"1289384870-15199-1-git-send-email-sojkam1@fel.cvut.cz"

In fact I find much easier writing
  cmd > output
  cmd2 >> output
than
  output=$(cmd)
  output+=$(cmd)

-Michal

• Previous message (by thread): Re: [PATCH] add `tag:' prefix to `--output=tags' results

Thread:

• Carl Worth—Inconsistent output from "notmuch search --output=<foo>" [inbox, unread]
  • Jameson Rollins—Re: Inconsistent output from "notmuch search --output=<foo>" [inbox, signed, unread]
  • Sebastian Spaeth—Re: Inconsistent output from "notmuch search --output=<foo>" [inbox, signed, unread]
    • David Edmondson—Re: Inconsistent output from "notmuch search --output=<foo>" [inbox, signed, unread]
    • Carl Worth—Re: Inconsistent output from "notmuch search --output=<foo>" [inbox, signed, unread]
      • Jameson Rollins—Re: Inconsistent output from "notmuch search --output=<foo>" [inbox, signed, unread]
  • Michal Sojka—Re: Inconsistent output from "notmuch search --output=<foo>" [inbox, unread]
  • Sebastian Spaeth—Re: Inconsistent output from "notmuch search --output=<foo>" [inbox, signed, unread]
    • Pieter Praet—[PATCH] remove prefixes from `--output={threads,messages}' results [inbox, notmuch::obsolete, notmuch::patch, unread]
      • Carl Worth—Re: [PATCH] remove prefixes from `--output={threads, messages}' results [inbox, signed, unread]
        • Pieter Praet—Re: [PATCH] remove prefixes from `--output={threads, messages}' results [inbox, unread]
    • Pieter Praet—[PATCH] add `tag:' prefix to `--output=tags' results [inbox, notmuch::obsolete, notmuch::patch, unread]
      • Pieter Praet—[PATCH] fix breakage in `notmuch-hello-generate-tag-alist' due to `tag:' prefix [inbox, notmuch::obsolete, notmuch::patch, unread]
      • Pieter Praet—[PATCH] fix breakage in `notmuch-select-tag-with-completion' due to `tag:' prefix [inbox, notmuch::obsolete, notmuch::patch, unread]
      • David Bremner—Re: [PATCH] add `tag:' prefix to `--output=tags' results [inbox, unread]
        • Jani Nikula—Re: [PATCH] add `tag:' prefix to `--output=tags' results [inbox, unread]
          • Pieter Praet—Re: [PATCH] add `tag:' prefix to `--output=tags' results [inbox, unread]
        • Jameson Graef Rollins—Re: [PATCH] add `tag:' prefix to `--output=tags' results [inbox, signed, unread]
          • Dmitry Kurochkin—Re: [PATCH] add `tag:' prefix to `--output=tags' results [inbox, unread]
          • Pieter Praet—Re: [PATCH] add `tag:' prefix to `--output=tags' results [inbox, unread]
      • David Edmondson—Re: [PATCH] add `tag:' prefix to `--output=tags' results [inbox, signed, unread]
        • Pieter Praet—Re: [PATCH] add `tag:' prefix to `--output=tags' results [inbox, unread]
    • Pieter Praet—[PATCH] emacs: add keybind and function to stash Message-ID without prefix [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, unread]
      • Pieter Praet—[PATCH] test: stashing in notmuch-{show,search} [inbox, notmuch::patch, notmuch::pushed, notmuch::test, unread]
        • David Bremner—Re: [PATCH] test: stashing in notmuch-{show,search} [inbox, unread]
          • Pieter Praet—[PATCH] test: emacs: tidy up "Stashing in notmuch-show" test [inbox, notmuch::patch, notmuch::pushed, notmuch::test, unread]
            • David Bremner—Re: [PATCH] test: emacs: tidy up "Stashing in notmuch-show" test [inbox, unread]