From mboxrd@z Thu Jan  1 00:00:00 1970
From: Holger Sunke <holger.sunke@posteo.de>
To: development@lists.ipfire.org
Subject:
 Re: [RFC PATCH] initscripts: Add switch to start processes in background
Date: Wed, 28 Jul 2021 17:31:47 +0000
Message-ID: <5811d6b8-80c2-6ae5-5ac6-bc73e566e07f@posteo.de>
In-Reply-To: <20210727085900.2253-1-michael.tremer@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============8776067469148987077=="
List-Id: <development.lists.ipfire.org>

--===============8776067469148987077==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

This works for me, thank you.

Am 27.07.21 um 10:59 schrieb Michael Tremer:
> Since systemd, many programs no longer behave like a well-behaved
> daemon. To avoid any extra solutions, this patch adds a -b switch which
> will start a program in the background and throw away any output.
>=20
> The behaviour remains unchanged for any other programs.
>=20
> Signed-off-by: Michael Tremer <michael.tremer(a)ipfire.org>
> ---
>   src/initscripts/system/functions | 17 +++++++++++++++--
>   1 file changed, 15 insertions(+), 2 deletions(-)
>=20
> diff --git a/src/initscripts/system/functions b/src/initscripts/system/func=
tions
> index d610a524d..e44a2b4a1 100644
> --- a/src/initscripts/system/functions
> +++ b/src/initscripts/system/functions
> @@ -436,6 +436,7 @@ getpids()
>   #************************************************************************=
*******
>   loadproc()
>   {
> +	local background=3D""
>   	local pidfile=3D""
>   	local forcestart=3D""
>   	local nicelevel=3D""
> @@ -448,6 +449,10 @@ loadproc()
>     while true
>   	do
>   		case "${1}" in
> +			-b)
> +				background=3D"1"
> +				shift 1
> +				;;
>   			-f)
>   				forcestart=3D"1"
>   				shift 1
> @@ -506,8 +511,16 @@ loadproc()
>   		cmd=3D"nice -n "${nicelevel}" ${cmd}"
>   	fi
>  =20
> -	${cmd}
> -	evaluate_retval # This is "Probably" not LSB compliant, but required to b=
e compatible with older bootscripts
> +	if [ -n "${background}" ]; then
> +		(
> +			${cmd} &>/dev/null
> +		) &
> +		evaluate_retval
> +	else
> +		${cmd}
> +		evaluate_retval # This is "Probably" not LSB compliant, but required to =
be compatible with older bootscripts
> +	fi
> +
>   	return 0
>   }
>  =20
>=20

--===============8776067469148987077==--