[4/4] hood/sector files: Move files from /www to /tmp and link there

Submitted by Adrian Schmutzler on June 13, 2018, 2:36 p.m.

Details

Message ID 1528900566-9555-5-git-send-email-freifunk@adrianschmutzler.de
State Superseded
Headers show

Commit Message

Adrian Schmutzler June 13, 2018, 2:36 p.m.
With this patch, changing the hood and sector files happens in
memory, so we do not have to write on flash "frequently".

Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
---
 src/packages/fff/fff-hoods/files/usr/sbin/configurehood           | 1 -
 src/packages/fff/fff-hoodutils/files/lib/functions/fff/keyxchange | 4 +++-
 src/packages/fff/fff-web/files/etc/uci-defaults/93-fff-uhttpd     | 5 +++++
 3 files changed, 8 insertions(+), 2 deletions(-)

Patch hide | download patch | download mbox

diff --git a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
index 1c1eda7..dd76af9 100755
--- a/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
+++ b/src/packages/fff/fff-hoods/files/usr/sbin/configurehood
@@ -9,7 +9,6 @@ 
 
 sectorfilelocal=/etc/sectorfile
 sectorfiletmp=/tmp/sectorfile
-sectorfilewww=/www/hood/sectorfile
 
 hoodfilelocal=/etc/hoodfile
 hoodfileisauth=/tmp/hoodfileisauth
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..c0e35ac 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,9 @@ 
 
 hoodfiletmp="/tmp/hoodfile"
 hoodfileref="/tmp/hoodfileref"
-hoodfilewww="/www/hood/keyxchangev2data"
+hoodfilewww="/tmp/hoodfilewww"
+
+sectorfilewww="/tmp/sectorfilewww"
 
 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..bb99900 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,10 @@ 
 #!/bin/sh
 
+. /lib/functions/fff/keyxchange
+
+ln -s "$hoodfilewww" /www/hood/keyxchangev2data
+ln -s "$sectorfilewww" /www/hood/sectorfile
+
 uci batch <<EOF
   delete uhttpd.main
   delete uhttpd.public

Comments

Fabian Blaese June 24, 2018, 9:27 a.m.
Moin.

> On 13. Jun 2018, at 16:36, Adrian Schmutzler <freifunk@adrianschmutzler.de> wrote:
> 
> 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..bb99900 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,10 @@
> #!/bin/sh
> 
> +. /lib/functions/fff/keyxchange
> +
> +ln -s "$hoodfilewww" /www/hood/keyxchangev2data
> +ln -s "$sectorfilewww" /www/hood/sectorfile
> +

Da kann man sich jetzt wohl drüber streiten, wo man den Symlink haben möchte.
Ich hätte eher zu einem uci-defaults Skript tendiert, was mit in fff-hoodutils drin liegt.
Andererseits klingt das auch nicht falsch.

Fabian
Adrian Schmutzler June 24, 2018, 5:55 p.m.
Hallo,

siehe unten.

> -----Original Message-----
> From: Fabian Bläse [mailto:fabian@blaese.de]
> Sent: Sonntag, 24. Juni 2018 11:27
> To: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> Cc: franken-dev <franken-dev@freifunk.net>
> Subject: Re: [PATCH 4/4] hood/sector files: Move files from /www to /tmp
> and link there
> 
> Moin.
> 
> > On 13. Jun 2018, at 16:36, Adrian Schmutzler
> <freifunk@adrianschmutzler.de> wrote:
> >
> > 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..bb99900 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,10 @@
> > #!/bin/sh
> >
> > +. /lib/functions/fff/keyxchange
> > +
> > +ln -s "$hoodfilewww" /www/hood/keyxchangev2data ln -s
> > +"$sectorfilewww" /www/hood/sectorfile
> > +
> 
> Da kann man sich jetzt wohl drüber streiten, wo man den Symlink haben
> möchte.
> Ich hätte eher zu einem uci-defaults Skript tendiert, was mit in fff-hoodutils
> drin liegt.
> Andererseits klingt das auch nicht falsch.

Mir ist das prinzipiell wurscht. Allerdings nützt das Hoodfile ohne den Webserver nichts, und der Link zum Hoodfile ist für mich ein Feature des Webservers. Deshalb habe ich das da hin. Wenn das jemand lieber in fff-hoodutils will, soll es mir auch recht sein.

Grüße

Adrian

> 
> Fabian
Fabian Blaese July 8, 2018, 7:21 p.m.
Hallo Adrian,

> On 24. Jun 2018, at 19:55, Adrian Schmutzler <mail@adrianschmutzler.de> wrote:
> 
> Mir ist das prinzipiell wurscht. Allerdings nützt das Hoodfile ohne den Webserver nichts, und der Link zum Hoodfile ist für mich ein Feature des Webservers. Deshalb habe ich das da hin. Wenn das jemand lieber in fff-hoodutils will, soll es mir auch recht sein.

Hmm, tatsächlich schwierig, denn das hängt irgendwie alles voneinander ab.
Hat da sonst noch jemand ne Meinung zu?

Ansonsten würde ich es im uhttpd lassen, da alle anderen Skripte ja nicht mehr von der Datei in /www abhängen, sondern nur noch der Webserver darauf zugreift.

Da fällt mir grade auf, dass fff-hoods gar nicht auf fff-uhttpd dependet..

Fabian