[PATCH] build: fix out-of-tree builds

2013-05-15 Thread Jani Nikula
Support for out-of-tree builds was added in commit 3e4a9d60a9419621b08c647a306843d76c47c2cb Author: Carl Worth cwo...@cworth.org Date: Wed Mar 9 15:02:42 2011 -0800 build: Add support for non-source-directory builds. and broken in commit 7beeb8c88a014ecbc53d8241f10683b3c4c16228 Author:

[PATCH 4/4] perf-test: add notmuch-memory-test

2013-05-15 Thread david
From: David Bremner brem...@debian.org Somehow this file was not added to the patch set which split the tests into time and memory tests. Take advantage of the the new way of running tests to avoid listing the explicitly. --- performance-test/notmuch-memory-test | 21 + 1

[PATCH 3/4] perf-test: run all appropriately named time tests

2013-05-15 Thread david
From: David Bremner brem...@debian.org This avoids hassle with manually adding every test to the master list. --- performance-test/notmuch-time-test | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/performance-test/notmuch-time-test

[PATCH 1/4] perf-test: rename memory tests

2013-05-15 Thread david
From: David Bremner brem...@debian.org The common suffix will help running them all automatically. --- performance-test/M00-new | 15 --- performance-test/M00-new.sh | 15 +++ performance-test/M01-dump-restore| 15 ---

[PATCH 2/4] perf-test: rename time tests to have .sh suffix

2013-05-15 Thread david
From: David Bremner brem...@debian.org This will simplify notmuch time tests --- performance-test/T00-new | 15 --- performance-test/T00-new.sh | 15 +++ performance-test/T01-dump-restore| 13 - performance-test/T01-dump-restore.sh |

Re: simplify and fix performance tests

2013-05-15 Thread Tomi Ollila
On Wed, May 15 2013, da...@tethera.net wrote: Jani noticed the alleged file notmuch-memory-test was missing. Austin grumbled about explicitly adding tests to a list. This series should give them both something else to grumble about. This test series looks good to me. It is consistent what was

Re: [PATCH] build: fix out-of-tree builds

2013-05-15 Thread David Bremner
Jani Nikula j...@nikula.org writes: # List all subdirectories here. Each contains its own Makefile.local -subdirs := compat completion emacs lib man parse-time-string -subdirs += performance-test util test +subdirs = compat completion emacs lib man parse-time-string performance-test util

Re: [PATCH] build: fix out-of-tree builds

2013-05-15 Thread Jani Nikula
On Wed, 15 May 2013, David Bremner da...@tethera.net wrote: Jani Nikula j...@nikula.org writes: # List all subdirectories here. Each contains its own Makefile.local -subdirs := compat completion emacs lib man parse-time-string -subdirs += performance-test util test +subdirs = compat

Re: [PATCH] emacs: tag: fix compile warning

2013-05-15 Thread Tomi Ollila
On Tue, May 14 2013, Mark Walters markwalters1...@gmail.com wrote: When compiling notmuch-tag.el there is a compile warning: notmuch-tag.el:27:1:Warning: cl package required at runtime Since we have decided to allow runtime use of cl we suppress this warning by adding a tail comment to the

Re: simplify and fix performance tests

2013-05-15 Thread Austin Clements
LGTM. Quoth da...@tethera.net on May 15 at 8:02 am: Jani noticed the alleged file notmuch-memory-test was missing. Austin grumbled about explicitly adding tests to a list. This series should give them both something else to grumble about. ___

Re: simplify and fix performance tests

2013-05-15 Thread David Bremner
da...@tethera.net writes: Jani noticed the alleged file notmuch-memory-test was missing. Austin grumbled about explicitly adding tests to a list. This series should give them both something else to grumble about. Pushed, d ___ notmuch mailing list

Re: [PATCH] emacs: tag: fix compile warning

2013-05-15 Thread David Bremner
Mark Walters markwalters1...@gmail.com writes: When compiling notmuch-tag.el there is a compile warning: notmuch-tag.el:27:1:Warning: cl package required at runtime pushed the patch. Someone (TM) should investigate the new cl- prefixed functions, e.g. cl-loop which maybe me less hassle in

[PATCH] build: fix out-of-tree builds

2013-05-15 Thread Jani Nikula
Support for out-of-tree builds was added in commit 3e4a9d60a9419621b08c647a306843d76c47c2cb Author: Carl Worth Date: Wed Mar 9 15:02:42 2011 -0800 build: Add support for non-source-directory builds. and broken in commit 7beeb8c88a014ecbc53d8241f10683b3c4c16228 Author: David Bremner Date:

simplify and fix performance tests

2013-05-15 Thread da...@tethera.net
Jani noticed the alleged file notmuch-memory-test was missing. Austin grumbled about explicitly adding tests to a list. This series should give them both something else to grumble about.

[PATCH 4/4] perf-test: add notmuch-memory-test

2013-05-15 Thread da...@tethera.net
From: David Bremner Somehow this file was not added to the patch set which split the tests into time and memory tests. Take advantage of the the "new" way of running tests to avoid listing the explicitly. --- performance-test/notmuch-memory-test | 21 + 1

[PATCH 3/4] perf-test: run all appropriately named time tests

2013-05-15 Thread da...@tethera.net
From: David Bremner This avoids hassle with manually adding every test to the master list. --- performance-test/notmuch-time-test | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/performance-test/notmuch-time-test

[PATCH 1/4] perf-test: rename memory tests

2013-05-15 Thread da...@tethera.net
From: David Bremner The common suffix will help running them all automatically. --- performance-test/M00-new | 15 --- performance-test/M00-new.sh | 15 +++ performance-test/M01-dump-restore| 15 ---

[PATCH 2/4] perf-test: rename time tests to have .sh suffix

2013-05-15 Thread da...@tethera.net
From: David Bremner This will simplify notmuch time tests --- performance-test/T00-new | 15 --- performance-test/T00-new.sh | 15 +++ performance-test/T01-dump-restore| 13 - performance-test/T01-dump-restore.sh |

simplify and fix performance tests

2013-05-15 Thread Tomi Ollila
On Wed, May 15 2013, david at tethera.net wrote: > Jani noticed the alleged file notmuch-memory-test was missing. > Austin grumbled about explicitly adding tests to a list. > This series should give them both something else to grumble about. This test series looks good to me. It is consistent

[PATCH] build: fix out-of-tree builds

2013-05-15 Thread David Bremner
Jani Nikula writes: > # List all subdirectories here. Each contains its own Makefile.local > -subdirs := compat completion emacs lib man parse-time-string > -subdirs += performance-test util test > +subdirs = compat completion emacs lib man parse-time-string performance-test > util test Is

[PATCH] emacs: tag: fix compile warning

2013-05-15 Thread Tomi Ollila
On Tue, May 14 2013, Mark Walters wrote: > When compiling notmuch-tag.el there is a compile warning: > notmuch-tag.el:27:1:Warning: cl package required at runtime > > Since we have decided to allow runtime use of cl we suppress this > warning by adding a tail comment to the file. > --- LGTM.

simplify and fix performance tests

2013-05-15 Thread Austin Clements
LGTM. Quoth david at tethera.net on May 15 at 8:02 am: > Jani noticed the alleged file notmuch-memory-test was missing. > Austin grumbled about explicitly adding tests to a list. > This series should give them both something else to grumble about.