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 608138 - Add AutostartCondition to gnome-at-session.desktop
Add AutostartCondition to gnome-at-session.desktop
Status: RESOLVED FIXED
Product: gnome-control-center
Classification: Core
Component: [obsolete] Assistive Technology Preferences
git master
Other All
: Normal enhancement
: ---
Assigned To: Control-Center Maintainers
Control-Center Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-01-26 10:55 UTC by Martin Pitt
Modified: 2010-01-26 13:19 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed git-formatted patch (1.24 KB, patch)
2010-01-26 10:56 UTC, Martin Pitt
accepted-commit_now Details | Review

Description Martin Pitt 2010-01-26 10:55:19 UTC
This autostart desktop file currently launches a shell script, which in turn calls gconftool-2, just to find out that it's disabled, and exit. This wastes unnecessary cycles on starting the desktop.

I think it would be appropriate to add an AutostartCondition to the .desktop file, so that it's not run in the first place if a11y is disabled.

What do you think?
Comment 1 Martin Pitt 2010-01-26 10:56:59 UTC
Created attachment 152298 [details] [review]
proposed git-formatted patch

Proposed patch.
Comment 2 Martin Pitt 2010-01-26 13:19:42 UTC
Jens, thanks for the review. I pushed the patch.