[PATCH] bug #53345 - startkde

[Oswald Buddenhagen]

moin,

the first patch does what the report log suggests, i think it is a good
idea.

the second patch is unrelated, but as i was already at it ...
theoretically it could make the background appear earlier if the "xset
fp"s (particularily rehash) take some some - somebody can provide hard
numbers for this?

greetings

-- 
Hi! I'm a .signature virus! Copy me into your ~/.signature, please!
--
Chaos, panic, and disorder - my work here is done.
-------------- next part --------------
Index: startkde
===================================================================
RCS file: /home/kde/kdebase/startkde,v
retrieving revision 1.122
diff -U2 -r1.122 startkde
--- startkde	13 Sep 2003 03:15:35 -0000	1.122
+++ startkde	19 Sep 2003 14:56:18 -0000
@@ -140,4 +140,28 @@
 echo 'startkde: Starting up...'  1>&2
 
+exedirs=`kde-config --path exe`
+
+echo 'startkde: Running startup scripts...'  1>&2
+
+# Run scripts found in $KDEDIRS/startup
+for prefix in `echo $exedirs | sed -e 's^bin/^startup/^g;s^:^ ^'`
+do 
+  for file in $prefix/*
+  do 
+    if test -f $file
+    then
+      # Executables are simply run; everything else is assumed to be a shell
+      # script to source. The idea is, that you can set environment variables
+      # then.
+      if test -x $file
+      then
+        $file
+      else
+        . $file
+      fi
+    fi
+  done
+done
+
 # run KPersonalizer before the session, if this is the first login
 if kreadconfig --file kpersonalizerrc --group General --key FirstLogin --default true --type bool; then
@@ -195,5 +219,5 @@
 
 # Run scripts found in $KDEDIRS/shutdown
-for prefix in `kde-config --path exe| sed -e 's^bin/^shutdown/^g;s^:^ ^'`
+for prefix in `echo $exedirs | sed -e 's^bin/^shutdown/^g;s^:^ ^'`
 do 
   for file in $prefix/*
@@ -201,5 +225,12 @@
     if test -f $file
     then 
-       $file
+      # Setting variables is not really useful here, but sourcing is still
+      # faster than simply running it.
+      if test -x $file
+      then
+        $file
+      else
+        . $file
+      fi
     fi
   done
-------------- next part --------------
Index: startkde
===================================================================
RCS file: /home/kde/kdebase/startkde,v
retrieving revision 1.122
diff -U2 -r1.122 startkde
--- startkde	13 Sep 2003 03:15:35 -0000	1.122
+++ startkde	19 Sep 2003 14:56:18 -0000
@@ -11,4 +11,10 @@
 unset DYLD_FORCE_FLAT_NAMESPACE
 
+# Set the background to plain grey.
+# The standard X background is nasty, causing moire effects and exploding
+# people's heads. We use colours from the standard KDE palette for those with
+# palettised displays.
+test "$XDM_MANAGED" || xsetroot -solid "#C0C0C0"
+
 # Boot sequence:
 #
@@ -98,7 +104,4 @@
 xset fp rehash
 
-# if the user has overwritten fonts, the cursor font may be different now
-# so don't move this up
-
 # Set a left cursor instead of the standard X11 "X" cursor, since I've heard
 # from some users that they're confused and don't know what to do. This is
@@ -106,11 +109,8 @@
 # minutes until anything appears on the screen.
 #
-# Set the background to plain grey.
-# The standard X background is nasty, causing moire effects and exploding
-# people's heads. We use colours from the standard KDE palette for those with
-# palettised displays.
-
-test "$XDM_MANAGED" || bkg="-solid #C0C0C0"
-xsetroot -cursor_name left_ptr $bkg
+# If the user has overwritten fonts, the cursor font may be different now
+# so don't move this up.
+#
+xsetroot -cursor_name left_ptr
 
 # Get Ghostscript to look into user's KDE fonts dir for additional Fontmap