Re: [cp-patches] FYI: Some quick fixes for accessibility in Swing

[Mark Wielaard <mark at klomp dot org>]

On Thu, 2005-09-08 at 14:24 +0200, Mark Wielaard wrote:
> Disturbing. It also breaks (in the same way as reported above) with
> 'gcj (GCC) 4.0.2 20050908 (prerelease)' and 'gcj (GCC) 4.0.2 20050821
> (prerelease) (Debian 4.0.1-6)' so this seems to be a regression from
> 4.0-preX to 4.0.x.
> 
> I am inclined to just make the AccessibleAWTContainer and
> AccessibleAWTComponent public. while a little wrong at least it makes us
> able to compile again.

Tracked this down to a compile order bug in gcj. I added a small example
to http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21418

The workaround for now is just sorting the files in an order that
doesn't trigger the bug. That is a bit fragile of course, but it does
make compilation with gcj4 succeed again.

2005-09-11  Mark Wielaard  <mark@klomp.org>

    * lib/gen-classlist.sh.in: Use sort -r to work around bug #21418.

Committed,

Mark

? lib/Makefile.gcj.mark
Index: lib/gen-classlist.sh.in
===================================================================
RCS file: /cvsroot/classpath/classpath/lib/gen-classlist.sh.in,v
retrieving revision 1.30
diff -u -r1.30 gen-classlist.sh.in
--- lib/gen-classlist.sh.in	22 Aug 2005 21:04:35 -0000	1.30
+++ lib/gen-classlist.sh.in	11 Sep 2005 23:11:03 -0000
@@ -2,6 +2,8 @@
 # @configure_input@
 
 # Make sure sorting is done the same on all configurations
+# Note the use of sort -r below. This is done explicitly to work around
+# a gcj bug (http://gcc.gnu.org/bugzilla/show_bug.cgi?id=21418)
 LC_ALL=C; export LC_ALL
 LANG=C; export LANG
 
@@ -12,7 +14,7 @@
 @FIND@ @top_srcdir@/java @top_srcdir@/javax @top_srcdir@/gnu \
        @top_srcdir@/org \
        @top_srcdir@/external/w3c_dom @top_srcdir@/external/sax \
-       -follow -type f -print | sort | grep '\.java$' \
+       -follow -type f -print | sort -r | grep '\.java$' \
        > ${top_builddir}/lib/classes.1
 
 # Generate files for the VM classes.

Attachment: signature.asc
Description: This is a digitally signed message part