Re: [cp-patches] FYI: Disable gnome accessibility

2007-04-02 Thread Francis Kung

Francis> (is there any place where we can
Francis> document this requirement for at-spi/atk/gail >= 1.18.0?).

Perhaps configure could check for it with PKG_CHECK_MODULES.


I don't think we should introduce a dependency on those packages, as 
everything's fine if the accessibility packages aren't installed at all. 
 But *if* they are installed, they need to be at least a certain version.


Robert Schuster suggested the INSTALL file, which makes sense (duh, why 
didn't I think of that...).


So, committing this change as:

2007-04-02  Francis Kung  <[EMAIL PROTECTED]>

* INSTALL: Document known bug in at-spi, atk, and gail packages prior to
version 1.18.0.

Thanks,
Francis
Index: INSTALL
===
RCS file: /cvsroot/classpath/classpath/INSTALL,v
retrieving revision 1.42
diff -u -r1.42 INSTALL
--- INSTALL	1 Mar 2007 00:26:08 -	1.42
+++ INSTALL	2 Apr 2007 20:39:24 -
@@ -50,6 +50,11 @@
 - gdk-pixbuf
 
 	- XTest Extension (libXtst) for GdkRobot support in java.awt.
+	
+	- There is a bug in earlier versions of at-spi, atk, and gail, which are
+	used for GNOME accessibility.  Prior to version 1.18.0 of these packages,
+	gtk graphical applications should be run without accessibility (clear the
+	GTK_MODULES environment variable).
 
 	For building gcjwebplugin you'll need the Mozilla plugin
 	support headers and libraries.


Re: [cp-patches] FYI: Disable gnome accessibility

2007-03-30 Thread Tom Tromey
> "Francis" == Francis Kung <[EMAIL PROTECTED]> writes:

Francis> Yes, the problem is fixed after updating the set of accessibility
Francis> packages.  I'll revert the patch (is there any place where we can
Francis> document this requirement for at-spi/atk/gail >= 1.18.0?).

Perhaps configure could check for it with PKG_CHECK_MODULES.

Tom



Re: [cp-patches] FYI: Disable gnome accessibility

2007-03-29 Thread Francis Kung

 > Interesting, it appears to have been fixed in at-spi.
 > 
 > See also:
 > 
 > http://bugzilla.gnome.org/show_bug.cgi?id=329454
 > 
 > ... a link to this was just added to (gnome bz) 423057.


From reading all the discussion that seems to be the same bug.


Yes, the problem is fixed after updating the set of accessibility 
packages.  I'll revert the patch (is there any place where we can 
document this requirement for at-spi/atk/gail >= 1.18.0?).


Francis

2007-03-29  Francis Kung  <[EMAIL PROTECTED]>

* native/jni/gtk-peer/gnu_java_awt_peer_gtk_GtkToolkit.c
(Java_gnu_java_awt_peer_gtk_GtkToolkit_gtkInit): Revert last patch.




Re: [cp-patches] FYI: Disable gnome accessibility

2007-03-29 Thread Andrew Haley
Tom Tromey writes:
 > > "Andrew" == Andrew Haley <[EMAIL PROTECTED]> writes:
 > 
 > >> This is a gnome accessibility bug with multithreading, filed as 
 > >> http://bugzilla.gnome.org/show_bug.cgi?id=423057 , and this patch is a 
 > >> workaround until that issue is resolved.
 > 
 > Andrew> Is this the same bug as
 > Andrew> https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226959, or is it
 > Andrew> another one?
 > 
 > Interesting, it appears to have been fixed in at-spi.
 > 
 > See also:
 > 
 > http://bugzilla.gnome.org/show_bug.cgi?id=329454
 > 
 > ... a link to this was just added to (gnome bz) 423057.

>From reading all the discussion that seems to be the same bug.

Andrew.

-- 
Red Hat UK Ltd, Amberley Place, 107-111 Peascod Street, Windsor, Berkshire, SL4 
1TE, UK
Registered in England and Wales No. 3798903



Re: [cp-patches] FYI: Disable gnome accessibility

2007-03-29 Thread Tom Tromey
> "Andrew" == Andrew Haley <[EMAIL PROTECTED]> writes:

>> This is a gnome accessibility bug with multithreading, filed as 
>> http://bugzilla.gnome.org/show_bug.cgi?id=423057 , and this patch is a 
>> workaround until that issue is resolved.

Andrew> Is this the same bug as
Andrew> https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226959, or is it
Andrew> another one?

Interesting, it appears to have been fixed in at-spi.

See also:

http://bugzilla.gnome.org/show_bug.cgi?id=329454

... a link to this was just added to (gnome bz) 423057.

So maybe (hopefully) we don't need this after all... Francis, could
you try Classpath without your patch but with the latest (rawhide)
at-spi?  Thanks.

Tom



Re: [cp-patches] FYI: Disable gnome accessibility

2007-03-29 Thread Andrew Haley
Francis Kung writes:
 > This patch explicitly prevents GNOME accessibility modules from being 
 > loaded, as our GTK peers hang when accessibility is enabled.
 > 
 > This is a gnome accessibility bug with multithreading, filed as 
 > http://bugzilla.gnome.org/show_bug.cgi?id=423057 , and this patch is a 
 > workaround until that issue is resolved.

Is this the same bug as
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226959, or is it
another one?

Andrew.

-- 
Red Hat UK Ltd, Amberley Place, 107-111 Peascod Street, Windsor, Berkshire, SL4 
1TE, UK
Registered in England and Wales No. 3798903