[PATCH] test: fix compact backup / restore test

2013-11-03 Thread Jani Nikula
On Sat, 02 Nov 2013, David Bremner wrote: > It was looking in completely the wrong place for the backup and the > (test) xapian database. Unfortunately test_begin_subtest hides the > relevant errors. I included this, unchanged, in my series [1] that depends on it. It makes no difference which one

Re: [PATCH] test: fix compact backup / restore test

2013-11-03 Thread Jani Nikula
On Sat, 02 Nov 2013, David Bremner wrote: > It was looking in completely the wrong place for the backup and the > (test) xapian database. Unfortunately test_begin_subtest hides the > relevant errors. I included this, unchanged, in my series [1] that depends on it. It makes no difference which one

[PATCH] test: fix compact backup / restore test

2013-11-02 Thread Tomi Ollila
On Sat, Nov 02 2013, David Bremner wrote: > It was looking in completely the wrong place for the backup and the > (test) xapian database. Unfortunately test_begin_subtest hides the > relevant errors. > --- LGTM. It took me a while noticing that also ${TEST_TMPDIR} was changed to ${MAIL_DIR} Tom

[PATCH] test: fix compact backup / restore test

2013-11-02 Thread David Bremner
Tomi Ollila writes: > On Sat, Nov 02 2013, David Bremner wrote: > >> It was looking in completely the wrong place for the backup and the >> (test) xapian database. Unfortunately test_begin_subtest hides the >> relevant errors. >> --- > > LGTM. It took me a while noticing that also ${TEST_TMPDIR}

Re: [PATCH] test: fix compact backup / restore test

2013-11-02 Thread David Bremner
Tomi Ollila writes: > On Sat, Nov 02 2013, David Bremner wrote: > >> It was looking in completely the wrong place for the backup and the >> (test) xapian database. Unfortunately test_begin_subtest hides the >> relevant errors. >> --- > > LGTM. It took me a while noticing that also ${TEST_TMPDIR}

Re: [PATCH] test: fix compact backup / restore test

2013-11-02 Thread Tomi Ollila
On Sat, Nov 02 2013, David Bremner wrote: > It was looking in completely the wrong place for the backup and the > (test) xapian database. Unfortunately test_begin_subtest hides the > relevant errors. > --- LGTM. It took me a while noticing that also ${TEST_TMPDIR} was changed to ${MAIL_DIR} Tom

[PATCH] test: fix compact backup / restore test

2013-11-01 Thread David Bremner
It was looking in completely the wrong place for the backup and the (test) xapian database. Unfortunately test_begin_subtest hides the relevant errors. --- I found this bug because id:9ee3f2334a117b0a1b88650f44432423cbe95fd7.1383315568.git.jani at nikula.org did _not_ break any tests. Which w

[PATCH] test: fix compact backup / restore test

2013-11-01 Thread David Bremner
It was looking in completely the wrong place for the backup and the (test) xapian database. Unfortunately test_begin_subtest hides the relevant errors. --- I found this bug because id:9ee3f2334a117b0a1b88650f44432423cbe95fd7.1383315568.git.j...@nikula.org did _not_ break any tests. Which was