[PATCH v2] fff-gateway: Add sanity checks

mail at adrianschmutzler.de mail at adrianschmutzler.de
So Aug 18 10:47:32 CEST 2019


Hallo,

nennt die uci section "meta" und die option "version" oder "config_version".
Dann ist das in Zukunft noch erweiterbar, und ein doppeltes version bringt recht wenig.

Grüße

Adrian

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf
> Of Robert Langhammer
> Sent: Sonntag, 18. August 2019 10:44
> To: franken-dev at freifunk.net
> Subject: Re: [PATCH v2] fff-gateway: Add sanity checks
> 
> Hallo nochmal,
> ich finde es besser gateway.version als named section zu definieren und das
> so zu lassen. Die section sollte es ja auch nur einmal geben.
> Robert
> 
> Am 18. August 2019 09:02:42 MESZ schrieb robert <rlanghammer at web.de>:
> >hi Fabian,
> >
> >Am 17.08.19 um 22:36 schrieb Fabian Bläse:
> >> Dieser Patch ist echt verteufelt.
> >>
> >>
> >> +configure() {
> >> +	local expected_version=1
> >> +	local config_version=$(uci -q get gateway.version.config_version)
> >> Hier muss es gateway. at version[0].config_version heißen.
> >
> >muss es gar nicht. Kommt darauf an, was in /etc/config/gateway steht.
> >Und da kann man das so reinschreiben wie es hier gebraucht wird. Die
> >Doku muss nur passen.
> >
> >Gruss Robert
> >
> >>
> >> Damit kann dann auch ein
> >> Tested-by: Fabian Bläse <fabian at blaese.de> dran.
> >>
> >> Gruß
> >> Fabian
> >>
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : openpgp-digital-signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190818/dc41bcdf/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev