libvpx: don't depend on yasm on PowerPC

[ballapete (Peter "Pete" Dyballa)]

The description tells:

Description:          A modular x86/amd64 assembler supporting NASM and GAS asm syntaxes and a
                      variety of output formats. It is licensed under the 3-clause BSD license
                      except the nasm preprocessor which is LGPL.

so it's pretty useless on PPC hardware – except, maybe, one wants to build an universal binary. This is not the case for default libvpx (or gd2) port, so there is no reason to install it or to make standard libvpx depend on yasm.

lipo -detailed_info /opt/local/lib/libvpx.a
input file /opt/local/lib/libvpx.a is not a fat file
Non-fat file: /opt/local/lib/libvpx.a is architecture: ppc

[ryandesign (Ryan Carsten Schmidt)]

Sounds like a reasonable request.

libvpx will use yasm if available, and try to use nasm (which is included in OS X) otherwise. I verified that on Mountain Lion, the build fails using nasm, so yasm is definitely required there. I'll test on PowerPC and see what happens without yasm there.

[ballapete (Peter "Pete" Dyballa)]

The same happens on PPC Leopard, Mac OS X 10.5.8.

[dbevans (David B. Evans)]

While I understand the motivation behind the request, but removing yasm on PPC platforms won't do much unless a reasonable replacement that builds successfully on these platforms can be provided. Although, I am the libvpx maintainer, I don't have access to any PPC systems at this point so I must reply on those who do to help test and debug any proposed solution.

So please be patient while I come up to speed on this issue. It would be helpful to clarify exactly what works and what doesn't and how that is accomplished. The initial report here seems to imply that building without yasm works on Tiger at least, but later it is reported that nasm does not work on Leopard. A precise statement of results on the different PPC platforms using specific configurations including build logs and patches where appropriate would be helpful.

Thanks in advance for any help and thanks to ryandesign for bringing this issue to my attention.

[dbevans (David B. Evans)]

As ryandesign has pointed out in #47907, libvpx 1.4.0 is now available.

In this version, the following PPC targets are explicitly supported

    ppc32-darwin8-gcc        ppc32-darwin9-gcc        
    ppc64-darwin8-gcc        ppc64-darwin9-gcc         

I plan to incorporate these PPC targets into the 1.4.0 update and will post a patch here for testing prior to committing the update. I'll be doing this pretty much in the blind, so those of you who have PPC platforms available, please let me know which of these targets you can test.

For darwin10+ yasm remains the assembler preferred upstream.

Thanks again.

[larryv (Lawrence Velázquez)]

I can test Rosetta builds on x86_64 Leopard and Snow Leopard.

[dbevans (David B. Evans)]

Replying to larryv@…:

I can test Rosetta builds on x86_64 Leopard and Snow Leopard.

Great. See initial libvpx 1.4.0 test port attached to #47907. All testing help is gratefully received.

[dbevans (David B. Evans)]

Proposed upgrade to libvpx-1.4.0 with yasm dependency fix

[dbevans (David B. Evans)]

Proposed patch to upgrade libvpx to version 1.4.0 with fix for yasm dependency issue attached. See #47907 for discussion.

Would appreciate it if you could test this patch on darwin 8/9 ppc and verify that it does indeed fix the issue reported here. Note that yasm is required if an x86 target is specified so universal builds will still depend on yasm but, hopefully, non-universal builds will not.

[ryandesign (Ryan Carsten Schmidt)]

As far as I can tell, my proposed fix works, and the yasm dependency is now only added when building for an Intel arch. Committed in r137226.

[dbevans (David B. Evans)]

Thanks for your reply, Ryan. I was hoping to get a response from the reporter of this ticket.

[ryandesign (Ryan Carsten Schmidt)]

I also built libvpx 1.4.0 with this dependency change for ppc on Tiger without yasm installed, which worked fine.

I was not able to build universal ppc i386 on Tiger, but I think that affects all ports, because the apple-gcc42 port, which all ports use to compile on Tiger, only supports either all-Intel or all-PowerPC universal builds.

[ballapete (Peter "Pete" Dyballa)]

Replying to devans@…:

Proposed patch to upgrade libvpx to version 1.4.0 with fix for yasm dependency issue attached. See #47907 for discussion.

Would appreciate it if you could test this patch on darwin 8/9 ppc and verify that it does indeed fix the issue reported here. Note that yasm is required if an x86 target is specified so universal builds will still depend on yasm but, hopefully, non-universal builds will not.

Last month I had too many other things to do, sorry! Yesterday I could successfully update libvpx.