Supercollider fails to build

[Dan S]

2012/7/21 Felipe Sateler <fsateler at debian.org>:
> On Sat, Jul 21, 2012 at 4:18 AM, Dan S <danstowell+debmm at gmail.com> wrote:
>> 2012/7/21 Felipe Sateler <fsateler at debian.org>:
>>> On Thu, Jul 12, 2012 at 5:50 PM, Dan S <danstowell+debmm at gmail.com> wrote:
>>>>
>>>> 2012/7/10 Felipe Sateler <fsateler at debian.org>:
>>>> > OK, so we have more build failures, but this time different on
>>>> > different architectures:
>>>> >
>>>> > 1. Arm*: qreal is not double but float. Patch is in the repository.
>>>> > 2. PowerPc64: Undefined variable order. It seems that there is a bogus
>>>> > call in line 255 of
>>>> > external_libraries/boost_lockfree/boost/atomic/detail/gcc-ppc.hpp, but
>>>> > I'm not sure.
>>>> > 3. ia64: missing ifdef case in
>>>> > external_libraries/nova-simd/benchmarks/perf_counter.hpp (starting at
>>>> > line 466). I don't know how to fix this.
>>>> > 4. all big-endian archs: Typo in
>>>> > external_libraries/boost_endian/boost/integer/endian.hpp:285. It seems
>>>> > like an extra ) crept in in the least-tested #ifdef path. Patch in the
>>>> > repository too
>>>>
>>>> I believe issues 2 and 3 are now fixed, with the small patches I added in
>>>> git.
>>>> The upstream fixes are different (importing a more recent version of
>>>> boost.atomic) but the patches would be big and complex.
>>>
>>> Uploaded.
>>
>> Thanks
>
> ... and it fails again :(
>
> https://buildd.debian.org/status/package.php?p=supercollider

OK. I'll have to pass the powerpc asm issue onwards. For the others:

The ia64 issue might be the same as this in powertop:
<http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=570662> and so I
propose untested patch [1].
For the arm*s, I'd suggest adding [2] to your existing
"arm-qreal-is-float.patch". There may be more "double"-->"qreal"
changes needed.

If there's a way for me to test these so I can iterate, do say.

Best
Dan



[1]
--- a/external_libraries/nova-simd/benchmarks/perf_counter.hpp
+++ b/external_libraries/nova-simd/benchmarks/perf_counter.hpp
@@ -49,6 +49,13 @@
 #include <linux/ioctl.h>
 #include <asm/byteorder.h>

+#ifdef __ia64__
+#include <asm-generic/unistd.h>
+#endif
+#ifdef __alpha__
+#include <asm-generic/unistd.h>
+#endif
+
 /*
  * User-space ABI bits:
  */


[2]
--- a/QtCollider/widgets/QcMultiSlider.cpp
+++ b/QtCollider/widgets/QcMultiSlider.cpp
@@ -182,7 +182,7 @@ void QcMultiSlider::mousePressEvent( QMouseEvent *e )
   if( !c ) return;

   bool horiz = ort == Qt::Horizontal;
-  double spacing;
+  qreal spacing;

   QRect r( valueRect( c - startIndex, spacing ) );

@@ -226,7 +226,7 @@ void QcMultiSlider::mouseMoveEvent( QMouseEvent *e )
     return;
   }

-  double xStep;
+  qreal xStep;

   QRect r( valueRect( c - startIndex, xStep ) );

@@ -329,10 +329,10 @@ void QcMultiSlider::paintEvent( QPaintEvent *e )
   }

   int count = _values.count() - startIndex;
-  double spacing, width, yscale;
+  qreal spacing, width, yscale;

-  spacing = elastic ? (double) bounds.width() / count :
thumbSize.width() + gap;
-  width = elastic ? qMin( spacing, (double) thumbSize.width() ) :
thumbSize.width();
+  spacing = elastic ? (qreal) bounds.width() / count : thumbSize.width() + gap;
+  width = elastic ? qMin( spacing, (qreal) thumbSize.width() ) :
thumbSize.width();
   yscale = bounds.height();
   if( !isFilled ) yscale -= thumbSize.height();