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

[Christian Dresel]

Hi

On 27.01.19 18:00, mail at adrianschmutzler.de wrote:
> 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.
ein noch besserer Fund ;) Ja stimmt dann brauchen wir es hier auch. Ich
überlege aber gerade ob dieses ändern hier überhaupt korrekt ist oder wo
anders vielleicht besser aufgehoben ist? Bin mir da noch nicht sicher.
>
> Ich würde daher die Abhängigkeit zu fff-alfred in BEIDE Makefiles reinschreiben. Mach ich aber in nem extra Patch.

Das heißt du willst diesen applien und dann in einen weiteren Patch
jeweils fff-alfred hinzufügen? Ist in meinen Augen ok:

Reviewed-by: Christian Dresel <fff at chrisi01.de>

Gruß

Christian

>
> 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