Hi Kilian,
Thanks for your review.
The file debian/po/templates.pot has a lot of template headers still
though.
Please fill in all fields that are still holding bogus data.
Done. Also added Catalan and Spanish translations.
However, every time I run debconf-updatepo this file gets overwritten
and so the headers. Is there a way to update this file without losing
the headers?
Regarding the activation I'd still vote for a group to be created and the
chmod to be 4750 if SUID and 0755 if not SUID. You may want to use
dpkg-statoverride for this to set both user:group and chmod
permissions in
one line.
If you need a good template I'd need to dig among the last packages I
have
reviewed. There was a really good postinst doing exactly this.
I tried the dpkg-override way (I took mlocate package as reference) but
I didn't find a proper way to reset ownership/permissions when the user
chooses not SUID without resorting to chown/chmod commands. Thus I left
the chown/chmod commands.
Your text then should also include the name of the group (probably
eviacam)
and that the sysadmin should add users if they're supposed to use the
program.
Done.
b) which users should be added to the group interactively
I would need some help here. Can you point a good document (or
better, an example) on how to interactively add users to a group
using debconf?
I was thinking of the libc version asking for which processes need to be
restarted. Not sure though if that's useful considering that e.g. sudo
leaves this to the sysadmin too. I guess we can live with just the
SUID-yes
and SUID-no question in debconf.
OK. Fine for me also.
Unfortunately, due to problems with mentors site (I've already contacted
the support team to find a solution) I haven't been able to upload the
new changes. Once the problems are sorted out I'll ping you again. In
the meantime you might find the latest changes in the git repo
(sorry for the inconvenience):
http://eviacam.git.sourceforge.net/git/gitweb-index.cgi
Best regards!
Cesar