[MPlayer-dev-eng] [PATCH] configure: enhancement for runtime cpu detection and --target support under x86

Reimar Doeffinger Reimar.Doeffinger at stud.uni-karlsruhe.de
Tue Dec 12 15:50:05 CET 2006


Hello,

Build system is not my area, but some random comments:

On Mon, Dec 04, 2006 at 10:26:53PM +0800, Zuxy Meng wrote:
> +if [ "$_runtime_cpudetection" = yes ] ; then

This applies to many, many places in the patch. I think for
compatibility reasons you're supposed to use the 'test' command instead
of []

> +  if ! ( x86 || x86_64 || ppc ) ; then

and "not" instead of !, there are shells that do not support '!'

> -elif x86 || x86_64; then
> +elif x86 || x86_64 && [ "$_runtime_cpudetection" = no -a -z "$_target" ] ; then

And do not use -a, use two checks and combine them with && (many more
cases of that)

>  	;;
> -    6) iproc=686
> +      6) iproc=686

cosmetics (also in some more places)

Greetings,
Reimar Doeffinger



More information about the MPlayer-dev-eng mailing list