[PATCH 2/6] fff-alfred-monitoring-proxy: Remove dependency from nodewatcher

[mail at adrianschmutzler.de]

Applied.

 

From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf Of mail at adrianschmutzler.de
Sent: Sonntag, 27. Januar 2019 18:01
To: fff at chrisi01.de; franken-dev at freifunk.net
Subject: RE: [PATCH 2/6] fff-alfred-monitoring-proxy: Remove dependency from nodewatcher

 

Hallo Christian, 

guter Fund: 

Meines Erachtens brauchen sowohl alfred-json als auch der proxy jeweils die Abhängigkeit zu fff-alfred: 

alfred-json hast du schon begründet. 
alfred-monitoring-proxy ändert im uci-defaults Skript die alfred uci config, die vorher in fff-alfred erzeugt werden muss.

Ich würde daher die Abhängigkeit zu fff-alfred in BEIDE Makefiles reinschreiben. Mach ich aber in nem extra Patch. 

Grüße 

Adrian 

> -----Original Message----- 
> From: Christian Dresel [mailto:fff at chrisi01.de] 
> Sent: Sonntag, 27. Januar 2019 17:48 
> To: Adrian Schmutzler <freifunk at adrianschmutzler.de <mailto:freifunk at adrianschmutzler.de> >; franken- 
> dev at freifunk.net <mailto:dev at freifunk.net>  
> Subject: Re: [PATCH 2/6] fff-alfred-monitoring-proxy: Remove dependency 
> from nodewatcher 
> 
> hi 
> 
> Rückfrage: 
> 
> Der Nodewatcher hat folgende Abhängigkeiten drinnen: 
> 
> DEPENDS:=+fff-alfred +micrond +fff-config +fff-random +ip +ip-bridge 
> 
> Speziell fällt mir hier fff-alfred auf anhieb auf. Allerdings verwendet fff- 
> alfred-monitoring-proxy nicht direkt fff-alfred sondern alfred-json greift auf 
> den Alfred zu und alfred-json wird von fff-alfred-monitoring-proxy 
> verwenden und haben wir hier korrekterweise dann auch mit drinnen. 
> 
> Meines erachtens fehlt dann aber im alfred-json die Abhängigkeit zu fff- 
> alfred oder? 
> 
> Wenn ich es richtig zusammengefasst habe, sollte es hier passen (wenn es 
> jemand bestätigt gibts auch ein Review) aber wir müssen fff-alfred noch als 
> Abhängigkeit in alfred-json hinzufügen. Richtig soweit? 
> 
> Gruß 
> 
> Christian 
> 
> On 27.01.19 17:38, Adrian Schmutzler wrote: 
> > The proxy only sends data from alfred master to the Monitoring. 
> > It does not require the device to produce alfred-data in the 
> > nodewatcher itself. 
> > 
> > Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de <mailto:freifunk at adrianschmutzler.de> > 
> > --- 
> >  src/packages/fff/fff-alfred-monitoring-proxy/Makefile | 2 +- 
> >  1 file changed, 1 insertion(+), 1 deletion(-) 
> > 
> > diff --git a/src/packages/fff/fff-alfred-monitoring-proxy/Makefile 
> > b/src/packages/fff/fff-alfred-monitoring-proxy/Makefile 
> > index 93d982e0..7ab592d9 100644 
> > --- a/src/packages/fff/fff-alfred-monitoring-proxy/Makefile 
> > +++ b/src/packages/fff/fff-alfred-monitoring-proxy/Makefile 
> > @@ -13,7 +13,7 @@ define Package/$(PKG_NAME) 
> >      CATEGORY:=Freifunk 
> >      TITLE:= Freifunk-Franken Alfred-Monitoring-Proxy 
> >      URL:=https://www.freifunk-franken.de/ 
> > -    DEPENDS:=+micrond +fff-random +curl +alfred-json +fff-nodewatcher 
> > +    DEPENDS:=+micrond +fff-random +curl +alfred-json 
> >  endef 
> > 
> >  define Package/$(PKG_NAME)/description 
-------------- nächster Teil --------------
Ein Dateianhang mit HTML-Daten wurde abgetrennt...
URL: <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190129/6f50e64e/attachment.html>
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : nicht verfügbar
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190129/6f50e64e/attachment.sig>