svn commit: r330645 - head/audio/audacity

[Alexey Dokuchaev]

On Sat, Oct 19, 2013 at 06:09:42PM +0200, Tijl Coosemans wrote:
> I looked into why the port is trying to use SSE on PowerPC and it looks
> like there's a bug in the Makefile: SSE_CONFIGURE_WITH=sse should be
> SSE_CONFIGURE_ENABLE=sse.  And "SSE" should be moved from OPTIONS_DEFAULT
> to OPTIONS_DEFAULT_amd64.  Then you can remove SSE_CFLAGS everywhere.
> 
> The question is if this should be an option at all.  I think it's ok to
> just enable SSE on all CPUs that support it like this:
> 
> .if ${MACHINE_CPU:Msse}
> CONFIGURE_ARGS+=	--enable-sse
> .else
> CONFIGURE_ARGS+=	--disable-sse
> .endif

Yes, that's what I was thinking of.  This way, package cluster will produce
universally runnable packages (since it defaults to i486), and correctly
optimized binaries when user sets his CPUTYPE in /etc/make.conf.

./danfe