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

Zuxy Meng zuxy.meng at gmail.com
Tue Dec 12 15:59:25 CET 2006


Hi,

2006/12/12, Reimar Doeffinger <Reimar.Doeffinger at stud.uni-karlsruhe.de>:
> 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)


Hmm...[], ! and -a/-o are used elsewhere in the configure. Forgive me
I really didn't have any idea which alternative provides better
compatibility.

> >       ;;
> > -    6) iproc=686
> > +      6) iproc=686
>
> cosmetics (also in some more places)

No. It's intended to accommodate a new surrounding 'case..esac' pair.

-- 
Zuxy
Beauty is truth,
While truth is beauty.
PGP KeyID: E8555ED6



More information about the MPlayer-dev-eng mailing list