Request for upload: supercollider 3.5.2 new version

[Felipe Sateler]

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.


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


>
>  * 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?

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

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


-- 

Saludos,
Felipe Sateler