Request for upload: supercollider 3.5.2 new version

[Felipe Sateler]

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...

-- 

Saludos,
Felipe Sateler