Re: Supercollider fails to build
On Sat, Jul 21, 2012 at 5:07 PM, Dan S danstowell+de...@gmail.com wrote: 2012/7/21 Felipe Sateler fsate...@debian.org: On Sat, Jul 21, 2012 at 4:18 AM, Dan S danstowell+de...@gmail.com wrote: 2012/7/21 Felipe Sateler fsate...@debian.org: On Thu, Jul 12, 2012 at 5:50 PM, Dan S danstowell+de...@gmail.com wrote: 2012/7/10 Felipe Sateler fsate...@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. Googling tells me that apparently there is some bug in gcc causes it to generate too big asm sections, which is why the assembler chokes. Applying --- a/external_libraries/boost_lockfree/boost/atomic/detail/gcc-ppc.hpp +++ b/external_libraries/boost_lockfree/boost/atomic/detail/gcc-ppc.hpp @@ -124,7 +124,7 @@ public: T expected, T desired, memory_order success_order, - memory_order failure_order) volatile + memory_order failure_order) volatile __attribute__((noinline)) Seems to fix the issue. SC builds on the porterbox with that change. 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]. Indeed, supercollider builds with this patch applied. 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. I'm testing the patch. Arm is slow to build, though. -- Saludos, Felipe Sateler ___ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers
Re: Supercollider fails to build
2012/7/21 Felipe Sateler fsate...@debian.org: On Sat, Jul 21, 2012 at 4:18 AM, Dan S danstowell+de...@gmail.com wrote: 2012/7/21 Felipe Sateler fsate...@debian.org: On Thu, Jul 12, 2012 at 5:50 PM, Dan S danstowell+de...@gmail.com wrote: 2012/7/10 Felipe Sateler fsate...@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(); ___ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers
Re: Supercollider fails to build
On Thu, Jul 12, 2012 at 5:50 PM, Dan S danstowell+de...@gmail.com wrote: 2012/7/10 Felipe Sateler fsate...@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. Please next time separate the actual change commits from the changelog editing commit. This helps in cherry-picking and avoids the wrong entry problem. -- Saludos, Felipe Sateler ___ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers
Re: Supercollider fails to build
2012/7/11 Felipe Sateler fsate...@debian.org: On Wed, Jul 11, 2012 at 3:21 AM, Dan S danstowell+de...@gmail.com wrote: A fix for issue 2 will be along in a week or two, after the developer has integrated an updated source file. Any temporary fix we can apply? The longer we take to fix this, the less likely the release managers are going to give sc an exception. For issue 3, we don't have an ia64 to test on, but it looks to me like the attached patch might fix it. Shall I push it? Maybe it is better to do it the other way around. Special case the cases that are known to work and enable it? True BTW, any reason you replied to me personally instead of the list? I was convinced you'd emailed offlist - but I was wrong. I'm on-listing this now. Dan ___ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers
Re: Supercollider fails to build
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 Dan, could you forward these issues upsteam? -- Saludos, Felipe Sateler ___ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers
Re: Supercollider fails to build
2012/7/3 Felipe Sateler fsate...@debian.org: On Mon, Jul 2, 2012 at 12:46 PM, Felipe Sateler fsate...@gmail.com wrote: Supercollider is failing to build everywhere except i386 and amd64. Boost.Lockfree seems to be the culprit everywhere. I think the problem is that BOOST_LOCKFREE_DCAS_ALIGNMENT is only defined in a few specific cases (for x86 and x64). Dan, could you check with upstream what could be causing this? I can confirm that after defining the above macro the offending file does compile (although wether it works is another thing I haven't tested). I cannot confirm that the build continues, though, since the porterbox I'm logged to doesn't have all the required build depends. I've pushed a patch recommended by Tim, who developed boost_lockfree. The patch simply updates that file to the latest version in boost's SVN, which as you'll see has the effect of simply defining that macro. I don't have access to machines for testing this, so I'd be grateful if anyone could test. Thanks Dan ___ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers
Re: Supercollider fails to build
On Tue, Jul 3, 2012 at 3:34 AM, Dan S danstowell+de...@gmail.com wrote: 2012/7/3 Felipe Sateler fsate...@debian.org: On Mon, Jul 2, 2012 at 12:46 PM, Felipe Sateler fsate...@gmail.com wrote: Supercollider is failing to build everywhere except i386 and amd64. Boost.Lockfree seems to be the culprit everywhere. I think the problem is that BOOST_LOCKFREE_DCAS_ALIGNMENT is only defined in a few specific cases (for x86 and x64). Dan, could you check with upstream what could be causing this? I can confirm that after defining the above macro the offending file does compile (although wether it works is another thing I haven't tested). I cannot confirm that the build continues, though, since the porterbox I'm logged to doesn't have all the required build depends. I've pushed a patch recommended by Tim, who developed boost_lockfree. The patch simply updates that file to the latest version in boost's SVN, which as you'll see has the effect of simply defining that macro. I don't have access to machines for testing this, so I'd be grateful if anyone could test. The patch looks very similar to what I did to build, so at least the build failure should be corrected. I will test, but probably during the weekend. If someone else can test first, it would be appreciated. -- Saludos, Felipe Sateler ___ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers
Re: Supercollider fails to build
On Mon, Jul 2, 2012 at 12:46 PM, Felipe Sateler fsate...@gmail.com wrote: Supercollider is failing to build everywhere except i386 and amd64. Boost.Lockfree seems to be the culprit everywhere. I think the problem is that BOOST_LOCKFREE_DCAS_ALIGNMENT is only defined in a few specific cases (for x86 and x64). Dan, could you check with upstream what could be causing this? I can confirm that after defining the above macro the offending file does compile (although wether it works is another thing I haven't tested). I cannot confirm that the build continues, though, since the porterbox I'm logged to doesn't have all the required build depends. -- Saludos, Felipe Sateler ___ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/pkg-multimedia-maintainers
