Bug 24106 - Allow at-spi2-core files to co-exist with AT-SPI/CORBA
Summary: Allow at-spi2-core files to co-exist with AT-SPI/CORBA
Status: NEW
Alias: None
Product: at-spi2
Classification: Unclassified
Component: core (show other bugs)
Version: unspecified
Hardware: Other All
: medium normal
Assignee: Mark Doffman
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 24108
  Show dependency treegraph
 
Reported: 2009-09-23 05:43 UTC by Willie Walker
Modified: 2009-10-01 12:51 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
Patch (7.49 KB, patch)
2009-09-23 05:45 UTC, Willie Walker
Details | Splinter Review
Patch to remove all notions of session management and a *.desktop file (6.71 KB, patch)
2009-10-01 11:57 UTC, Willie Walker
Details | Splinter Review

Description Willie Walker 2009-09-23 05:43:04 UTC
This patch helps the files from the AT-SPI core package to
co-exist with the files from the AT-SPI/CORBA implementation.
Note that this is about the existence of files on the
files ystem and not the ability to run the two implementations
simultaneously.

A summary of the changes is as follows:

* Wherever we use the string 'spi', a '2' is appended.  For example,

-		 registryd/at-spi-registryd.desktop.in
+		 registryd/at-spi2-registryd.desktop.in

* The *.desktop file uses /desktop/gnome/interface/accessibility2 as
  the condition for starting.  The general notion is to unset
  /desktop/gnome/interface/accessibility to turn off the CORBA
  implementation and set /desktop/gnome/interface/accessibility2 to
  turn on the new implementation.

gconftool-2 --set /desktop/gnome/interface/accessibility --type bool false
gconftool-2 --set /desktop/gnome/interface/accessibility2 --type bool true

* Via a patch from Mike Gorse, the registry connects with gnome-session
  to set its GTK_MODULES environment variable.  This needs to be done
  in the gnome-session initialization phase since that's a constraint
  of gnome-session.  Note that setting GTK_MODULES is important for
  gnome-session because it launches many GUI objects for the desktop.

When we ultimately decide to completely drop CORBA for GNOME 3.0, we'll
need to make the following changes:

* Drop the '2' suffix.  The occurrences of this can be found by running 
  the following command on a freshly cloned space:

find . -type f | xargs egrep -i "spi2|accessibility2"
Comment 1 Willie Walker 2009-09-23 05:45:12 UTC
Created attachment 29801 [details] [review]
Patch
Comment 2 Willie Walker 2009-09-23 06:13:12 UTC
See also bug #24108 for the at-spi2-atk patch associated with this patch.
Comment 3 Willie Walker 2009-10-01 11:57:45 UTC
Created attachment 29983 [details] [review]
Patch to remove all notions of session management and a *.desktop file

This patch cleans up core, ridding it of any knowledge of session management and a *.desktop file.  As intended, the launching of at-spi-registryd now relies upon D-Bus activation.

In looking at the GTK+ AT-SPI mechanics some more, there is already magic in gnome-session (gnome-settings-daemon more specifically) and GTK+ that eliminates the need to set the GTK_MODULES environment variable.  So, there's really no need for the AT-SPI registry daemon to talk to the session manager now.

NOTE - after applying the patch, if you want to build/install at-spi2-core in a way that allows it to co-exist with the CORBA/Bonobo-based at-spi, you can do this:

./{autgen.sh,configure} --prefix=/usr --libexecdir=/opt/atspi-dbus/libexec --disable-xevie

This will place most files (e.g., droute, dbind, etc.) under /usr, but will install at-spi-registryd under /opt/atspi-dbus/libexec/at-spi-registryd and make sure the D-Bus service file points to the location of the at-spi-registryd.
Comment 4 Willie Walker 2009-10-01 12:51:04 UTC
NOTE: it would be nice to find a way to disable the launching of the CORBA-based at-spi-registryd.  Right now it is launched via the /etc/xdg/at-spi-registry*.desktop files based upon the /desktop/gnome/interface/accessibility gconf setting.

While having the CORBA-based registry start and not talk to anything doesn't seem so bad, apparently there *might* be an interaction with it and the input device code (e.g., keyboard event interception) that can cause a lock up.  So, it would be desirable to not have this the CORBA one launch if we don't want it launched.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.