2010/10/31 Felipe Sateler <fsate...@debian.org>:
> Артём, you are CCed because I don't know if you are subscribed to the list.
>
> On Sat, Oct 16, 2010 at 11:09, Dan S <danstowell+de...@gmail.com> wrote:
>>
>> 2010/10/6 Felipe Sateler <fsate...@debian.org>:
>> > Hi all, (Dan CCed because I'm not sure if you are subscribed to the list)
>> >
>> > I managed to take a few minutes to take a look at the package and it is
>> > not in a very good shape (it was still using simple-patchsys!). I have
>> > worked a bit on it, but it still has a long way to go. I will try to
>> > work on it during this week, I think I can find one hour or two.
>> >
>> > Dan, as you are part of upstream, could you comment on the patches
>> > included in the packaging? I can see they are in upstream svn ubuntu
>> > packaging module, with yourself as last commiter on most of them. They
>> > are older than the latest sc release, though.
>>
>> Hi - sorry for slow reply, I missed this thread. Most of the patches
>> are by other people but I will try to comment:
>>
>> 02_disable_elisp_compilation.diff
>>  - not sure I'm afraid, it's connected with the emacs sc3 mode, which
>> I don't use. Looks like it might disable something from running simply
>> because the build machine isn't the target machine.
>>
>> 03_fix_elisp_install_path.diff
>>  - another emacs mode thing, not sure.
>
> Does anyone use emacs and can comment on wether this should be applied
> upstream? The patch changes the elisp install path from
> /usr/share/emacs/site-lisp to /usr/share/emacs/site-lisp/supercollider
>
>>
>> 06_deb_scvim.diff
>>  - build machine != target machine so don't error out if no ruby executable
>>
>> 07_deb_sced.diff
>>  - build machine != target machine so don't modify machine's mime database
>
> Dan, can you comment on whether these can be upstreamed? I don't see
> why they should be debian-specific.

Right, that makes sense. I've had a look at the scons scripts and not
been able to integrate them in neatly (was hoping to add a nice option
for not-installing-here - if anyone has the scons chops to suggest
something then please do, I'd be grateful.)

FYI, supercollider 3.4.1 (bugfix release) has just been agreed, so is
likely to come out very very soon without any further upstreaming. I
hope that doesn't get in the way of debianising... would these patches
be considered blocking issues, do you think?

Dan


>> 08_deb_scvim_checkactive.diff
>>  - the 'scvim' script is a wrapper which launches vim with the sc3
>> plugin. debian vim-addons policy requires the plugin not to be
>> auto-enabled when installing, so we must modify the scvim script to
>> detect if the addon is not enabled, and to help the user enable it if
>> they want to. (Needs to be a patch since vim-addon-manager not
>> necessarily used on all linuxes)
>
> This one has been upstreamed, great!
>
>>
>> 10_fix_rpath.diff
>>  - I'm not clear on why the change from -rpath to -rpath-link has to
>> be done here as a patch. cc'ing this message to Артём who may be able
>> to comment on this, and possibly others.
>
> This looks like it could be upstreamed too?
>
>
>
>
> --
>
> Saludos,
> Felipe Sateler
>
> _______________________________________________
> pkg-multimedia-maintainers mailing list
> pkg-multimedia-maintainers@lists.alioth.debian.org
> http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers
>

_______________________________________________
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers

Reply via email to