cvs commit: ports/audio/openal Makefile

Dmitry Marakasov amdmi3 at amdmi3.ru
Mon Mar 16 07:37:38 PDT 2009


* Oliver Lehmann (lehmann at ans-netz.de) wrote:

> Will this also fix the following PR? (scroll below for the openal patch)

Seems so. I guess the fix in that PR is even better, as optimized
assembly code is used.

I think we can merge both fixes, something like that:

.if ${ARCH} == "i386"
OPTIONS+=	X86_ASM	"Enable x86 assembly code" on
.endif

...

.if ${ARCH} == "i386" && !defined(WITHOUT_X86_ASM)
BUILD_DEPENDS+=	nasm:${LOCALBASE}/devel/nasm
.endif

...

post-patch:
.if ${ARCH} != "i386" || defined(WITHOUT_X86_ASM)
	@${REINPLACE_CMD} -e '/^NASM=/ s|=.*|=no|' ${WRKSRC}/configure
.endif

and patches from ports/129362.

What do you think?

-- 
Dmitry Marakasov   .   55B5 0596 FF1E 8D84 5F56  9510 D35A 80DD F9D2 F77D
amdmi3 at amdmi3.ru  ..:  jabber: amdmi3 at jabber.ru    http://www.amdmi3.ru


More information about the cvs-all mailing list