[Buildroot] [PATCH v2] modem-manager: support a defaults file in the init script

[Petr Vorel <petr.vorel@gmail.com>]

Hi Carlos,

> Sometimes it is useful to pass some parameters to ModemManager when it
> starts (e.g. --log-level). Allow the user add a file with such flags in
> a MODEMMANAGER_ARGS variable. This is simpler than overriding the whole
> startup script (e.g. by means of a rootfs overlay).

> Signed-off-by: Carlos Santos <casantos@datacom.ind.br>
> ---
> Changes v1->v2
>   - Rebase after commit ef0b7e7521 (modem-manager: don't test if the
>     binary exists in the init script).
> ---
>  package/modem-manager/S44modem-manager | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)

> diff --git a/package/modem-manager/S44modem-manager b/package/modem-manager/S44modem-manager
> index a3f5003aa1..416c1e2ee6 100755
> --- a/package/modem-manager/S44modem-manager
> +++ b/package/modem-manager/S44modem-manager
> @@ -3,13 +3,16 @@
>  # Starts ModemManager


> +# Allow a few customizations from a config file
> +test -r /etc/default/ModemManager && . /etc/default/ModemManager
> +
Out of curiosity, what is supposed to be in /etc/default/ModemManager ?

>  PIDFILE=/var/run/ModemManager.pid

>  start() {
>  	printf "Starting ModemManager: "
>  	umask 077
>  	start-stop-daemon -S -q -b -m -p $PIDFILE \
> -		--exec /usr/sbin/ModemManager
> +		--exec /usr/sbin/ModemManager -- $MODEMMANAGER_ARGS
LGTM.

>  	[ $? = 0 ] && echo "OK" || echo "FAIL"
>  }
>  stop() {


Kind regards,
Petr