[oe] xterm: either fix it, or remove it. please.

[Holger Hans Peter Freyther]

On Thursday 12 November 2009 08:20:08 Holger Hans Peter Freyther wrote:
> On Thursday 12 November 2009 07:36:32 Robert P. J. Day wrote:
> >   anyway, this is about the fourth time i've explained this, either
> > here or on the angstrom-devel list.  that's the error, and it's
> > entirely reproducible on my f11 system.  at this point, i'm leaving it
> > with the powers that be, and you're free to deal with it or totally
> > ignore it.  whatever suits you.
> 
> Robert,
> 
> I totally agree that bugs should be fixed. Your attitude is somehow weird
>  as you demand others to fix problems you are experiencing. Honestly
>  speaking I am a bit disturbed by this attitude. Work on OE is purely
>  community driven, if something reported is not picked up, it is saddening,
>  but you really can't play the escalation game from the commercial world.
> 
> Did you consider updating xterm from 207 to something more recent, or did
>  you consider taking a newer install script and patch it into 207?
>  Apparently between 207 and 243 (as of ubuntu) there must have been
>  changes.

And diffing sinstall.sh from 207 to 243 one is seeing some interesting
changes... the first is to set the locale to C (good change) and the second
is in changing PROGRAM_MODE.. Maybe you want to test this and then
come up with a patch? I'm happy to review and land it.

z.


-       PROG_MODE=`echo ".$cf_mode" | sed -e 's/^..//' -e 's/rw./7/g' -e 's/r-./5/g' -e 's/--[sxt]/1/g'`
+       PROG_MODE=`echo ".$cf_mode" | sed -e 's/^..//' -e 's/rw./7/g' -e 's/r-./5/g' -e 's/---/0/g' -e 's/--[sxt]/1/g' -e 's/+//g'`