[PATCH v3 4/5] hood files: Move files from /www to /tmp and link there

[Tim Niemeyer]

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 at 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 at blaese.de>
> > 
> > Fabian
> > 
> > > On 8. Jul 2018, at 19:57, Adrian Schmutzler <freifunk at 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 at 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
> > > 
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 488 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20180725/d54748be/attachment.sig>