Re: [PATCH 2/9] test: set EMACS_SERVER variable only after dtach(1) was successfully started

Subject: Re: [PATCH 2/9] test: set EMACS_SERVER variable only after dtach(1) was successfully started

Date: Thu, 17 Nov 2011 01:14:13 -0800

To: Dmitry Kurochkin, notmuch@notmuchmail.org

Cc:

On Thu, 17 Nov 2011 05:56:19 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmail.com> wrote:
> Otherwise, we can set the EMACS_SERVER and return with an error.  And
> subsequent calls to test_emacs would assume that emacs server is running.

Hey, Dmitry.  Can you explain this a bit more?  I'm not sure I
understand the case this is protecting against.

jamie.

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

Previous message (by thread): Re: [PATCH 4/9] test: add support for external executable dependencies

Thread:

Dmitry Kurochkin—[PATCH 0/9] test: (hopefully) better test prerequisites [inbox, unread]
- Dmitry Kurochkin—[PATCH 1/9] test: move subtest variables reset into a dedicated function [inbox, notmuch::obsolete, notmuch::patch, unread]
- Dmitry Kurochkin—[PATCH 2/9] test: set EMACS_SERVER variable only after dtach(1) was successfully started [inbox, notmuch::obsolete, notmuch::patch, unread]
  - Jameson Graef Rollins—Re: [PATCH 2/9] test: set EMACS_SERVER variable only after dtach(1) was successfully started [inbox, notmuch::review, signed, unread]
    - Dmitry Kurochkin—Re: [PATCH 2/9] test: set EMACS_SERVER variable only after dtach(1) was successfully started [inbox, unread]
- Dmitry Kurochkin—[PATCH 3/9] test: add test state reset to test_expect_* functions that did not have it [inbox, notmuch::obsolete, notmuch::patch, unread]
- Dmitry Kurochkin—[PATCH 4/9] test: add support for external executable dependencies [inbox, notmuch::obsolete, notmuch::patch, unread]
  - Tomi Ollila—Re: [PATCH 4/9] test: add support for external executable dependencies [inbox, unread]
    - Dmitry Kurochkin—Re: [PATCH 4/9] test: add support for external executable dependencies [inbox, unread]
      - Dmitry Kurochkin—Re: [PATCH 4/9] test: add support for external executable dependencies [inbox, unread]
        Dmitry Kurochkin—Re: [PATCH 4/9] test: add support for external executable dependencies [inbox, unread]
- Dmitry Kurochkin—[PATCH 5/9] test: fix "skipping test" verbose output [inbox, notmuch::obsolete, notmuch::patch, unread]
- Dmitry Kurochkin—[PATCH 6/9] test: skip all subtests if external dependencies are missing during init [inbox, notmuch::obsolete, notmuch::patch, unread]
- Dmitry Kurochkin—[PATCH 7/9] test: declare external dependencies for the tests [inbox, notmuch::obsolete, notmuch::patch, unread]
- Dmitry Kurochkin—[PATCH 8/9] test: check if emacs is available in the beginning of test_emacs [inbox, notmuch::obsolete, notmuch::patch, unread]
  - Tomi Ollila—Re: [PATCH 8/9] test: check if emacs is available in the beginning of test_emacs [inbox, notmuch::review, unread]
    - Dmitry Kurochkin—Re: [PATCH 8/9] test: check if emacs is available in the beginning of test_emacs [inbox, unread]
      - Dmitry Kurochkin—Re: [PATCH 8/9] test: check if emacs is available in the beginning of test_emacs [inbox, unread]
- Dmitry Kurochkin—[PATCH 9/9] test: fix "Stashing in notmuch-search" test when emacs is not available [inbox, notmuch::obsolete, notmuch::patch, unread]
- Jameson Graef Rollins—Re: [PATCH 0/9] test: (hopefully) better test prerequisites [inbox, notmuch::review, signed, unread]
  - Dmitry Kurochkin—Re: [PATCH 0/9] test: (hopefully) better test prerequisites [inbox, unread]
  - Tomi Ollila—Re: [PATCH 0/9] test: (hopefully) better test prerequisites [inbox, unread]
    - Tomi Ollila—Re: [PATCH 0/9] test: (hopefully) better test prerequisites [inbox, unread]
    - Dmitry Kurochkin—Re: [PATCH 0/9] test: (hopefully) better test prerequisites [inbox, unread]
      - Tomi Ollila—Re: [PATCH 0/9] test: (hopefully) better test prerequisites [inbox, unread]
        Dmitry Kurochkin—Re: [PATCH 0/9] test: (hopefully) better test prerequisites [inbox, unread]
        Tomi Ollila—Re: [PATCH 0/9] test: (hopefully) better test prerequisites [inbox, unread]
- Thomas Jost—Re: [PATCH 0/9] test: (hopefully) better test prerequisites [inbox, notmuch::review, signed, unread]
  - Dmitry Kurochkin—Re: [PATCH 0/9] test: (hopefully) better test prerequisites [inbox, unread]