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 _______________________________________________ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers
