After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 386411 - Depricate gnome-session from directly starting any ATs
Depricate gnome-session from directly starting any ATs
Status: RESOLVED FIXED
Product: gnome-session
Classification: Core
Component: gnome-session
unspecified
Other All
: Normal normal
: ---
Assigned To: Session Maintainers
Session Maintainers
Depends on: 350263
Blocks:
 
 
Reported: 2006-12-16 06:09 UTC by George Kraft IV
Modified: 2007-05-06 08:02 UTC
See Also:
GNOME target: ---
GNOME version: 2.17/2.18


Attachments
Don' launch any ATs from gnome-session (2.20 KB, patch)
2006-12-20 20:07 UTC, Ariel Rios
committed Details | Review
Patch used to revert (3.32 KB, patch)
2007-02-05 22:44 UTC, Vincent Untz
committed Details | Review

Description George Kraft IV 2006-12-16 06:09:27 UTC
gnome-session should no longer try to start the assistive technologies. /usr/share/gnome/autostart/gnome-at-session.desktop with the help of /usr/bin/gnome-at-visual now starts the assistive technologies according to what was set in Preferred Applications.  See bug 350263.
Comment 1 George Kraft IV 2006-12-18 20:37:45 UTC
Rationale http://live.gnome.org/GAP/ScratchPad/PreferredAppliations
Comment 2 Ariel Rios 2006-12-20 20:07:09 UTC
Created attachment 78710 [details] [review]
Don' launch any ATs from gnome-session

This patch removes launching the apps that were included in the now deprecated exec_ats key. Might want to try it with your environment.

ariel
Comment 3 Tom Tromey 2007-01-06 06:13:51 UTC
This patch looks reasonable to me (but I can't approve patches).
A ChangeLog entry would be nice :-)
Comment 4 Vincent Untz 2007-01-06 10:10:18 UTC
I removed some more code and committed this.

2007-01-06  Vincent Untz  <vuntz@gnome.org>

	* gsm-at-startup.[ch]: (gsm_assistive_tech_exec): kill
	(gsm_assistive_registry_start): directly call
	gsm_exec_command_line_async() instead of gsm_assistive_tech_exec()
	(gsm_assistive_technologies_start): kill
	* main.c: (main): don't call gsm_assistive_technologies_start()
	Fix bug #386411, based on patch by Ariel Rios <ariel@gnu.org>
Comment 5 George Kraft IV 2007-01-24 21:52:34 UTC
Vincent, thanks for committing this code.  Unfortunately, the gnome-cc maintainers are deferring 350263 until 2.20.   This patch without the other breaks accessibility.  I'm afraid this patch needs to be backed out.
Comment 6 Vincent Untz 2007-02-05 22:44:13 UTC
Created attachment 81960 [details] [review]
Patch used to revert

This is how it got reverted. Will be useful when fixing this in 2.19.
Comment 7 George Kraft IV 2007-04-04 17:44:10 UTC
Please reapply patch after 350263 is committed. 
Comment 8 Rodrigo Moya 2007-04-23 15:14:31 UTC
350263 is now committed, so you can reapply this
Comment 9 George Kraft IV 2007-04-25 13:13:43 UTC
The prerequsite patch has been applied, so /usr/share/gnome/autostart/gnome-at-session.desktop is now responsible for starting the assistive technologies.  Please reapply this this patch to remove the dead code in gnome-session which tries to start orca, magnifier, and gok.  Thanks.