Request for upload: supercollider 3.5.2 new version

[Dan S]

2012/5/14 Felipe Sateler <fsateler at debian.org>:
> On Sun, May 13, 2012 at 1:30 PM, Dan S <danstowell+debmm at gmail.com> wrote:
>> 2012/5/10 Felipe Sateler <fsateler at debian.org>:
>>> On Thu, May 10, 2012 at 4:41 AM, Dan S <danstowell+debmm at gmail.com> wrote:
>>>> 2012/5/9 Felipe Sateler <fsateler at debian.org>:
>>>>> On Wed, May 9, 2012 at 11:13 AM, Felipe Sateler <fsateler at debian.org> wrote:
>>>>>> On Wed, May 9, 2012 at 10:06 AM, Dan S <danstowell+debmm at gmail.com> wrote:
>>>>>>> Hi -
>>>>>>>
>>>>>>> SuperCollider 3.5.x is now out - 3.5.2 just released. (Curently the
>>>>>>> sid package is 3.4.5.) I've imported it into pkg-multimedia git.
>>>>>>>
>>>>>>> I'd be grateful if someone could have a look, and upload if ok:
>>>>>>> <http://anonscm.debian.org/gitweb/?p=pkg-multimedia/supercollider.git;a=summary>
>>>>>>
>>>>>> I'm taking a look. Good to see scons is gone :)
>>>>>
>>>>> There are a whole bunch of external libraries. In debian we should use
>>>>> the system libraries whenever possible, and it seems like a lot of the
>>>>> libraries are already included in debian. Can't we use them?
>>>>
>>>> There are different statuses for different items in the
>>>> "external_libraries" folder:
>>>>
>>>>  * although headers are included in the source download, on linux the
>>>> system headers&libs are used for curl and libsndfile.
>>>
>>> Good.
>>>
>>>>
>>>>  * boost_endian and boost_lockfree are not in core boost yet, so we
>>>> include them. (They are header-only libs.)
>>>
>>> OK, but copyright must be documented for these libraries. It might be
>>> worth talking to the boost maintainers, maybe they are willing to
>>> package non-core libs too.
>>
>> Pushed a copyright-file update covering all these.
>>
>>>>  * oscpack and tlsf are statically built into the new "supernova"
>>>> executable. TLSF is not available as a package in debian; and the
>>>> version of oscpack is slightly patched for array boundary type tags.
>>>
>>> Is the patch disruptive? We could ask the debian maintainer to add it
>>> and link to the system library. I'm assuming the patch has been
>>> forwarded to the oscpack upstream.
>>
>> The patch allows it to handle array-type values. The patch was
>> upstreamed, but the upstream maintainer wanted to support the array
>> boundaries differently (maybe differently from how supercollider
>> does...). This suggests we have to keep a patched version.
>>
>>>>  * nova-simd and nova-tt are libraries developed by tim blechmann (who
>>>> also develops supercollider), and don't yet have debian packages.
>>>> (They are header-only libs.)
>>>
>>> OK, so they are almost not-external libraries?
>>
>> That's one way of thinking of it. They are potentially useful to
>> others though, so I'm encouraging the author to think about making
>> debs of them. That's for the future.
>>
>>>>  * these don't have a deb package: threadpool, yaml-cpp.
>>>
>>> Threadpool seems to be header-only too (I'm not on my debian machine
>>> here). Yaml-cpp is not. Is yaml-cpp useful for others? If so, it
>>> should be packaged on its own. There is another yaml parser/emitter in
>>> debian, but it may be too much effort to port to that.
>>
>> There has been someone looking at packaging yaml-cpp:
>> <http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=636985> though the
>> most recent activity was a few months ago.
>>
>>>>  * these ones are unused on linux: simplejson (manually installed on
>>>> windows), icu (headers only used on mac).
>>>
>>> Good.
>>>
>>>>
>>>> System boost is not used in 3.5.2, but there is a commit in the
>>>> development version which enables this:
>>>> <https://github.com/supercollider/supercollider/commit/9913b2d92f>
>>>> We could backport this commit as a patch?
>>>
>>> Yes please.
>>
>> OK, done and pushed. Thanks both for the comments.
>
>
> I'm trying building and it is not using the system boost lib...

Which version of boost lib do you have installed? I think I need to
change the requirement (to >=1.48 not >=1.43, and a couple of libs I
missed off the list). My fault, sorry - I've bumped up the dependency
and pushed this now.

Dan