Re: coturn: specify pidfile and create directory in rc_pre

On 2023/03/13 10:20, Andre Stoebe wrote:
> Hey,
>
> I just remembered this issue is still open. Sorry for not following up
> the last time.
>
> So here is a diff without all the fancy variables and such.
>
> I'm running with this change locally since then, but it gets overridden
> on upgrade. It would be great if this could make it into the release.

Thanks, committed.

> Thanks a lot.
>
> Regards,
> Andre
>
> --- telephony/coturn/Makefile
> +++ telephony/coturn/Makefile
> @@ -2,7 +2,7 @@ REVISION = 3
>
> V = 4.5.2
> DISTNAME = turnserver-${V}
> -REVISION = 3
> +REVISION = 4
>
> CATEGORIES = telephony
>
> --- telephony/coturn/pkg/turnserver.rc
> +++ telephony/coturn/pkg/turnserver.rc
> @@ -1,10 +1,15 @@
> #!/bin/ksh
>
> daemon="${TRUEPREFIX}/bin/turnserver --daemon"
> +daemon_flags="--pidfile=/var/run/turnserver/turnserver.pid"
> daemon_user="_turnserver"
>
> . /etc/rc.d/rc.subr
>
> rc_reload=NO
>
> +rc_pre() {
> + install -d -o ${daemon_user} /var/run/turnserver
> +}
> +
> rc_cmd $1
>