[Sugar-devel] [PATCH sugar] Make sure we use the Sugar cursor in the intro screen, OLPC #10712

[Simon Schampijer]

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 <simon at 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