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

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

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 at blaese.de>
> 
> Fabian
> 
> > On 8. Jul 2018, at 19:57, Adrian Schmutzler <freifunk at 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 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/7c42546e/attachment.sig>