Subject: | |
From: | |
Reply To: | |
Date: | Fri, 15 Jun 2007 10:47:02 -0500 |
Content-Type: | text/plain |
Parts/Attachments: |
|
|
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
__________________________________________________
|
|
|