[SCM] supercollider/master: supernova compile fix on x86

[Dan S]

2012/12/25 Felipe Sateler <fsateler at debian.org>:
> On Fri, Dec 21, 2012 at 3:03 PM,
> <danstowell-guest at users.alioth.debian.org> wrote:
>> The following commit has been merged in the master branch:
>> commit b0855e79951393de8d0e044af3abd679b463a20f
>> Author: Dan Stowell <danstowell at users.sourceforge.net>
>> Date:   Fri Dec 21 17:42:31 2012 +0000
>>
>>     supernova compile fix on x86
>>
>> diff --git a/debian/patches/series b/debian/patches/series
>> index 2f7e98b..2ba49f0 100644
>> --- a/debian/patches/series
>> +++ b/debian/patches/series
>> @@ -1,2 +1,3 @@
>>  no-inline-ppc-ftbfs.patch
>>  perf-counter-include.patch
>> +supernova-i686-march-flag.patch
>> diff --git a/debian/patches/supernova-i686-march-flag.patch b/debian/patches/supernova-i686-march-flag.patch
>> new file mode 100644
>> index 0000000..a692303
>> --- /dev/null
>> +++ b/debian/patches/supernova-i686-march-flag.patch
>> @@ -0,0 +1,15 @@
>> +Index: supercollider/server/supernova/CMakeLists.txt
>> +===================================================================
>> +--- supercollider.orig/server/supernova/CMakeLists.txt 2012-12-21 03:44:14.806118662 +0000
>> ++++ supercollider/server/supernova/CMakeLists.txt      2012-12-21 03:44:37.562118316 +0000
>> +@@ -12,6 +12,10 @@
>> +     endif()
>> + endif()
>> +
>> ++if(${CMAKE_SYSTEM_PROCESSOR} MATCHES "i686")
>> ++  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -march=i686")
>> ++endif()
>
> This is wrong. Debian is supposed to work from i486 onwards. Why is this needed?

I discussed this with upstream and with debian-mentors. I'm afraid it
is needed because supernova uses some pentium+ cpu instructions (and
they can be emulated but not in a realtime system). Please note that
"supernova" is the SIMDified parallel-processing drop-in replacement
for "scsynth", and so i486 users would be advised to use the scsynth
package, losing no functionality.

Dan