[MPlayer-dev-eng] [PATCH] configure LANGUAGE settings patch

Diego Biurrun diego at biurrun.de
Sun Aug 1 00:17:14 CEST 2004


Chris White writes:
> 
> "It seems that the configure script for mplayer does not handle when 
> LINGUAS have multiple similar languages set. E.g: LINGUAS="sv sv_SE".
> Configure contains some sed lines for excluding languages not supported, 
> but those sed lines does not account for when LINGUAS is set as said 
> above."
> --- MPlayer-1.0pre5_orig/configure	2004-07-23 18:06:56.422863704 +0200
> +++ MPlayer-1.0pre5/configure	2004-07-23 18:08:05.252400024 +0200
> @@ -2079,15 +2079,15 @@
>      break
>    else
>      echo -n "$i not found, "
> -    _language=`echo $_language | sed "s/$i *//g"`
> +    _language=`echo $_language | sed "s/\<$i\> *//g"`

This is not overly portable and it only papers over the problem.  The
real bug is the extra "g" at the end of the sed command.  It makes the
sed command remove all instances of "sv", thus transforming "sv_SE"
into "_SE".

Fixed.

Diego




More information about the MPlayer-dev-eng mailing list