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

[mail at adrianschmutzler.de]

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>; franken-
> 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>
> > ---
> >  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 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/20190127/46407f9a/attachment.sig>