[PATCH] fff-alfred: Add dependency in monitoring-proxy and alfred-json

mail at adrianschmutzler.de mail at adrianschmutzler.de
So Jan 27 18:18:54 CET 2019


Hallo Christian,

in meinen Augen müsste man das ("richtig") ganz anders machen und zwei unterschiedliche alfred-Packages für node-Firmware (alfred-client) und gw-Firmware (alfred-master) anbieten. So machen wir es ja auch mit den batman-Paketen.

Deinen Kommentar in der anderen Mail hatte ich wohl bei ersten Mal überlesen oder nicht so verstanden. Baut man keine eigene Package für den master ist das dann eine philosophische Diskussion darüber, ob alfred-json ein Hilfstool für den Proxy ist oder der Proxy ein Hilfstool für den alfred-master. Entsprechend kann ich mir durchaus auch vorstellen, deinem Standpunkt zu folgen. Im Moment tendiere ich aber am meisten zur Separate-Packages-Lösung. (Die macht dann aber wieder Probleme beim nodewatcher).

Ich werde da erstmal ein bisschen drüber nachdenken, vll. hat ja sonst auch noch jemand eine Meinung.

Grüße

Adrian

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf
> Of Christian Dresel
> Sent: Sonntag, 27. Januar 2019 18:13
> To: franken-dev at freifunk.net
> Subject: Re: [PATCH] fff-alfred: Add dependency in monitoring-proxy and
> alfred-json
> 
> hi
> 
> wie in der anderen Mail schon erwähnt glaube ich, dass das umschalten auf
> Alfredmaster eigentlich im fff-alfred-monitoring-proxy falsch aufgehoben ist.
> 
> Je länger ich drüber nachdenke umso eher fände ich es besser, wenn der
> alfred-json diesen Modus umschaltet da er auf den Alfred als Master zugreift
> und eben nicht das fff-alfred-monitoring-proxy package welches ja nur den
> alfred-json verwendet.
> 
> Gruß
> 
> Christian
> 
> On 27.01.19 18:06, Adrian Schmutzler wrote:
> > Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> > ---
> >  src/packages/fff/alfred-json/Makefile                 | 2 +-
> >  src/packages/fff/fff-alfred-monitoring-proxy/Makefile | 2 +-
> >  2 files changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/src/packages/fff/alfred-json/Makefile
> > b/src/packages/fff/alfred-json/Makefile
> > index f3534636..51b47165 100644
> > --- a/src/packages/fff/alfred-json/Makefile
> > +++ b/src/packages/fff/alfred-json/Makefile
> > @@ -16,7 +16,7 @@ define Package/$(PKG_NAME)
> >      CATEGORY:=Utilities
> >      TITLE:=A.L.F.R.E.D - JSON Client
> >      URL:=https://github.com/FreifunkFranken/alfred-json
> > -    DEPENDS:=+jansson +zlib
> > +    DEPENDS:=+jansson +zlib +fff-alfred
> >  endef
> >
> >  define Package/$(PKG_NAME)/install
> > diff --git a/src/packages/fff/fff-alfred-monitoring-proxy/Makefile
> > b/src/packages/fff/fff-alfred-monitoring-proxy/Makefile
> > index 4370bd9c..5834d44c 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
> > +    DEPENDS:=+curl +micrond +alfred-json +fff-alfred +fff-random
> >  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/b2fad612/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev