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 348643 - orca back-compat for session AT startup
orca back-compat for session AT startup
Status: RESOLVED FIXED
Product: gnome-session
Classification: Core
Component: gnome-session
unspecified
Other Linux
: Normal normal
: ---
Assigned To: bill.haneman
Session Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-07-25 14:24 UTC by bill.haneman
Modified: 2006-08-07 14:33 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch to preserve compat with legacy gconf value of exec_ats (4.68 KB, patch)
2006-07-25 14:26 UTC, bill.haneman
none Details | Review
patch to preserve compat with legacy gconf value of exec_ats (678 bytes, patch)
2006-07-26 08:37 UTC, bill.haneman
none Details | Review
Patch that does the same as the previous, but really runs orca. (753 bytes, patch)
2006-08-02 19:01 UTC, Willie Walker
none Details | Review

Description bill.haneman 2006-07-25 14:24:10 UTC
orca is proposed for inclusion in 2.16 as the screen reading technology, by a consensus including the gnopernicus team.  However in a new 2.16 install with a shared user directory left over from 2.14, the old legacy screenreader AT startup would fail since it points to 'gnopernicus'.  The small attached patch provides fallback/migration behavior for the rare cases where the user's home directory exists already but the rest of the desktop operating environment is a 'fresh install'.
Comment 1 bill.haneman 2006-07-25 14:26:29 UTC
Created attachment 69583 [details] [review]
patch to preserve compat with legacy gconf value of exec_ats
Comment 2 bill.haneman 2006-07-26 08:37:10 UTC
Created attachment 69644 [details] [review]
patch to preserve compat with legacy gconf value of exec_ats
Comment 3 bill.haneman 2006-07-26 08:38:01 UTC
(thanks Will for noting that the previous patch was for the related bug.  The new patch solves the gnome-session side of the issue).
Comment 4 Willie Walker 2006-08-02 14:37:54 UTC
Hi All - hold off on this patch for a bit.  It's incomplete (main.c needs updating as well to reflect the AtStartupState struct changes).
Comment 5 bill.haneman 2006-08-02 17:15:51 UTC
No, this patch is fine. Will's talking about a different bug.
Comment 6 bill.haneman 2006-08-02 17:20:26 UTC
FYI the 'other bug' is 648630, and the problem seems to be that the original diff didn't include the change to main.c.  I have updated the bugzilla patch to include that part of the diff.

Will, please confirm that this patch (gnome-session patch, 348643) is correct, for the benefit of the maintainers :-)
Comment 7 Willie Walker 2006-08-02 19:01:40 UTC
Created attachment 70089 [details] [review]
Patch that does the same as the previous, but really runs orca.

After testing, I determined that the previous patch was flawed: it attempted to run gnopernicus yet again rather than using orca (i.e., it reused 'exec_string' instead of the newly created 's').  This new patch will really run orca in the event that gnopernicus is not found (assuming orca is installed).
Comment 8 Vincent Untz 2006-08-07 14:33:50 UTC
I committed a modified patch.
Thanks!