Summary: | media-sound/yoshimi fails to build if -march does not allow SSE | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Diego Elio Pettenò (RETIRED) <flameeyes> |
Component: | New packages | Assignee: | Gentoo Sound Team <sound> |
Status: | RESOLVED UPSTREAM | ||
Severity: | normal | CC: | cal |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: | Build log |
Description
Diego Elio Pettenò (RETIRED)
2010-04-22 08:42:08 UTC
Created attachment 228711 [details]
Build log
I've added 0.056 to tree... Unlikely to solve this issue though, it's not clear to me why this is failing for you. Far as I can see it hits the same code here and builds fine(?) The CMake output is messed up, quick glance through the include files tells me the problem: In file included from /var/tmp/portage/media-sound/yoshimi-0.055.6/work/yoshimi-0.055.6/src/Misc/Util.cpp:201: /usr/lib/gcc/i686-pc-linux-gnu/4.4.3/include/xmmintrin.h:32:3: error: #error "SSE instruction set not enabled" Have fun. OK, it was likely a mistake to keyword this x86 in the first place. It should be safe to assume amd64 systems will always have SSE, right? flame@yamato tree % echo | gcc -dM -E - | grep -i sse #define __SSE2_MATH__ 1 #define __SSE_MATH__ 1 #define __SSE2__ 1 #define __SSE__ 1 Yeah :) That is unless you force it off (-mno-sse), but that will likely break other software as well. I guess you should ask upstream to check for __SSE__ definition before trying to use that header, and have a pure-C alternative… otherwise it'll never work outside x86-based arches. I've contacted upstream (by mailinglist because they don't have a bug tracking system) and dropped ~x86 for now. Is that good enough for resolved, upstream resolution? Yeah sounds good enough to me. ok then and upstream replied they have no intentions on supporting non-SSE platforms... lovely :) Oh yeah who cares about ARM… it's not going anywhere, after all… Limiting to x86_64 only sounds a little over exclusive. For what it's worth I've added checking for SSE availability to what will eventually become the next version of yoshimi (most likely 0.057). |