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