package pkg-multimedia/crtmpserver looking for mentor(s)

Reinhard Tartler siretart at tauware.de
Fri Apr 15 19:38:42 UTC 2011


On Fri, Apr 15, 2011 at 14:32:34 (CEST), Andriy Beregovenko wrote:

> Hi.
>
> Any news ? :)

please don't top quote

> понеділок 11 квітень 2011 23:58:42 Reinhard Tartler ви написали:
>> On Wed, Apr 06, 2011 at 18:40:12 (CEST), Reinhard Tartler wrote:
>> > On Wed, Apr 06, 2011 at 16:02:26 (CEST), Andriy Beregovenko wrote:
>> >> Hi,
>> >> 
>> >> Still looking for uploader :)
>> >> As I understand package now is clean.
>> > 
>> > I'll try to have a look tonight or tomorrow evening.
>> 
>> It seems that Alessio is already on it. Alessio, do you want to take
>> over?
>> 
>> I took a brief look and found the system user creation a bit
>> strange. If you created the user in postinst, you wouldn't need the
>> pre-dependency on adduser. And moreover, since the default configuration
>> doesn't start the daemon anyways, I'd rather skip the system user
>> creation completely and install the crtmpserver.lua config file in
>> /usr/share/doc/crtmpserver/examples instead of /etc/crtmpserver along
>> with some instructions to the users how to start the daemon.
>> 
>> This will save you a lot of headaches and bugreports.

I had another look at the package. In debian/crtmpserver.install, the
package installs crtmpserver/crtmpserver.lua ->
etc/crtmpserver. However, this configuration file seems horribly
specific to me; i.e., we can safely expect that every user *will*
definitly want to edit and adjust the contents. This also means that
dpkg *will* prompt on every package upgrade that changes anything in the
file.

Sorry, the more I think about it, I come to the conclusion that

 a) we should not install the config file to /etc/
 b) not install the init script
 c) not create the system user

This will reduce a lot of complexity in the package, which wouldn't help
our users much anyway.

-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4



More information about the pkg-multimedia-maintainers mailing list