понеділок 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. 
Has no reason to finish package install if user can not be created(or not 
already exist). That is why I put user creation in preinst. Also adduser dep 
was added looking into other packages. Need I remove this dep ?

> And moreover, since the default configuration
> doesn't start the daemon anyways, 
Hm... By default after install, daemon must start. This works for me and some 
another test platforms. So if it not start this is bug. In that way please 
describe me a little you situation about this.

> 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.
> The file debian/source/local-options is missing, see the wiki for
> details

> In debian/rules, where do the variables ${SVER} and ${DEBPATH} come from?
This was used by me, at time of build in my env. Of course this is 
unacceptable for package at all. So I move ${SVER} to static string in rules,
and ${DEBPATH} to $(shell pwd). 
Also I understand that "$(shell pwd)" is exactly wrong form, but right now I 
do not see any way to provide value for CRMPSERVER_SOURCE_ROOT variable. If 
you know how, please tell me(CRMPSERVER_SOURCE_ROOT must be pointed to package 
source root).

pkg-multimedia-maintainers mailing list

Reply via email to