[PATCH v3 4/5] hood files: Move files from /www to /tmp and link there
Tim Niemeyer
tim at tn-x.org
Mi Jul 25 14:59:23 CEST 2018
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>
Mehr Informationen über die Mailingliste franken-dev