Received: from vps892.directvps.nl (ikke.info [178.21.113.177])
	by nld3-dev1.alpinelinux.org (Postfix) with ESMTPS id 0C43C781A67
	for <alpine-aports@lists.alpinelinux.org>; Mon,  9 Mar 2020 19:26:51 +0000 (UTC)
Received: by vps892.directvps.nl (Postfix, from userid 1008)
	id 5382444010F; Mon,  9 Mar 2020 20:26:49 +0100 (CET)
Date: Mon, 9 Mar 2020 20:26:49 +0100
From: Kevin Daudt <kdaudt@alpinelinux.org>
To: Stefano Sasso <stesasso@gmail.com>
Cc: alpine-aports@lists.alpinelinux.org
Subject: Re: [PATCH] main/openvpn: Add instance name to OpenVPN --daemon param
Message-ID: <20200309192649.GA1380825@alpha>
References: <1583505286-12723-1-git-send-email-stesasso@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <1583505286-12723-1-git-send-email-stesasso@gmail.com>

On Fri, Mar 06, 2020 at 03:34:46PM +0100, Stefano Sasso wrote:
> The --daemon OpenVPN params also controls the tag OpenVPN uses to ship
> logs to syslog. If multiple OpenVPN instances are used, the user may
> want to apply syslog filters, to have different log files for the
> different instances.
> If you specify an additional --daemon option in the config file, it will
> be ignored, since the one on the command line is preferred.
> ---
>  main/openvpn/openvpn.initd | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/main/openvpn/openvpn.initd b/main/openvpn/openvpn.initd
> index 8c11106..085ee5e 100644
> --- a/main/openvpn/openvpn.initd
> +++ b/main/openvpn/openvpn.initd
> @@ -22,7 +22,7 @@ instance_name=${RC_SVCNAME#*.}
>  pidfile="/run/$RC_SVCNAME.pid"
>  command="/usr/sbin/openvpn"
>  command_args="
> -	--daemon
> +	--daemon $instance_name
>  	--config $cfgfile
>  	--writepid $pidfile
>  	--setenv RC_SVCNAME $RC_SVCNAME
> -- 
> 1.8.3.1

This has been applied. Note that you forgot to bump the pkgrel and fix
the checksums.