[MPlayer-dev-eng] Patch to support Linux on MIPS/MIPS64

Matthew Johnson matt at matthew.ath.cx
Sun Feb 12 22:32:43 CET 2006


On Sun, 12 Feb 2006, Diego Biurrun wrote:

> Try the 'linux' command instead, it is defined at the top of configure.

aha, I know nothing about the mplayer build system, fair enough

> You are duplicating 10 lines, this is not good.  Instead you should use
> a different command to generate the value stored in 'proc'.  For Linux
> it should be your command, for Irix the one that is already there.

Yeah, good point.

>
> I'll apply a patch that features my suggestions right away.

Good, as long as it gets fixed, fine.

Thanks,
Matt

-- 
Matthew Johnson
http://www.matthew.ath.cx/




More information about the MPlayer-dev-eng mailing list