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
__________________________________________________