svn commit: r530685 - in head/net-mgmt/netdata: . files

[Mateusz Piotrowski]

On 4/4/20 5:41 PM, Matthias Andree wrote:
> Author: mandree
> Date: Sat Apr  4 15:41:26 2020
> New Revision: 530685
> URL: https://svnweb.freebsd.org/changeset/ports/530685
>
> Log:
>    net-mgmt/netdata rc script overhaul, fix termination, user config
...
> Modified: head/net-mgmt/netdata/files/netdata.in
> ==============================================================================
> --- head/net-mgmt/netdata/files/netdata.in	Sat Apr  4 15:37:31 2020	(r530684)
> +++ head/net-mgmt/netdata/files/netdata.in	Sat Apr  4 15:41:26 2020	(r530685)
> @@ -13,24 +13,111 @@
...
> +# check if the rc.conf* file set and netdata_conf agree on the name to run the bundle as
> +check_user_consistency() {
> +	cfusr=$(get_user_from_conf)
> +	if [ -z "$cfusr" -o "$cfusr" = "${netdata_user}" ] ; then return 0 ; fi
> +	echo >&2 "$0: ERROR: ${netdata_conf} sets 'run as user = $cfusr', but rc.conf* sets '${netdata_user}'. Both must be consistent."
Could this use err() from rc.subr(8) instead?