[v2] package/wpa_supplicant: Fix systemd service stop

Fix systemd service stop when using only new DBus bus name.

By default systemd service description comes with old DBus bus name for
wpa_supplicant. If only new bus name is used, then systemd stops
wpa_supplicant.service, because it has not started old bus name with
specified time. In order to suppress stopping wpa_supplicant.service in
such case, we just rename BusName property to the new one, so systemd
sees that the service has started properly.

Signed-off-by: Marcin Niestroj <m.niestroj@grinn-global.com>
---
 package/wpa_supplicant/wpa_supplicant.mk | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

Subject line is not entirely accurate, as far as I understand, it's not really 
started so I think it should just be "Fix systemd service for new DBus bus name".

On 04/08/16 15:59, Marcin Niestroj wrote:
> Fix systemd service stop when using only new DBus bus name.

  This repetition of the subject line is not needed.

>
> By default systemd service description comes with old DBus bus name for
> wpa_supplicant. If only new bus name is used, then systemd stops
> wpa_supplicant.service, because it has not started old bus name with
> specified time. In order to suppress stopping wpa_supplicant.service in
> such case, we just rename BusName property to the new one, so systemd
> sees that the service has started properly.

  I don't think this is very clear, let me try to rephrase it.

By default, the systemd service supplied with the wpa_supplicant package comes 
with the old DBus bus name. However, when the new bus name is used by 
wpa_supplicant, systemd will not be able to detect its appearance on the bus. 
Therefore it will terminate the service again after some timeout, because 
systemd assumes it is malfunctioning.

To fix this, make sure that the correct BusName property is used.


  Otherwise looks good to me. However, I'd prefer to have an upstreamable patch 
instead that uses the CONFIG_CTRL_IFACE_DBUS_NEW config knob to set it correctly 
in wpa_supplicant/Makefile.

>
> Signed-off-by: Marcin Niestroj <m.niestroj@grinn-global.com>
> ---
>   package/wpa_supplicant/wpa_supplicant.mk | 14 ++++++++++++++
>   1 file changed, 14 insertions(+)
>
> diff --git a/package/wpa_supplicant/wpa_supplicant.mk b/package/wpa_supplicant/wpa_supplicant.mk
> index ba1180d..7d4ab42 100644
> --- a/package/wpa_supplicant/wpa_supplicant.mk
> +++ b/package/wpa_supplicant/wpa_supplicant.mk
> @@ -193,6 +193,19 @@ define WPA_SUPPLICANT_INSTALL_TARGET_CMDS
>   	$(WPA_SUPPLICANT_INSTALL_WPA_CLIENT_SO)
>   endef
>
> +# By default systemd service description comes with old DBus bus name for
> +# wpa_supplicant. If only new bus name is used, then systemd stops
> +# wpa_supplicant.service, because it has not started old bus name with
> +# specified time.
> +# To suppress stopping wpa_supplicant.service in such case, just rename
> +# BusName property to the new one.

  The comment should be clarified as well.

  Regards,
  Arnout

> +ifeq ($(BR2_PACKAGE_WPA_SUPPLICANT_DBUS_OLD),)
> +define WPA_SUPPLICANT_UPDATE_DBUS_NAME
> +	sed -i -e 's/$(WPA_SUPPLICANT_DBUS_OLD_SERVICE)/$(WPA_SUPPLICANT_DBUS_NEW_SERVICE)/g' \
> +		$(TARGET_DIR)/usr/lib/systemd/system/wpa_supplicant.service
> +endef
> +endif
> +
>   define WPA_SUPPLICANT_INSTALL_INIT_SYSTEMD
>   	$(INSTALL) -m 0644 -D $(@D)/$(WPA_SUPPLICANT_SUBDIR)/systemd/wpa_supplicant.service \
>   		$(TARGET_DIR)/usr/lib/systemd/system/wpa_supplicant.service
> @@ -202,6 +215,7 @@ define WPA_SUPPLICANT_INSTALL_INIT_SYSTEMD
>   		$(TARGET_DIR)/usr/lib/systemd/system/wpa_supplicant-nl80211@.service
>   	$(INSTALL) -m 0644 -D $(@D)/$(WPA_SUPPLICANT_SUBDIR)/systemd/wpa_supplicant-wired@.service \
>   		$(TARGET_DIR)/usr/lib/systemd/system/wpa_supplicant-wired@.service
> +	$(WPA_SUPPLICANT_UPDATE_DBUS_NAME)
>   endef
>
>   $(eval $(generic-package))
>