SPECS: mysqlstat.spec - with user mysqlstat

[Andrzej Krzysztofowicz]

glen wrote:
> +Requires:	group(http)

Hmmm... daes this package require a webserver?
If no, I suggest P: group(http) and try to add it in %pre
If requires, it is not enough: R(pre,postun) might be necessary.
Or even PreReq. ((un)installation sequence while installing many packages at
once matters, IMO)

> +%pre
> +[ "`/bin/id -u mysqlstat 2>/dev/null`" ] || \
> +	/usr/sbin/useradd -u %{userid} -d /usr/share/mysqlstat \
> +		-s /bin/false -g http -c "MySQL Statistics" mysqlstat

Eg.: you cannot remove a group being primary group of a user...

> -%attr(700,root,root) %dir %{_sysconfdir}
> -%attr(600,root,root) %config(noreplace) %verify(not size mtime md5) %{_sysconfdir}/*
> -%dir %attr(710,root,http) /var/lib/%{name}
> +%attr(700,mysqlstat,root) %dir %{_sysconfdir}
> +%attr(600,mysqlstat,root) %config(noreplace) %verify(not size mtime md5) %{_sysconfdir}/*

What is the role of this user? Why does he need write access to config files?

> +%dir %attr(710,mysqlstat,http) /var/lib/%{name}

-- 
=======================================================================
  Andrzej M. Krzysztofowicz               ankry at mif.pg.gda.pl
  phone (48)(58) 347 14 61
Faculty of Applied Phys. & Math.,   Gdansk University of Technology