As stated in the ticket Metacity sets the cursor after it's startup, so we need to make sure Metacity is already running before we set the cursor. This is what the current code has been doing. We do show the intro screen before the window manager may be available. For a non invasive bug fix that avoids races we just call 'xsetroot -cursor_name' another time before the intro screen is displayed.
We have a long history of setting the Sugar cursor with many races. To improve the situation SL #3204 is filed as a follow up. Signed-off-by: Simon Schampijer <si...@laptop.org> --- bin/sugar-session | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/bin/sugar-session b/bin/sugar-session index 8ea195f..7df52f8 100755 --- a/bin/sugar-session +++ b/bin/sugar-session @@ -245,6 +245,9 @@ def main(): # even when we initially get blocked on the intro screen gobject.idle_add(unfreeze_dcon_cb) + # make sure we have the correct cursor in the intro screen TODO #3204 + if subprocess.call('echo $DISPLAY; xsetroot -cursor_name left_ptr', shell=True): + logging.warning('Can not reset cursor') intro.check_profile() start_ui_service() -- 1.7.4.4 _______________________________________________ Sugar-devel mailing list Sugar-devel@lists.sugarlabs.org http://lists.sugarlabs.org/listinfo/sugar-devel