[kaffe] Re: Kaffe 1.1.4 on GNU/Linux PowerPC: 2 of 144 tests failed

[Tony Wyatt]

Hi Dalibor,

>>  The first failed test is TestSerialVersions. The behaviour is exactly 
>> the same I obtain compiling to native code with GCJ or using the GCJ JVM 
>> gij (either from bytecode generated by gcj or by the Kaffe compiler, 
>> nothing seems to change).
> 
On PPC Debian Linux, TestSerialVersions is the only test that fails (sob:
I'd love to see the "All tests passed" banner).

However, I get a number of Null Pointer, IO and other exceptions.

This is the 1.1.4 distro downloaded this morning (midnight GMT 2004-03-01).

> This was a bug in our and GNU Classpath Serialization code. It was fixed 
> after 1.1.4 was released by Guilhem.
> 
Has a patch been released?

(OT) I have made a slight change to test/regression/Makefile.in to make the
output a little more user-friendly.

The original version prints (while testing):
PASS/FAIL: xxxx.java

As a result of this patch, it prints:
xxxx.java . . . PASS/FAIL (3/4)

where 3 & 4 are the passes and total so far.

The main advantages of this change are that if the test falls over, you have a
record of which file it was. Before, the name was not printed out until the
test was completed. Also, you can see in a screenful of PASS-PASS-PASS, just
how far it has got in the test and how many failures you have had.

cheers
-------------- next part --------------
--- Makefile.in.old	Tue Mar  2 14:57:57 2004
+++ Makefile.in	Tue Mar  2 15:02:01 2004
@@ -737,7 +737,7 @@
 	-rm -f TAGS ID GTAGS GRTAGS GSYMS GPATH tags
 
 check-TESTS: $(TESTS)
-	@failed=0; all=0; xfail=0; xpass=0; skip=0; \
+	@failed=0; all=0; xfail=0; pass=0; xpass=0; skip=0; \
 	srcdir=$(srcdir); export srcdir; \
 	list='$(TESTS)'; \
 	if test -n "$$list"; then \
@@ -745,16 +745,18 @@
 	    if test -f ./$$tst; then dir=./; \
 	    elif test -f $$tst; then dir=; \
 	    else dir="$(srcdir)/"; fi; \
+	    echo -n "$$tst . . . ";	\
 	    if $(TESTS_ENVIRONMENT) $${dir}$$tst; then \
 	      all=`expr $$all + 1`; \
 	      case " $(XFAIL_TESTS) " in \
 	      *" $$tst "*) \
 		xpass=`expr $$xpass + 1`; \
 		failed=`expr $$failed + 1`; \
-		echo "XPASS: $$tst"; \
+		echo -n "XPASS"; \
 	      ;; \
 	      *) \
-		echo "PASS: $$tst"; \
+	      	pass=`expr $$pass + 1`;	\
+		echo -n "PASS"; \
 	      ;; \
 	      esac; \
 	    elif test $$? -ne 77; then \
@@ -762,17 +764,18 @@
 	      case " $(XFAIL_TESTS) " in \
 	      *" $$tst "*) \
 		xfail=`expr $$xfail + 1`; \
-		echo "XFAIL: $$tst"; \
+		echo -n "XFAIL"; \
 	      ;; \
 	      *) \
 		failed=`expr $$failed + 1`; \
-		echo "FAIL: $$tst"; \
+		echo -n "FAIL"; \
 	      ;; \
 	      esac; \
 	    else \
 	      skip=`expr $$skip + 1`; \
-	      echo "SKIP: $$tst"; \
+	      echo -n "SKIP"; \
 	    fi; \
+	    echo " ($$pass/$$all)";	\
 	  done; \
 	  if test "$$failed" -eq 0; then \
 	    if test "$$xfail" -eq 0; then \