[MPlayer-cvslog] CVS: main configure,1.1075,1.1076

[Alexander Strasser]

Hi,

Richard Felker CVS wrote:
> CVS change done by Richard Felker CVS
> 
> Update of /cvsroot/mplayer/main
> In directory mail:/var2/tmp/cvs-serv23361
> 
> Modified Files:
> 	configure 
> Log Message:
> work around (buggy?) solaris tr. hope this helps.. please report if its still broken
> 
> Index: configure
> ===================================================================
> RCS file: /cvsroot/mplayer/main/configure,v
> retrieving revision 1.1075
> retrieving revision 1.1076
> diff -u -r1.1075 -r1.1076
> --- configure	8 Oct 2005 22:28:25 -0000	1.1075
> +++ configure	11 Oct 2005 04:37:38 -0000	1.1076
> @@ -7027,7 +7027,7 @@
>  AMR_NB=$_amr_nb
>  AMR_NB_FIXED=$_amr_nb_fixed
>  AMR_WB=$_amr_wb
> -`echo $_libavcodecs | tr 'a-z ' 'A-Z\n' | sed 's/^/CONFIG_/;s/$/=yes/'`
> +`echo $_libavcodecs | tr a-z A-Z | tr ' ' '\n' | sed 's/^/CONFIG_/;s/$/=yes/'`

  I did some testing about this and AFAIK your changes won't help.
It is somehow like this, on newer solaris versions another tr is used
which needs character ranges as e.g. [A-Z]. There is normally also
a version which also works with A-Z also available.
  It can be fixed like tr '[a-z] ' '[A-Z]\n' and should still work
on other systems as you replace the brackets with the brackets there.

  So what should we do about this one?

  Alex (beastd)