Patch: FYI: PR 5874

Tom Tromey tromey@redhat.com
Sun Mar 17 14:21:00 GMT 2002


I'm checking this in to the trunk and the branch.

This fixes a libgcj test suite buglet.  Tested on x86 using the
example in the PR.

Tom

Index: ChangeLog
from  Tom Tromey  <tromey@redhat.com>

	* libjava.jni/jni.exp (gcj_jni_build_header): Use local_exec, not
	target_compile.  Fixes PR other/5874.

Index: libjava.jni/jni.exp
===================================================================
RCS file: /cvs/gcc/gcc/libjava/testsuite/libjava.jni/jni.exp,v
retrieving revision 1.4
diff -u -r1.4 jni.exp
--- libjava.jni/jni.exp 2002/01/05 22:26:58 1.4
+++ libjava.jni/jni.exp 2002/03/17 20:33:08
@@ -30,11 +30,10 @@
 proc gcj_jni_build_header {file} {
   set gcjh [find_gcjh]
   set file [file rootname $file]
-  set options [list "compiler=$gcjh" \
-		 "additional_flags=-jni"]
-  set x [prune_warnings [target_compile $file "" none $options]]
+  set x [string trim [prune_warnings \
+			[lindex [local_exec "$gcjh -jni $file" "" "" 300] 1]]]
   if {$x != ""} {
-    verbose "target_compile failed: $x" 2
+    verbose "local_exec failed: $x" 2
     fail "$file header generation"
     return 0
   }



More information about the Java-patches mailing list