OK, good and better news.

I've written an updated SL_desktop_tweaks that checks to see if the startkde is 
broken, and if it is, fixes it.  It's currently in testing, but has been tested.

   yum --enablerepo=sl-testing update SL_desktop_tweaks

Troy

Troy Dawson wrote:
> Hi,
> I have good news and bad news.
> The good news is that I have fixed SL_desktop_tweaks, the other good 
> news is that I have recompiled kdebase with a higher number so that it 
> will be installed, bringing in a fresh startkde.
> 
> The bad news.
> If you point yum to both of those files, it will install the new kdebase 
> and *then* the new SL_desktop_tweaks, meaning your startkde still get's 
> wiped out.
> 
> What to do in the present.
> If you want to fix it by hand.
> 
>    yum --enablerepo=sl-testing clean all
>    yum --enablerepo=sl-testing update SL_desktop_tweaks
>    yum --enablerepo=sl-testing update kdebase
> 
> If you want to wait and let it be done automatically?
> I will be putting the new SL_desktop_tweaks into the errata today.
> I will be putting in the new kdebase into the errata in the middle of 
> next week.
> 
> Does anyone see the problem yet?
> Yep, fresh installs after next week that then do an update.
> 
> Solution?
> Well ... in my mind, this is critical.  We might have to do a re-spin of 
> S.L. 5.0.
> That's not the end of the world, we've done it before, we've just never 
> done it so late after a release.
> 
> Troy
> 
> Troy Dawson wrote:
>> Just to let everyone know, I'm working on this.  I have several 
>> comments from people on the correct way to fix it, it should be at 
>> least in testing in a couple hours.
>> I apologize for this.
>> Troy
>>
>> Jon Peatfield wrote:
>>> On Thu, 14 Jun 2007, Urs Beyerle wrote:
>>>
>>>>>
>>>>>
>>>>> I don't see why this happens ... Who edits /usr/bin/startkde with sed?
>>>>>
>>>>
>>>> ok, I found it. It is in trigger script of  SL_desktop_tweaks!
>>>>
>>>> this produces the error (in triggerpostun scriptlet):
>>>> # cat /usr/bin/startkde | sed 's:xsetroot -solid \"#5C0B13\":s:xsetroot
>>>> -solid \"#000000\":' > /usr/bin/startkde
>>>>
>>>> fails with
>>>> sed: -e expression #1, char 33: unknown option to `s'
>>>>
>>>> should it be, like this???
>>>> # cat /usr/bin/startkde | sed 's:xsetroot -solid \"#5C0B13\":xsetroot
>>>> -solid \"#000000\":' /usr/bin/startkde
>>>
>>> Apart from the destination...
>>>
>>> I think the typo is a little clearer if you do:
>>>
>>> $ rpm -q SL_desktop_tweaks --triggers | grep 'startkde.*sed' |sed 
>>> 's/.*|//g'
>>>  sed 's:xsetroot -solid \"#5C0B13\":xsetroot -solid \"#000000\":' > 
>>> /usr/bin/startkde.tmp
>>>  sed 's:xsetroot -solid \"#5C0B13\":s:xsetroot -solid \"#000000\":' > 
>>> /usr/bin/startkde.tmp
>>>
>>> the spurious 's:' isn't in the other relevant trigger...
>>>
>>>> if [ -f /usr/bin/startkde ] ; then
>>>>        cat /usr/bin/startkde | sed 's:xsetroot -solid 
>>>> \"#5C0B13\":s:xsetroot -solid \"#000000\":' > /usr/bin/startkde.tmp
>>>>        mv -f /usr/bin/startkde.tmp /usr/bin/startkde
>>>>        chmod 755 /usr/bin/startkde
>>>> fi
>>>
>>> Hmm I don't want to critisize because of the glass-houses stuff but 
>>> possibly using && might be good belt-and-braces here.
>>>
>>> Having said that I just spent 20 mins trying to figure out problems 
>>> caused by my use of:
>>>
>>>   printf "/isunix\na\ndoccmd = 'WWW'\n.\nw\nq\n" | ed docopt.m
>>>
>>> in a script which didn't expct someone to stick a comment in it's way 
>>> (oops).
>>>
>>> If you remove SL_desktop_tweaks before the upgrade and put it back 
>>> afterwards it would probably work too.  Given that we now are doing 
>>> our own 'desktop-tweaks' (though not as cleanly as this package!!), 
>>> I'm no longer sure I need SL_desktop_tweaks.
>>>
>>>> ...
>>>>
>>>> SL_desktop_tweaks-5-4 should be fixed !!!
>>>
>>> The sed on startkde stuff wasn't in the SL_desktop_tweaks for SL4 as 
>>> far as I can see.
>>>
>>> If it is being worked on, then the code to 'fix' 
>>> /etc/X11/gdm/gdm.conf can be removed since that no longer exists I 
>>> think it is /etc/gdm/custom.conf these days...
>>>
>>>  -- Jon
>>
>>
> 
> 


-- 
__________________________________________________
Troy Dawson  [log in to unmask]  (630)840-6468
Fermilab  ComputingDivision/LCSI/CSI DSS Group
__________________________________________________