| Message ID | 1531072659-4217-4-git-send-email-freifunk@adrianschmutzler.de |
|---|---|
| State | Accepted |
| Headers | show |
diff --git a/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange b/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange index 40ebe4f..e601e0d 100644 --- a/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange +++ b/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange @@ -6,7 +6,7 @@ hoodfiletmp="/tmp/hoodfile" hoodfileref="/tmp/hoodfileref" -hoodfilewww="/www/hood/keyxchangev2data" +hoodfilewww="/tmp/hoodfilewww" getJsonPath() { jsonfile="" diff --git a/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd b/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd index 42376ad..248a78b 100644 --- a/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd +++ b/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd @@ -1,5 +1,9 @@ #!/bin/sh +. /lib/functions/fff/keyxchange + +ln -s "$hoodfilewww" /www/hood/keyxchangev2data + uci batch <<EOF delete uhttpd.main delete uhttpd.public
Ja, das in /tmp zu machen finde ich auch gut. Reviewed-by: Robert Langhammer <rlanghammer@web.de> Am 08.07.2018 um 19:57 schrieb Adrian Schmutzler: > With this patch, changing the hood file happens in > memory, so we do not have to write on flash "frequently". > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > --- > > Changes in v2: > none > > Changes in v3: > Remove references to sector file > --- > src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange | 2 +- > src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd | 4 ++++ > 2 files changed, 5 insertions(+), 1 deletion(-) > > diff --git a/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange b/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange > index 40ebe4f..e601e0d 100644 > --- a/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange > +++ b/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange > @@ -6,7 +6,7 @@ > > hoodfiletmp="/tmp/hoodfile" > hoodfileref="/tmp/hoodfileref" > -hoodfilewww="/www/hood/keyxchangev2data" > +hoodfilewww="/tmp/hoodfilewww" > > getJsonPath() { > jsonfile="" > diff --git a/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd b/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd > index 42376ad..248a78b 100644 > --- a/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd > +++ b/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd > @@ -1,5 +1,9 @@ > #!/bin/sh > > +. /lib/functions/fff/keyxchange > + > +ln -s "$hoodfilewww" /www/hood/keyxchangev2data > + > uci batch <<EOF > delete uhttpd.main > delete uhttpd.public
Moin, ist nicht nur gut, sondern in Verbindung mit dem letzten Patch unbedingt nötig, weil dann alle 5 Minuten nach hoodfilewww geschrieben wird. Nur um das nochmal irgendwo deutlich stehen zu haben. :-) Man könnte um den Kopiervorgang nochmal ne Checksum drumrum bauen und nur kopieren, wenn sich was verändert hat.. Aber wirklich nötig ist das mit diesem Patch nicht mehr und würde das ganze wohl nur deutlich komplizierter machen. Gruß Fabian > On 9. Jul 2018, at 14:28, Robert Langhammer <rlanghammer@web.de> wrote: > > Ja, das in /tmp zu machen finde ich auch gut. > > Reviewed-by: Robert Langhammer <rlanghammer@web.de>
In Bezug auf meinen Kommentar zur v1 dieses Patches: Ich denke das Anlegen des Symlink beim Webserver zu behalten ist das schlaueste, zudem sollte man mal irgendwann noch die fehlende Abhängigkeit in fff-hoods ergänzen. Reviewed-by: Fabian Bläse <fabian@blaese.de> Fabian > On 8. Jul 2018, at 19:57, Adrian Schmutzler <freifunk@adrianschmutzler.de> wrote: > > With this patch, changing the hood file happens in > memory, so we do not have to write on flash "frequently". > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > --- > > Changes in v2: > none > > Changes in v3: > Remove references to sector file > --- > src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange | 2 +- > src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd | 4 ++++ > 2 files changed, 5 insertions(+), 1 deletion(-) > > diff --git a/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange b/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange > index 40ebe4f..e601e0d 100644 > --- a/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange > +++ b/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange > @@ -6,7 +6,7 @@ > > hoodfiletmp="/tmp/hoodfile" > hoodfileref="/tmp/hoodfileref" > -hoodfilewww="/www/hood/keyxchangev2data" > +hoodfilewww="/tmp/hoodfilewww" > > getJsonPath() { > jsonfile="" > diff --git a/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd b/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd > index 42376ad..248a78b 100644 > --- a/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd > +++ b/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd > @@ -1,5 +1,9 @@ > #!/bin/sh > > +. /lib/functions/fff/keyxchange > + > +ln -s "$hoodfilewww" /www/hood/keyxchangev2data > + > uci batch <<EOF > delete uhttpd.main > delete uhttpd.public > -- > 2.7.4 >
Tested-by: Fabian Bläse <fabian@blaese.de> Fabian > On 9. Jul 2018, at 15:18, Fabian Bläse <fabian@blaese.de> wrote: > > In Bezug auf meinen Kommentar zur v1 dieses Patches: > Ich denke das Anlegen des Symlink beim Webserver zu behalten ist das schlaueste, zudem sollte man mal irgendwann noch die fehlende Abhängigkeit in fff-hoods ergänzen. > > Reviewed-by: Fabian Bläse <fabian@blaese.de> > > Fabian > >> On 8. Jul 2018, at 19:57, Adrian Schmutzler <freifunk@adrianschmutzler.de> wrote: >> >> With this patch, changing the hood file happens in >> memory, so we do not have to write on flash "frequently". >> >> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> >> >> --- >> >> Changes in v2: >> none >> >> Changes in v3: >> Remove references to sector file >> --- >> src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange | 2 +- >> src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd | 4 ++++ >> 2 files changed, 5 insertions(+), 1 deletion(-) >> >> diff --git a/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange b/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange >> index 40ebe4f..e601e0d 100644 >> --- a/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange >> +++ b/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange >> @@ -6,7 +6,7 @@ >> >> hoodfiletmp="/tmp/hoodfile" >> hoodfileref="/tmp/hoodfileref" >> -hoodfilewww="/www/hood/keyxchangev2data" >> +hoodfilewww="/tmp/hoodfilewww" >> >> getJsonPath() { >> jsonfile="" >> diff --git a/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd b/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd >> index 42376ad..248a78b 100644 >> --- a/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd >> +++ b/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd >> @@ -1,5 +1,9 @@ >> #!/bin/sh >> >> +. /lib/functions/fff/keyxchange >> + >> +ln -s "$hoodfilewww" /www/hood/keyxchangev2data >> + >> uci batch <<EOF >> delete uhttpd.main >> delete uhttpd.public >> -- >> 2.7.4 >> >
Am Montag, den 09.07.2018, 15:18 +0200 schrieb Fabian Bläse: > In Bezug auf meinen Kommentar zur v1 dieses Patches: > Ich denke das Anlegen des Symlink beim Webserver zu behalten ist das > schlaueste, zudem sollte man mal irgendwann noch die fehlende > Abhängigkeit in fff-hoods ergänzen. Die Abhängigkeit müsste in fff-hoodutils sein, weil dort die Datei ja logisch definiert. Angelegt wird sie nun aber in fff-web. fff-hoodutils müsste also von fff-web abhängen, weil es die Datei zwar definiert, aber nicht anlegt. Das geht aber nicht, fff-web hängt bereits von fff-hoodutils ab. @Fabian: warum meinst du, dass fff-hoods von fff-web abhängen sollte? Benötigt fff-hoods irgendwas von fff-web? Ist das vielleicht eine Verwechselung, weil fff-hoods letztlich den Webserver auf einem anderen Gerät abfragt? Das wäre ja aber eine Abhängigkeit der Package innerhalb eines Gerätes. Tim > > Reviewed-by: Fabian Bläse <fabian@blaese.de> > > Fabian > > > On 8. Jul 2018, at 19:57, Adrian Schmutzler <freifunk@adrianschmutz > > ler.de> wrote: > > > > With this patch, changing the hood file happens in > > memory, so we do not have to write on flash "frequently". > > > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > > > --- > > > > Changes in v2: > > none > > > > Changes in v3: > > Remove references to sector file > > --- > > src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange | > > 2 +- > > src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd | > > 4 ++++ > > 2 files changed, 5 insertions(+), 1 deletion(-) > > > > diff --git a/src/packages/fff/fff- > > hoodutils/files/lib/functions/fff/keyxchange > > b/src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange > > index 40ebe4f..e601e0d 100644 > > --- a/src/packages/fff/fff- > > hoodutils/files/lib/functions/fff/keyxchange > > +++ b/src/packages/fff/fff- > > hoodutils/files/lib/functions/fff/keyxchange > > @@ -6,7 +6,7 @@ > > > > hoodfiletmp="/tmp/hoodfile" > > hoodfileref="/tmp/hoodfileref" > > -hoodfilewww="/www/hood/keyxchangev2data" > > +hoodfilewww="/tmp/hoodfilewww" > > > > getJsonPath() { > > jsonfile="" > > diff --git a/src/packages/fff/fff-web/files/etc/uci-defaults/93- > > fff-uhttpd b/src/packages/fff/fff-web/files/etc/uci-defaults/93- > > fff-uhttpd > > index 42376ad..248a78b 100644 > > --- a/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd > > +++ b/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd > > @@ -1,5 +1,9 @@ > > #!/bin/sh > > > > +. /lib/functions/fff/keyxchange > > + > > +ln -s "$hoodfilewww" /www/hood/keyxchangev2data > > + > > uci batch <<EOF > > delete uhttpd.main > > delete uhttpd.public > > -- > > 2.7.4 > > > >
Am Mittwoch, den 25.07.2018, 14:59 +0200 schrieb Tim Niemeyer: > Am Montag, den 09.07.2018, 15:18 +0200 schrieb Fabian Bläse: > > In Bezug auf meinen Kommentar zur v1 dieses Patches: > > Ich denke das Anlegen des Symlink beim Webserver zu behalten ist > > das > > schlaueste, zudem sollte man mal irgendwann noch die fehlende > > Abhängigkeit in fff-hoods ergänzen. > > Die Abhängigkeit müsste in fff-hoodutils sein, weil dort die Datei ja > logisch definiert. Angelegt wird sie nun aber in fff-web. Das ist falsch. Die Datei wird da nicht angelegt, sondern nur eine Referenz gesetzt, damit auch das Webroot weiß, wo die Datei liegt. > fff-hoodutils müsste also von fff-web abhängen, weil es die Datei > zwar > definiert, aber nicht anlegt. Das geht aber nicht, fff-web hängt > bereits von fff-hoodutils ab. Insofern ist diese Anmerkung murks, und der Patch passt. Reviewed-by: Tim Niemeyer <tim@tn-x.org> > @Fabian: warum meinst du, dass fff-hoods von fff-web abhängen sollte? > Benötigt fff-hoods irgendwas von fff-web? Ist das vielleicht eine > Verwechselung, weil fff-hoods letztlich den Webserver auf einem > anderen > Gerät abfragt? Das wäre ja aber eine Abhängigkeit der Package > innerhalb > eines Gerätes. Das ist ein anderes Thema. Tim > Tim > > > > > Reviewed-by: Fabian Bläse <fabian@blaese.de> > > > > Fabian > > > > > On 8. Jul 2018, at 19:57, Adrian Schmutzler <freifunk@adrianschmu > > > tz > > > ler.de> wrote: > > > > > > With this patch, changing the hood file happens in > > > memory, so we do not have to write on flash "frequently". > > > > > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > > > > > --- > > > > > > Changes in v2: > > > none > > > > > > Changes in v3: > > > Remove references to sector file > > > --- > > > src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange > > > | > > > 2 +- > > > src/packages/fff/fff-web/files/etc/uci-defaults/93-fff- > > > uhttpd | > > > 4 ++++ > > > 2 files changed, 5 insertions(+), 1 deletion(-) > > > > > > diff --git a/src/packages/fff/fff- > > > hoodutils/files/lib/functions/fff/keyxchange > > > b/src/packages/fff/fff- > > > hoodutils/files/lib/functions/fff/keyxchange > > > index 40ebe4f..e601e0d 100644 > > > --- a/src/packages/fff/fff- > > > hoodutils/files/lib/functions/fff/keyxchange > > > +++ b/src/packages/fff/fff- > > > hoodutils/files/lib/functions/fff/keyxchange > > > @@ -6,7 +6,7 @@ > > > > > > hoodfiletmp="/tmp/hoodfile" > > > hoodfileref="/tmp/hoodfileref" > > > -hoodfilewww="/www/hood/keyxchangev2data" > > > +hoodfilewww="/tmp/hoodfilewww" > > > > > > getJsonPath() { > > > jsonfile="" > > > diff --git a/src/packages/fff/fff-web/files/etc/uci-defaults/93- > > > fff-uhttpd b/src/packages/fff/fff-web/files/etc/uci-defaults/93- > > > fff-uhttpd > > > index 42376ad..248a78b 100644 > > > --- a/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff- > > > uhttpd > > > +++ b/src/packages/fff/fff-web/files/etc/uci-defaults/93-fff- > > > uhttpd > > > @@ -1,5 +1,9 @@ > > > #!/bin/sh > > > > > > +. /lib/functions/fff/keyxchange > > > + > > > +ln -s "$hoodfilewww" /www/hood/keyxchangev2data > > > + > > > uci batch <<EOF > > > delete uhttpd.main > > > delete uhttpd.public > > > -- > > > 2.7.4 > > >
With this patch, changing the hood file happens in memory, so we do not have to write on flash "frequently". Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> --- Changes in v2: none Changes in v3: Remove references to sector file --- src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange | 2 +- src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd | 4 ++++ 2 files changed, 5 insertions(+), 1 deletion(-)