Re: [PATCH 1/3] test: Don't return the result of checking for running emacs to the tester.

Subject: Re: [PATCH 1/3] test: Don't return the result of checking for running emacs to the tester.

Date: Tue, 17 Jan 2012 14:37:52 +0000

To: Dmitry Kurochkin, notmuch@notmuchmail.org

Cc:

(And for the list...)

On Tue, 17 Jan 2012 18:20:04 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:
> Can you please elaborate why this is needed?

This code:

		    # wait until the emacs server is up
		    until test_emacs '()' 2>/dev/null; do
			    sleep 1
		    done

outputs 'nil', so the first caller to test_emacs has 'nil\n' prepended
to their expected output.

part-000.sig (application/pgp-signature)

Previous message (by thread): Re: [PATCH 0/4 v43] emacs test helpers

Thread:

David Edmondson—emacs based tests, version 3 [inbox, unread]
- David Edmondson—[PATCH 1/4] test: Add `test_emacs_expect_t'. [inbox, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—Re: [PATCH 1/4] test: Add `test_emacs_expect_t'. [inbox, unread]
    - David Edmondson—Re: [PATCH 1/4] test: Add `test_emacs_expect_t'. [inbox, signed, unread]
    - David Edmondson—[PATCH 1/3] test: Don't return the result of checking for running emacs to the tester. [inbox, notmuch::obsolete, notmuch::patch, unread]
      - David Edmondson—[PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, notmuch::obsolete, notmuch::patch, unread]
        Dmitry Kurochkin—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, notmuch::review, unread]
        David Edmondson—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, signed, unread]
        Dmitry Kurochkin—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, unread]
        David Edmondson—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, signed, unread]
        Tomi Ollila—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, unread]
        Tomi Ollila—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, unread]
        David Edmondson—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, signed, unread]
        Tomi Ollila—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, unread]
        David Edmondson—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, signed, unread]
        Tomi Ollila—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, unread]
        David Edmondson—[PATCH 1/3] test: Don't return the result of checking for running emacs to the tester. [inbox, unread]
        David Edmondson—[PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, unread]
        David Edmondson—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, signed, unread]
        Dmitry Kurochkin—Re: [PATCH 2/3] test: Add `test_emacs_expect_t'. [inbox, notmuch::review, unread]
        David Edmondson—[PATCH 3/3] test: Add address cleaning tests. [inbox, unread]
        Dmitry Kurochkin—Re: [PATCH 3/3] test: Add address cleaning tests. [inbox, notmuch::review, unread]
        Dmitry Kurochkin—Re: [PATCH 1/3] test: Don't return the result of checking for running emacs to the tester. [inbox, notmuch::review, unread]
      - David Edmondson—[PATCH 3/3] test: Add address cleaning tests. [inbox, notmuch::obsolete, notmuch::patch, unread]
      - Dmitry Kurochkin—Re: [PATCH 1/3] test: Don't return the result of checking for running emacs to the tester. [inbox, unread]
        David Edmondson—Re: [PATCH 1/3] test: Don't return the result of checking for running emacs to the tester. [inbox, signed, unread]
        Dmitry Kurochkin—Re: [PATCH 1/3] test: Don't return the result of checking for running emacs to the tester. [inbox, unread]
  - David Edmondson—[PATCH 1/4 v42] test: Don't return the result of checking for running emacs to the tester. [inbox, notmuch::obsolete, notmuch::patch, notmuch::trivial, unread]
    - David Edmondson—[PATCH 2/4 v42] test: Add `test_emacs_expect_t'. [inbox, notmuch::obsolete, notmuch::patch, unread]
      - Dmitry Kurochkin—Re: [PATCH 2/4 v42] test: Add `test_emacs_expect_t'. [inbox, unread]
    - David Edmondson—[PATCH 3/4 v42] test: Add more helpers for emacs tests. [inbox, notmuch::obsolete, notmuch::patch, unread]
      - Dmitry Kurochkin—Re: [PATCH 3/4 v42] test: Add more helpers for emacs tests. [inbox, unread]
        David Edmondson—Re: [PATCH 3/4 v42] test: Add more helpers for emacs tests. [inbox, signed, unread]
    - David Edmondson—[PATCH 4/4 v42] test: Add address cleaning tests. [inbox, notmuch::obsolete, notmuch::patch, unread]
      - Dmitry Kurochkin—Re: [PATCH 4/4 v42] test: Add address cleaning tests. [inbox, unread]
    - Dmitry Kurochkin—Re: [PATCH 1/4 v42] test: Don't return the result of checking for running emacs to the tester. [inbox, unread]
  - David Edmondson—[PATCH 0/4 v43] emacs test helpers [inbox, unread]
    - David Edmondson—[PATCH 1/4 v43] test: Don't return the result of checking for running emacs to the tester. [inbox, notmuch::patch, notmuch::pushed, unread]
    - David Edmondson—[PATCH 2/4 v43] test: Add `test_emacs_expect_t'. [inbox, notmuch::patch, notmuch::pushed, unread]
    - David Edmondson—[PATCH 3/4 v43] test: Add more helpers for emacs tests. [inbox, notmuch::patch, notmuch::pushed, unread]
    - David Edmondson—[PATCH 4/4 v43] test: Add address cleaning tests. [inbox, notmuch::patch, notmuch::pushed, unread]
    - Dmitry Kurochkin—Re: [PATCH 0/4 v43] emacs test helpers [inbox, unread]
    - Tomi Ollila—Re: [PATCH 0/4 v43] emacs test helpers [inbox, unread]
    - David Bremner—Re: [PATCH 0/4 v43] emacs test helpers [inbox, unread]
- David Edmondson—[PATCH 2/4] test: Add address cleaning tests. [inbox, notmuch::obsolete, notmuch::patch, unread]
  - Dmitry Kurochkin—Re: [PATCH 2/4] test: Add address cleaning tests. [inbox, unread]
    - David Edmondson—Re: [PATCH 2/4] test: Add address cleaning tests. [inbox, signed, unread]
- David Edmondson—[PATCH 3/4] emacs: Avoid `mail-header-parse-address' in `notmuch-show-clean-address'. [inbox, notmuch::obsolete, notmuch::patch, unread]
- David Edmondson—[PATCH 4/4] emacs: Another special case for `notmuch-show-clean-address'. [inbox, notmuch::obsolete, notmuch::patch, unread]