[PATCH 2/3] test: Add `test_emacs_expect_t'.

David Edmondson dme at dme.org
Tue Jan 17 07:09:22 PST 2012


On Tue, 17 Jan 2012 18:49:36 +0400, Dmitry Kurochkin <dmitry.kurochkin at gmail.com> wrote:
> > +		# We cannot call 'test_emacs' in a subshell, because
> > +		# the setting of EMACS_SERVER would not persist
> > +		# throughout a sequence of tests, so we use a
> > +		# temporary file.
> > +		tmp="$TMPDIR"; if [ -z "$tmp" ]; then tmp=/tmp; fi
> > +		output="$tmp/test_emacs_output.$$"
> > +		test_emacs "$1" > "${output}"
> > +		result=$(cat "${output}")
> > +		rm -f "${output}"
> 
> I wonder if there is any bash trick which can help here?

I'm not aware of one, but I'm not a bash expert.

> Another option is to start emacs server before using test_emacs in
> subshell.  See emacs-subject-to-filename for an example.  I think this
> is a better option than using a temporary file.

I think that's a very poor option. Forcing knowledge the breakage into
all of the users may make applying any future fix more difficult.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120117/3df8fb09/attachment.pgp>


More information about the notmuch mailing list