| Message ID | 20190127170657.23885-1-freifunk@adrianschmutzler.de |
|---|---|
| State | Superseded |
| Headers | show |
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
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@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
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@freifunk.net] On Behalf
> Of Christian Dresel
> Sent: Sonntag, 27. Januar 2019 18:13
> To: franken-dev@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@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
hi
ich reiße das mal aus der eigentlichen Diskussion raus:
On 27.01.19 18:18, mail@adrianschmutzler.de wrote:
> 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.
Wie das mit Batman gemacht ist, habe ich mir eben erst das erste mal
angeguckt und muss sagen: Ich finde das ziemlich unhübsch.
Wäre es nicht möglich das in ein package zu vereinen und dort eine
Erkennung einzubauen ob der Router node oder gw ist und die
entsprechende die config auswählen?
Warscheinlich wäre das dann auch für Alfred der beste Weg?
Ich könnte mir z.b. vorstellen das selectvariant gw irgendwo ein anderes
Bit als selectvariant node setzt und div. Scripte dies dann auswerten
können was sie denn nun genau machen sollen.
Gruß
Christian
Hallo, On 27.01.19 18:12, Christian Dresel wrote: > 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. alfred-json hat damit überhaupt nichts zu tun. Das ist ein Package, das keine FFF-spezifische Konfiguration, sondern ein Tool bereitstellt. Da gehört das Modus umschalten auf jeden Fall nicht hin. Gruß Fabian
Hallo Adrian, On 27.01.19 18:06, Adrian Schmutzler wrote: > --- 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 Sehe ich wie gesagt nicht so, zu fff-alfred besteht in diesem Paket keine Abhängigkeit. Zu "alfred" schon eher, so richtig zwingend aber auch nicht. Könnte man aber aufnehmen, da das Tool ohne alfred nichts sinnvolles tun kann. > --- 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 Hier hingegen besteht diese Abhängigkeit definitiv, da sich das Paket auf Konfiguration aus fff-alfred verlässt. Gruß Fabian
hi On 27.01.19 21:51, Fabian Bläse wrote: > Hallo, > > On 27.01.19 18:12, Christian Dresel wrote: >> 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. > alfred-json hat damit überhaupt nichts zu tun. Das ist ein Package, das keine FFF-spezifische Konfiguration, sondern ein Tool bereitstellt. Mein Gedanke war, das Tool braucht Alfred als master damit es überhaupt funktioniert von daher hätte ich es dort mit hin geschoben Gruß Christian > > Da gehört das Modus umschalten auf jeden Fall nicht hin. > > Gruß > Fabian >
Hallo Christian, On 28.01.19 08:24, Christian Dresel wrote: > Mein Gedanke war, das Tool braucht Alfred als master damit es überhaupt > funktioniert von daher hätte ich es dort mit hin geschoben Nur braucht das Tool Alfred gar nicht als master, damit es funktioniert. Sollte auf einem Alfred Slave ganz genauso funktionieren. PHP liefert doch auch keinen Webserver als Dependency + Konfiguration dafür mit, nur weil das halt eine gängige Konfiguration ist.. ;-) Gruß Fabian
Hi Fabian On 28.01.19 10:39, Fabian Bläse wrote: > Hallo Christian, > > On 28.01.19 08:24, Christian Dresel wrote: >> Mein Gedanke war, das Tool braucht Alfred als master damit es überhaupt >> funktioniert von daher hätte ich es dort mit hin geschoben > Nur braucht das Tool Alfred gar nicht als master, damit es funktioniert. > Sollte auf einem Alfred Slave ganz genauso funktionieren. Mist. Wieder auf den Fehler reingefallen, du hast natürlich recht das hatten wir schon mal. Ich hatte wieder im Hinterkopf das man zwingend master sein muss zum abfragen aber das ist falsch. Demnach hast du auch beim Rest recht, dort passt es dann auch nicht. Ich verweise mal auf http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/2019-January/015030.html Demnach wäre wohl ein "config Alfred package" (das müsste das bereits vorhandene fff-alfred doch schon sein oder?) ganz gut, wo je nachdem ob es sich um Gateway oder Node handelt entsprechend konfiguriert wird. Gruß Christian > > PHP liefert doch auch keinen Webserver als Dependency + Konfiguration dafür mit, nur weil das halt eine gängige Konfiguration ist.. ;-) > > Gruß > Fabian >
Hallo, das heißt, unabhängig von der grundsätzlichen Lösung mache ich nun folgendes: alfred-json: +alfred fff-alfred-monitoring-proxy: +fff-alfred (in Ergänzung zum Entfernen von fff-nodewatcher) ? (Ich würde das dann als V2 schicken) Dann könnte man die aktuellen Dependencies erstmal reparieren, und danach ggf. in neuen Patches umverteilen, wenn man möchte. Grüße Adrian > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of > Christian Dresel > Sent: Montag, 28. Januar 2019 11:23 > To: Fabian Bläse <fabian@blaese.de>; franken-dev@freifunk.net > Subject: Re: [PATCH] fff-alfred: Add dependency in monitoring-proxy and > alfred-json > > Hi Fabian > > On 28.01.19 10:39, Fabian Bläse wrote: > > Hallo Christian, > > > > On 28.01.19 08:24, Christian Dresel wrote: > >> Mein Gedanke war, das Tool braucht Alfred als master damit es überhaupt > >> funktioniert von daher hätte ich es dort mit hin geschoben > > Nur braucht das Tool Alfred gar nicht als master, damit es funktioniert. > > Sollte auf einem Alfred Slave ganz genauso funktionieren. > > Mist. Wieder auf den Fehler reingefallen, du hast natürlich recht das > hatten wir schon mal. Ich hatte wieder im Hinterkopf das man zwingend > master sein muss zum abfragen aber das ist falsch. > > Demnach hast du auch beim Rest recht, dort passt es dann auch nicht. > > Ich verweise mal auf > > http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/2019- > January/015030.html > > Demnach wäre wohl ein "config Alfred package" (das müsste das bereits > vorhandene fff-alfred doch schon sein oder?) ganz gut, wo je nachdem ob > es sich um Gateway oder Node handelt entsprechend konfiguriert wird. > > Gruß > > Christian > > > > > PHP liefert doch auch keinen Webserver als Dependency + Konfiguration > dafür mit, nur weil das halt eine gängige Konfiguration ist.. ;-) > > > > Gruß > > Fabian > >
Hallo Christian, On 28.01.19 11:22, Christian Dresel wrote: > Ich verweise mal auf > > http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/2019-January/015030.html > > Demnach wäre wohl ein "config Alfred package" (das müsste das bereits > vorhandene fff-alfred doch schon sein oder?) ganz gut, wo je nachdem ob > es sich um Gateway oder Node handelt entsprechend konfiguriert wird. Dahingehend kann man Überlegungen anstellen. Ich bin mir aber noch nicht sicher, ob das wirklich besser wäre. Für den Moment bin ich mir der reparierten Dependency (siehe v2 von Adrians Patch) glücklich. Gruß Fabian
Hallo, ich hänge immer mehr der Idee an, für die GW-Firmware eine eigene Package fff-alfred-server zu bauen, die die entsprechende Funktionalität enthält (=umschalten). Das stelle ich mir durchaus so vor, wie fff-batman-adv-server im Moment bei Fabian und mir implementiert ist. Beim Proxy gehört 52-alfred-master eigentlich nicht mit hinein. Da es sich beim batman-server und dem alfred-master jeweils aber nur um zwei bis drei Zeilen handelt, könnte man das statt einer jeweils eigenen Package auch beides zusammen in einem fff-gateway parken. Dies wäre dann ähnlich wie wir bestimmte Funktionalitäten z.B. für die Firewall mit in fff-hoods maintainen. Beides mal als Vorschläge zur Diskussion. Grüße Adrian From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of Fabian Bläse Sent: Montag, 28. Januar 2019 10:39 To: fff@chrisi01.de; franken-dev@freifunk.net Subject: Re: [PATCH] fff-alfred: Add dependency in monitoring-proxy and alfred-json Hallo Christian, On 28.01.19 08:24, Christian Dresel wrote: > Mein Gedanke war, das Tool braucht Alfred als master damit es überhaupt > funktioniert von daher hätte ich es dort mit hin geschoben Nur braucht das Tool Alfred gar nicht als master, damit es funktioniert. Sollte auf einem Alfred Slave ganz genauso funktionieren. PHP liefert doch auch keinen Webserver als Dependency + Konfiguration dafür mit, nur weil das halt eine gängige Konfiguration ist.. ;-) Gruß Fabian
Hallo Christian, finde ich so wie vorschlagen gerade irgendwie komplizierter. So richtig entschieden habe ich mich aber noch nicht ... Mal ein bisschen sacken lassen... Grüße Adrian > -----Original Message----- > From: Christian Dresel [mailto:fff@chrisi01.de] > Sent: Sonntag, 27. Januar 2019 20:39 > To: mail@adrianschmutzler.de; franken-dev@freifunk.net > Subject: Re: [PATCH] fff-alfred: Add dependency in monitoring-proxy and > alfred-json > > hi > > ich reiße das mal aus der eigentlichen Diskussion raus: > > On 27.01.19 18:18, mail@adrianschmutzler.de wrote: > > 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. > > Wie das mit Batman gemacht ist, habe ich mir eben erst das erste mal > angeguckt und muss sagen: Ich finde das ziemlich unhübsch. > > Wäre es nicht möglich das in ein package zu vereinen und dort eine > Erkennung einzubauen ob der Router node oder gw ist und die > entsprechende die config auswählen? > > Warscheinlich wäre das dann auch für Alfred der beste Weg? > > Ich könnte mir z.b. vorstellen das selectvariant gw irgendwo ein anderes Bit > als selectvariant node setzt und div. Scripte dies dann auswerten können was > sie denn nun genau machen sollen. > > Gruß > > Christian
Hallo zusammen, man kann über beide Lösungen nachdenken. Im Moment sollten wir uns bis wir da einen halbwegs funktionierenden Stand haben nicht gar zu sehr an Detailfragen aufhalten, solange uns das Nachhaltig nicht groß auf die Füße fällt.. Gruß Fabian On 29.01.19 01:11, mail@adrianschmutzler.de wrote: > Hallo, > > > > ich hänge immer mehr der Idee an, für die GW-Firmware eine eigene Package fff-alfred-server zu bauen, die die entsprechende Funktionalität enthält (=umschalten). Das stelle ich mir durchaus so vor, wie fff-batman-adv-server im Moment bei Fabian und mir implementiert ist. > > Beim Proxy gehört 52-alfred-master eigentlich nicht mit hinein. > > > > Da es sich beim batman-server und dem alfred-master jeweils aber nur um zwei bis drei Zeilen handelt, könnte man das statt einer jeweils eigenen Package auch beides zusammen in einem fff-gateway parken. Dies wäre dann ähnlich wie wir bestimmte Funktionalitäten z.B. für die Firewall mit in fff-hoods maintainen. > > > > Beides mal als Vorschläge zur Diskussion. > > > > Grüße > > > > Adrian > > > > *From:*franken-dev [mailto:franken-dev-bounces@freifunk.net] *On Behalf Of *Fabian Bläse > *Sent:* Montag, 28. Januar 2019 10:39 > *To:* fff@chrisi01.de; franken-dev@freifunk.net > *Subject:* Re: [PATCH] fff-alfred: Add dependency in monitoring-proxy and alfred-json > > > > Hallo Christian, > > On 28.01.19 08:24, Christian Dresel wrote: >> Mein Gedanke war, das Tool braucht Alfred als master damit es überhaupt >> funktioniert von daher hätte ich es dort mit hin geschoben > Nur braucht das Tool Alfred gar nicht als master, damit es funktioniert. > Sollte auf einem Alfred Slave ganz genauso funktionieren. > > PHP liefert doch auch keinen Webserver als Dependency + Konfiguration dafür mit, nur weil das halt eine gängige Konfiguration ist.. ;-) > > Gruß > Fabian > > >
Signed-off-by: Adrian Schmutzler <freifunk@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(-)