* [Buildroot] [PATCH v3] package/wpa_supplicant: Fix systemd service for new DBus bus name @ 2016-04-12 15:35 Marcin Niestroj 2016-04-12 21:47 ` Arnout Vandecappelle 0 siblings, 1 reply; 5+ messages in thread From: Marcin Niestroj @ 2016-04-12 15:35 UTC (permalink / raw) To: buildroot systemd service files were supplied with old DBus bus name. After service activation systemd was waiting for appearance of specified bus name to consider it started successfully. However, if wpa_supplicant was compiled only with new dbus interface name, then systemd didn't notice configured (old) DBus bus name appearance. In the end service was considered malfunctioning and it was deactivated. Add upstream patch, which updates systemd service BusName property according to configured DBus interface version. Signed-off-by: Marcin Niestroj <m.niestroj@grinn-global.com> --- Hi, Sorry for posting this patch as v2 moment ago. I'm just resending it with proper version (v3) and updated changelog. Changes v2 -> v3: * Update subject line and commit description to make the problem more clear (suggested by Arnout) * Use upstreamable patch instead of change in buildroot package makefile (suggested by Arnout) Changes v1 -> v2: * Added missing signed-off tag ...ate-service-files-according-to-dbus-inter.patch | 77 ++++++++++++++++++++++ 1 file changed, 77 insertions(+) create mode 100644 package/wpa_supplicant/0007-systemd-Update-service-files-according-to-dbus-inter.patch diff --git a/package/wpa_supplicant/0007-systemd-Update-service-files-according-to-dbus-inter.patch b/package/wpa_supplicant/0007-systemd-Update-service-files-according-to-dbus-inter.patch new file mode 100644 index 0000000..59c5450 --- /dev/null +++ b/package/wpa_supplicant/0007-systemd-Update-service-files-according-to-dbus-inter.patch @@ -0,0 +1,77 @@ +From b6cea24d6191d9ccdcd1ac38a5322e3da73218db Mon Sep 17 00:00:00 2001 +From: Marcin Niestroj <m.niestroj@grinn-global.com> +Date: Mon, 11 Apr 2016 13:23:54 +0200 +Subject: [PATCH] systemd: Update service files according to dbus interface + version used + +systemd service files were supplied with old DBus bus name. After +service activation systemd was waiting for appearance of specified bus +name to consider it started successfully. However, if wpa_supplicant was +compiled only with new dbus interface name, then systemd didn't notice +configured (old) DBus bus name appearance. In the end service was +considered malfunctioning and it was deactivated. + +Update systemd service BusName property according to supported DBus +interface version. + +Signed-off-by: Marcin Niestroj <m.niestroj@grinn-global.com> +--- + wpa_supplicant/Makefile | 8 ++++++-- + wpa_supplicant/systemd/wpa_supplicant.service.in | 4 ++-- + 2 files changed, 8 insertions(+), 4 deletions(-) + +diff --git a/wpa_supplicant/Makefile b/wpa_supplicant/Makefile +index ad9ead9..fbd1b25 100644 +--- a/wpa_supplicant/Makefile ++++ b/wpa_supplicant/Makefile +@@ -1374,6 +1374,7 @@ ifndef DBUS_INCLUDE + DBUS_INCLUDE := $(shell $(PKG_CONFIG) --cflags dbus-1) + endif + DBUS_CFLAGS += $(DBUS_INCLUDE) ++DBUS_INTERFACE=fi.epitest.hostap.WPASupplicant + endif + + ifdef CONFIG_CTRL_IFACE_DBUS_NEW +@@ -1399,6 +1400,7 @@ DBUS_OBJS += dbus/dbus_new_introspect.o + DBUS_CFLAGS += -DCONFIG_CTRL_IFACE_DBUS_INTRO + endif + DBUS_CFLAGS += $(DBUS_INCLUDE) ++DBUS_INTERFACE=fi.w1.wpa_supplicant1 + endif + + ifdef DBUS +@@ -1760,11 +1762,13 @@ else + endif + + %.service: %.service.in +- $(Q)sed -e 's|\@BINDIR\@|$(BINDIR)|g' $< >$@ ++ $(Q)sed -e 's|\@BINDIR\@|$(BINDIR)|g' \ ++ -e 's|\@DBUS_INTERFACE\@|$(DBUS_INTERFACE)|g' $< >$@ + @$(E) " sed" $< + + %@.service: %.service.arg.in +- $(Q)sed -e 's|\@BINDIR\@|$(BINDIR)|g' $< >$@ ++ $(Q)sed -e 's|\@BINDIR\@|$(BINDIR)|g' \ ++ -e 's|\@DBUS_INTERFACE\@|$(DBUS_INTERFACE)|g' $< >$@ + @$(E) " sed" $< + + wpa_supplicant.exe: wpa_supplicant +diff --git a/wpa_supplicant/systemd/wpa_supplicant.service.in b/wpa_supplicant/systemd/wpa_supplicant.service.in +index ea964ce..bc5d49a 100644 +--- a/wpa_supplicant/systemd/wpa_supplicant.service.in ++++ b/wpa_supplicant/systemd/wpa_supplicant.service.in +@@ -5,9 +5,9 @@ Wants=network.target + + [Service] + Type=dbus +-BusName=fi.epitest.hostap.WPASupplicant ++BusName=@DBUS_INTERFACE@ + ExecStart=@BINDIR@/wpa_supplicant -u + + [Install] + WantedBy=multi-user.target +-Alias=dbus-fi.epitest.hostap.WPASupplicant.service ++Alias=dbus-@DBUS_INTERFACE@.service +-- +2.8.0 + -- 2.8.0 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH v3] package/wpa_supplicant: Fix systemd service for new DBus bus name 2016-04-12 15:35 [Buildroot] [PATCH v3] package/wpa_supplicant: Fix systemd service for new DBus bus name Marcin Niestroj @ 2016-04-12 21:47 ` Arnout Vandecappelle 2016-04-13 4:57 ` Baruch Siach 2016-04-13 8:01 ` Marcin Niestroj 0 siblings, 2 replies; 5+ messages in thread From: Arnout Vandecappelle @ 2016-04-12 21:47 UTC (permalink / raw) To: buildroot On 04/12/16 17:35, Marcin Niestroj wrote: > systemd service files were supplied with old DBus bus name. After > service activation systemd was waiting for appearance of specified bus > name to consider it started successfully. However, if wpa_supplicant was > compiled only with new dbus interface name, then systemd didn't notice > configured (old) DBus bus name appearance. In the end service was > considered malfunctioning and it was deactivated. > > Add upstream patch, which updates systemd service BusName property It's not an upstream patch, it's an upstreamable patch. At least, I can't find it in the upstream git repo nor on the mailing list. Did you even send it upstream already? > according to configured DBus interface version. > > Signed-off-by: Marcin Niestroj <m.niestroj@grinn-global.com> > --- > Hi, > > Sorry for posting this patch as v2 moment ago. I'm just resending it > with proper version (v3) and updated changelog. > > Changes v2 -> v3: > * Update subject line and commit description to make the problem more > clear (suggested by Arnout) > * Use upstreamable patch instead of change in buildroot package > makefile (suggested by Arnout) > > Changes v1 -> v2: > * Added missing signed-off tag > > ...ate-service-files-according-to-dbus-inter.patch | 77 ++++++++++++++++++++++ > 1 file changed, 77 insertions(+) > create mode 100644 package/wpa_supplicant/0007-systemd-Update-service-files-according-to-dbus-inter.patch > > diff --git a/package/wpa_supplicant/0007-systemd-Update-service-files-according-to-dbus-inter.patch b/package/wpa_supplicant/0007-systemd-Update-service-files-according-to-dbus-inter.patch > new file mode 100644 > index 0000000..59c5450 > --- /dev/null > +++ b/package/wpa_supplicant/0007-systemd-Update-service-files-according-to-dbus-inter.patch > @@ -0,0 +1,77 @@ > +From b6cea24d6191d9ccdcd1ac38a5322e3da73218db Mon Sep 17 00:00:00 2001 > +From: Marcin Niestroj <m.niestroj@grinn-global.com> > +Date: Mon, 11 Apr 2016 13:23:54 +0200 > +Subject: [PATCH] systemd: Update service files according to dbus interface > + version used > + > +systemd service files were supplied with old DBus bus name. After > +service activation systemd was waiting for appearance of specified bus > +name to consider it started successfully. However, if wpa_supplicant was > +compiled only with new dbus interface name, then systemd didn't notice > +configured (old) DBus bus name appearance. In the end service was > +considered malfunctioning and it was deactivated. > + > +Update systemd service BusName property according to supported DBus > +interface version. If you send this patch upstream before posting a v4, please include a reference to the upstream mailing list message (or better yet, the commit :-) > + > +Signed-off-by: Marcin Niestroj <m.niestroj@grinn-global.com> > +--- > + wpa_supplicant/Makefile | 8 ++++++-- > + wpa_supplicant/systemd/wpa_supplicant.service.in | 4 ++-- > + 2 files changed, 8 insertions(+), 4 deletions(-) > + > +diff --git a/wpa_supplicant/Makefile b/wpa_supplicant/Makefile > +index ad9ead9..fbd1b25 100644 > +--- a/wpa_supplicant/Makefile > ++++ b/wpa_supplicant/Makefile > +@@ -1374,6 +1374,7 @@ ifndef DBUS_INCLUDE > + DBUS_INCLUDE := $(shell $(PKG_CONFIG) --cflags dbus-1) > + endif > + DBUS_CFLAGS += $(DBUS_INCLUDE) > ++DBUS_INTERFACE=fi.epitest.hostap.WPASupplicant Coding style in this file is with spaces around the = Otherwise, the patch looks good to me. Since the spaces are just a triviality: Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> Regards, Arnout > + endif > + > + ifdef CONFIG_CTRL_IFACE_DBUS_NEW > +@@ -1399,6 +1400,7 @@ DBUS_OBJS += dbus/dbus_new_introspect.o > + DBUS_CFLAGS += -DCONFIG_CTRL_IFACE_DBUS_INTRO > + endif > + DBUS_CFLAGS += $(DBUS_INCLUDE) > ++DBUS_INTERFACE=fi.w1.wpa_supplicant1 > + endif > + > + ifdef DBUS > +@@ -1760,11 +1762,13 @@ else > + endif > + > + %.service: %.service.in > +- $(Q)sed -e 's|\@BINDIR\@|$(BINDIR)|g' $< >$@ > ++ $(Q)sed -e 's|\@BINDIR\@|$(BINDIR)|g' \ > ++ -e 's|\@DBUS_INTERFACE\@|$(DBUS_INTERFACE)|g' $< >$@ > + @$(E) " sed" $< > + > + %@.service: %.service.arg.in > +- $(Q)sed -e 's|\@BINDIR\@|$(BINDIR)|g' $< >$@ > ++ $(Q)sed -e 's|\@BINDIR\@|$(BINDIR)|g' \ > ++ -e 's|\@DBUS_INTERFACE\@|$(DBUS_INTERFACE)|g' $< >$@ > + @$(E) " sed" $< > + > + wpa_supplicant.exe: wpa_supplicant > +diff --git a/wpa_supplicant/systemd/wpa_supplicant.service.in b/wpa_supplicant/systemd/wpa_supplicant.service.in > +index ea964ce..bc5d49a 100644 > +--- a/wpa_supplicant/systemd/wpa_supplicant.service.in > ++++ b/wpa_supplicant/systemd/wpa_supplicant.service.in > +@@ -5,9 +5,9 @@ Wants=network.target > + > + [Service] > + Type=dbus > +-BusName=fi.epitest.hostap.WPASupplicant > ++BusName=@DBUS_INTERFACE@ > + ExecStart=@BINDIR@/wpa_supplicant -u > + > + [Install] > + WantedBy=multi-user.target > +-Alias=dbus-fi.epitest.hostap.WPASupplicant.service > ++Alias=dbus- at DBUS_INTERFACE@.service > +-- > +2.8.0 > + > -- Arnout Vandecappelle arnout at mind be Senior Embedded Software Architect +32-16-286500 Essensium/Mind http://www.mind.be G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH v3] package/wpa_supplicant: Fix systemd service for new DBus bus name 2016-04-12 21:47 ` Arnout Vandecappelle @ 2016-04-13 4:57 ` Baruch Siach 2016-04-13 8:01 ` Marcin Niestroj 1 sibling, 0 replies; 5+ messages in thread From: Baruch Siach @ 2016-04-13 4:57 UTC (permalink / raw) To: buildroot Hi Arnout, On Tue, Apr 12, 2016 at 11:47:59PM +0200, Arnout Vandecappelle wrote: > On 04/12/16 17:35, Marcin Niestroj wrote: > >systemd service files were supplied with old DBus bus name. After > >service activation systemd was waiting for appearance of specified bus > >name to consider it started successfully. However, if wpa_supplicant was > >compiled only with new dbus interface name, then systemd didn't notice > >configured (old) DBus bus name appearance. In the end service was > >considered malfunctioning and it was deactivated. > > > >Add upstream patch, which updates systemd service BusName property > > It's not an upstream patch, it's an upstreamable patch. At least, I can't > find it in the upstream git repo nor on the mailing list. Did you even send > it upstream already? It's at http://lists.infradead.org/pipermail/hostap/2016-April/035599.html. baruch -- http://baruch.siach.name/blog/ ~. .~ Tk Open Systems =}------------------------------------------------ooO--U--Ooo------------{= - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il - ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH v3] package/wpa_supplicant: Fix systemd service for new DBus bus name 2016-04-12 21:47 ` Arnout Vandecappelle 2016-04-13 4:57 ` Baruch Siach @ 2016-04-13 8:01 ` Marcin Niestroj 2016-04-13 9:13 ` Arnout Vandecappelle 1 sibling, 1 reply; 5+ messages in thread From: Marcin Niestroj @ 2016-04-13 8:01 UTC (permalink / raw) To: buildroot Hi, On 12.04.2016 23:47, Arnout Vandecappelle wrote: > On 04/12/16 17:35, Marcin Niestroj wrote: >> systemd service files were supplied with old DBus bus name. After >> service activation systemd was waiting for appearance of specified bus >> name to consider it started successfully. However, if wpa_supplicant was >> compiled only with new dbus interface name, then systemd didn't notice >> configured (old) DBus bus name appearance. In the end service was >> considered malfunctioning and it was deactivated. >> >> Add upstream patch, which updates systemd service BusName property > > It's not an upstream patch, it's an upstreamable patch. At least, I > can't find it in the upstream git repo nor on the mailing list. Did you > even send it upstream already? Sorry for misleading here, I will update description. As Baruch wrote earlier, it's already been sent to mailing list. > >> @@ -0,0 +1,77 @@ >> +From b6cea24d6191d9ccdcd1ac38a5322e3da73218db Mon Sep 17 00:00:00 2001 >> +From: Marcin Niestroj <m.niestroj@grinn-global.com> >> +Date: Mon, 11 Apr 2016 13:23:54 +0200 >> +Subject: [PATCH] systemd: Update service files according to dbus >> interface >> + version used >> + >> +systemd service files were supplied with old DBus bus name. After >> +service activation systemd was waiting for appearance of specified bus >> +name to consider it started successfully. However, if wpa_supplicant was >> +compiled only with new dbus interface name, then systemd didn't notice >> +configured (old) DBus bus name appearance. In the end service was >> +considered malfunctioning and it was deactivated. >> + >> +Update systemd service BusName property according to supported DBus >> +interface version. > > If you send this patch upstream before posting a v4, please include a > reference to the upstream mailing list message (or better yet, the > commit :-) Thanks for pointing it, I will add the reference. > >> + >> +Signed-off-by: Marcin Niestroj <m.niestroj@grinn-global.com> >> +--- >> + wpa_supplicant/Makefile | 8 ++++++-- >> + wpa_supplicant/systemd/wpa_supplicant.service.in | 4 ++-- >> + 2 files changed, 8 insertions(+), 4 deletions(-) >> + >> +diff --git a/wpa_supplicant/Makefile b/wpa_supplicant/Makefile >> +index ad9ead9..fbd1b25 100644 >> +--- a/wpa_supplicant/Makefile >> ++++ b/wpa_supplicant/Makefile >> +@@ -1374,6 +1374,7 @@ ifndef DBUS_INCLUDE >> + DBUS_INCLUDE := $(shell $(PKG_CONFIG) --cflags dbus-1) >> + endif >> + DBUS_CFLAGS += $(DBUS_INCLUDE) >> ++DBUS_INTERFACE=fi.epitest.hostap.WPASupplicant > > Coding style in this file is with spaces around the = > > Otherwise, the patch looks good to me. Since the spaces are just a > triviality: > > Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> As I am newbie in mailing list workflow: does this tag mean, that I have permission to add it to v4 patch directly (if I fix all pointed issues)? Or it's just for other reviewers, so they know what has been already reviewed in previous version of patch? Regards, Marcin > > Regards, > Arnout > >> + endif >> + >> + ifdef CONFIG_CTRL_IFACE_DBUS_NEW >> +@@ -1399,6 +1400,7 @@ DBUS_OBJS += dbus/dbus_new_introspect.o >> + DBUS_CFLAGS += -DCONFIG_CTRL_IFACE_DBUS_INTRO >> + endif >> + DBUS_CFLAGS += $(DBUS_INCLUDE) >> ++DBUS_INTERFACE=fi.w1.wpa_supplicant1 >> + endif >> + >> + ifdef DBUS >> +@@ -1760,11 +1762,13 @@ else >> + endif >> + >> + %.service: %.service.in >> +- $(Q)sed -e 's|\@BINDIR\@|$(BINDIR)|g' $< >$@ >> ++ $(Q)sed -e 's|\@BINDIR\@|$(BINDIR)|g' \ >> ++ -e 's|\@DBUS_INTERFACE\@|$(DBUS_INTERFACE)|g' $< >$@ >> + @$(E) " sed" $< >> + >> + %@.service: %.service.arg.in >> +- $(Q)sed -e 's|\@BINDIR\@|$(BINDIR)|g' $< >$@ >> ++ $(Q)sed -e 's|\@BINDIR\@|$(BINDIR)|g' \ >> ++ -e 's|\@DBUS_INTERFACE\@|$(DBUS_INTERFACE)|g' $< >$@ >> + @$(E) " sed" $< >> + >> + wpa_supplicant.exe: wpa_supplicant >> +diff --git a/wpa_supplicant/systemd/wpa_supplicant.service.in >> b/wpa_supplicant/systemd/wpa_supplicant.service.in >> +index ea964ce..bc5d49a 100644 >> +--- a/wpa_supplicant/systemd/wpa_supplicant.service.in >> ++++ b/wpa_supplicant/systemd/wpa_supplicant.service.in >> +@@ -5,9 +5,9 @@ Wants=network.target >> + >> + [Service] >> + Type=dbus >> +-BusName=fi.epitest.hostap.WPASupplicant >> ++BusName=@DBUS_INTERFACE@ >> + ExecStart=@BINDIR@/wpa_supplicant -u >> + >> + [Install] >> + WantedBy=multi-user.target >> +-Alias=dbus-fi.epitest.hostap.WPASupplicant.service >> ++Alias=dbus- at DBUS_INTERFACE@.service >> +-- >> +2.8.0 >> + >> > > -- Marcin Niestroj ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH v3] package/wpa_supplicant: Fix systemd service for new DBus bus name 2016-04-13 8:01 ` Marcin Niestroj @ 2016-04-13 9:13 ` Arnout Vandecappelle 0 siblings, 0 replies; 5+ messages in thread From: Arnout Vandecappelle @ 2016-04-13 9:13 UTC (permalink / raw) To: buildroot On 04/13/16 10:01, Marcin Niestroj wrote: > Hi, > > On 12.04.2016 23:47, Arnout Vandecappelle wrote: [snip] >> Otherwise, the patch looks good to me. Since the spaces are just a >> triviality: >> >> Reviewed-by: Arnout Vandecappelle (Essensium/Mind) <arnout@mind.be> > > As I am newbie in mailing list workflow: does this tag mean, that I > have permission to add it to v4 patch directly (if I fix all pointed issues)? Or > it's just for other reviewers, so they know what has been > already reviewed in previous version of patch? It means that you can add my Reviewed-by tag to your next iteration, on condition that you don't change anything except the things I mentioned. Regards, Arnout -- Arnout Vandecappelle arnout dot vandecappelle at essensium dot com Senior Embedded Software Architect . . . . . . +32-478-010353 (mobile) Essensium, Mind division . . . . . . . . . . . . . . http://www.mind.be G.Geenslaan 9, 3001 Leuven, Belgium . . . . . BE 872 984 063 RPR Leuven LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-04-13 9:13 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-04-12 15:35 [Buildroot] [PATCH v3] package/wpa_supplicant: Fix systemd service for new DBus bus name Marcin Niestroj 2016-04-12 21:47 ` Arnout Vandecappelle 2016-04-13 4:57 ` Baruch Siach 2016-04-13 8:01 ` Marcin Niestroj 2016-04-13 9:13 ` Arnout Vandecappelle
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox