On Mon, Aug 06, 2012 at 05:17:22PM +0100, Måns Rullgård wrote:
> Diego Biurrun <[email protected]> writes:
> >
> > So I would suggest making yasm 0.8.0 and nasm 2.09.07 the minimum
> > requirement.  This covers all current systems except RHEL 6 derivatives
> > and Cygwin while having minimal impact on our codebase.  Sounds like
> > a good deal to me.
> 
> I don't think RHEL matters much, but supporting out of the box cygwin
> might gains us some PR points.
> 
> The patch I posted for nasm 2.08 support is ugly, and I'd probably
> reject it myself if someone else sent it.  That doesn't mean that a
> better fix is impossible, someone just has to find it.  You're giving up
> a little too easily.

I just believe my suggestion is a good compromise between platform support
vs. longterm maintainability (i.e. fewer ifdefs).  Even if we did find a
better solution for nasm 2.08, we would have to keep the ifdefs for AVX.

I'll ask the Cygwin folks if/when they are going to upgrade nasm.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to