Fix 'make distcheck' problems with new unittest code, and a small

jimw@mysql.com · jimw@mysql.com · commit 0ed6517ab89e · 2006-04-13T13:01:57.000-07:00
portability problem.
diff --git a/.bzrignore b/.bzrignore
@@ -1760,3 +1760,7 @@ zlib/*.ds?
 zlib/*.vcproj
 mysql-test/r/*.log
 client/mysql_upgrade
+unittest/examples/*.t
+unittest/mysys/*.t
+unittest/mytap/t/*.t
+unittest/unit
diff --git a/unittest/Makefile.am b/unittest/Makefile.am
@@ -2,6 +2,8 @@ SUBDIRS      = mytap . mysys examples
 
 noinst_SCRIPTS = unit
 
+DISTCLEANFILES = unit
+
 unittests = mysys examples
 
 .PHONY: all mytap mysys examples test
@@ -18,6 +20,6 @@ mysys:
 examples:
 	cd examples && $(MAKE)
 
-unit: unit.pl
+unit: $(srcdir)/unit.pl
 	cp $< $@
 	chmod +x $@
diff --git a/unittest/examples/Makefile.am b/unittest/examples/Makefile.am
@@ -1,5 +1,5 @@
 AM_CPPFLAGS        = -I$(srcdir) -I$(top_builddir)/include
-AM_CPPFLAGS       += -I$(top_builddir)/unittest/mytap
+AM_CPPFLAGS       += -I$(top_srcdir)/unittest/mytap
 
 AM_LDFLAGS         = -L$(top_builddir)/unittest/mytap
 
diff --git a/unittest/mysys/Makefile.am b/unittest/mysys/Makefile.am
@@ -1,6 +1,6 @@
 
 AM_CPPFLAGS      = @ZLIB_INCLUDES@ -I$(top_builddir)/include 
-AM_CPPFLAGS     += -I$(top_srcdir)/include -I$(top_builddir)/unittest/mytap
+AM_CPPFLAGS     += -I$(top_srcdir)/include -I$(top_srcdir)/unittest/mytap
 
 AM_LDFLAGS       = -L$(top_builddir)/unittest/mytap -L$(top_builddir)/mysys
 AM_LDFLAGS      += -L$(top_builddir)/strings -L$(top_builddir)/dbug
diff --git a/unittest/mytap/t/Makefile.am b/unittest/mytap/t/Makefile.am
@@ -2,7 +2,7 @@
 AM_CPPFLAGS     = -I$(srcdir) -I$(top_builddir)/include
 AM_CPPFLAGS    += -I$(srcdir)/..
 
-AM_LDFLAGS      = -L$(srcdir)/..
+AM_LDFLAGS      = -L$(top_builddir)/unittest/mytap
 
 AM_CFLAGS       = -Wall -ansi -pedantic
 
diff --git a/unittest/mytap/tap.c b/unittest/mytap/tap.c
@@ -138,11 +138,12 @@ skip_all(char const *reason, ...)
 void
 ok(int const pass, char const *fmt, ...)
 {
+  va_list ap;
+  va_start(ap, fmt);
+
   if (!pass && *g_test.todo == '\0')
     ++g_test.failed;
 
-  va_list ap;
-  va_start(ap, fmt);
   emit_tap(pass, fmt, ap);
   va_end(ap);
   if (*g_test.todo != '\0')
