[Buildroot] [PATCH] optional DBus support in wpa_supplicant

[Peter Korsgaard]

>>>>> "Sven" == Sven Neumann <s.neumann at raumfeld.com> writes:

 Sven> Hi,
 Sven> attached is an updated patch that enables the DBus config interface in
 Sven> wpa-supplicant if DBus is enabled in buildroot without introducing
 Sven> another config option.

Thanks!

 Sven> --- ../buildroot-orig/package/wpa_supplicant/wpa_supplicant.mk	2009-07-12 21:13:04.000000000 +0200
 Sven> +++ package/wpa_supplicant/wpa_supplicant.mk	2009-07-13 11:07:52.000000000 +0200
 Sven> @@ -13,10 +13,14 @@
 Sven>  WPA_SUPPLICANT_CONFIG = $(WPA_SUPPLICANT_DIR)/wpa_supplicant/.config
 Sven>  WPA_SUPPLICANT_SUBDIR = wpa_supplicant
 Sven>  WPA_SUPPLICANT_TARGET_BINS = wpa_cli wpa_supplicant wpa_passphrase
 Sven> +WPA_SUPPLICANT_DBUS_SERVICE = fi.epitest.hostap.WPASupplicant

Aren't you missing a ".service" here? Service files seems to normally
have the .service extension, and the wpa supplicant file on my machine
(Debian) does as well.

 Sven> +	$(INSTALL) -D \
 Sven> +	  $(WPA_SUPPLICANT_DIR)/wpa_supplicant/dbus-wpa_supplicant.service \
 Sven> +	  $(TARGET_DIR)/usr/share/dbus-1/system-services/$(WPA_SUPPLICANT_DBUS_SERVICE)

Here you don't add it
 
 Sven> +	rm -f $(TARGET_DIR)/usr/share/dbus-1/system-services/$(WPA_SUPPLICANT_DBUS_SERVICE).service

But here you do.

-- 
Bye, Peter Korsgaard