Re: [PATCH] Only build tests when unit tests are enabled.

2011-03-23 Thread Gaetan Nadon
On Wed, 2011-03-23 at 09:14 +1000, Peter Hutterer wrote: On Tue, Mar 22, 2011 at 11:26:24AM -0400, Gaetan Nadon wrote: On Tue, 2011-03-22 at 11:56 +1000, Peter Hutterer wrote: Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- I'll squash this in with the other patch, no

Re: [PATCH] Only build tests when unit tests are enabled.

2011-03-22 Thread Dan Nicholson
On Mon, Mar 21, 2011 at 6:56 PM, Peter Hutterer peter.hutte...@who-t.net wrote: Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- I'll squash this in with the other patch, no need to have two separate ones.  test/Makefile.am |    2 +-  1 files changed, 1 insertions(+), 1

Re: [PATCH] Only build tests when unit tests are enabled.

2011-03-22 Thread Gaetan Nadon
On Tue, 2011-03-22 at 05:39 -0700, Dan Nicholson wrote: Not that it matters too much, but you just need to wrap the *_PROGRAMS declaration to get automake to not output the toplevel rules. I do notice here the TESTS = $(noinst_PROGRAMS) line. If check_PROGRAMS is used instead, then the

Re: [PATCH] Only build tests when unit tests are enabled.

2011-03-22 Thread Peter Hutterer
On Tue, Mar 22, 2011 at 11:26:24AM -0400, Gaetan Nadon wrote: On Tue, 2011-03-22 at 11:56 +1000, Peter Hutterer wrote: Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- I'll squash this in with the other patch, no need to have two separate ones. test/Makefile.am |2 +-

[PATCH] Only build tests when unit tests are enabled.

2011-03-21 Thread Peter Hutterer
Signed-off-by: Peter Hutterer peter.hutte...@who-t.net --- I'll squash this in with the other patch, no need to have two separate ones. test/Makefile.am |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/test/Makefile.am b/test/Makefile.am index 16502ee..0b45a2d 100644 ---