[Dextrose] [PATCH][sugar] check for empty cursor_theme

Martin Abente martin.abente.lahaye at gmail.com
Wed Mar 9 11:15:10 EST 2011


In dextrose2 caacupe tests we noticed that many laptops
had the standard x cursor theme instead of the sugar one.
Apparently this bug was also present on dextrose1 builds.

An interesting finding while looking at this report was
that the accesibility code for changing cursor it simply
does not affect the cursor at all, and the real change is
being done at bin/sugar.

I have not been able to determine when or how this
empty value is being set (yet), but this patch will
make sure it shows the sugar cursor theme whenever 
it happens.

Could this patch make sense for mainstream sugar also?
---
 bin/sugar.in |    8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/bin/sugar.in b/bin/sugar.in
index bd6e95e..73af03c 100644
--- a/bin/sugar.in
+++ b/bin/sugar.in
@@ -80,10 +80,12 @@ fi
 # Set cursor theme
 gconftool-2 --dir-exists=/desktop/sugar/peripherals/mouse
 EXIST_SUGAR_CURSOR_THEME=$(echo $?)
+CURSOR_THEME="sugar"
 if [[ $EXIST_SUGAR_CURSOR_THEME == 0 ]]; then
-    CURSOR_THEME=$(gconftool-2 --get /desktop/sugar/peripherals/mouse/cursor_theme);
-else
-    CURSOR_THEME="sugar"
+    STORED_CURSOR_THEME=$(gconftool-2 --get /desktop/sugar/peripherals/mouse/cursor_theme);
+    if [[ $STORED_CURSOR_THEME != "" ]]; then
+        CURSOR_THEME=$STORED_CURSOR_THEME
+    fi
 fi
 
 
-- 
1.7.1



More information about the Dextrose mailing list