[PATCH v2] fff-gateway: Add sanity checks

[mail at adrianschmutzler.de]

Also ich bin eher dagegen, da das effektiv ein undokumentiertes Feature ist.

 

Ich würde entweder Fabians Lösung mit show > dev/null oder den alten [ -s ] nehmen.

 

Grüße

 

Adrian

 

From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf Of robert
Sent: Dienstag, 20. August 2019 22:59
To: franken-dev at freifunk.net
Subject: Re: [PATCH v2] fff-gateway: Add sanity checks

 

Hi, 

ein uci get config prüft nur die Existenz der Datei. Test: 

root at c2600:/etc/config# > robert 
root at c2600:/etc/config# uci get robert 
root at c2600:/etc/config# echo $? 
0 
root at c2600:/etc/config# rm robert 
root at c2600:/etc/config# uci get robert 
uci: Entry not found 
root at c2600:/etc/config# echo $? 
1 

Also nicht mal den Inhalt. Man könnte also auch ein [ -f Datei ] nehmen. 

Da es keinen Unterschied macht, würde ich es so lassen. 

Grüße 
Robert 

Am 20.08.19 um 17:06 schrieb Fabian Bläse: 
> Hey Adrian, 
> 
> dann hab ich die Frage falsch verstanden oder unaufmerksam gelesen, sorry. 
> 
> Du hast recht, die uci help sagt, dass eine "section" mit angegeben werden soll. 
> In so fern ist es wohl fraglich, ob wir uns auf das Verhalten verlassen können. 
> 
> Man könnte auch "uci show gateway >/dev/null" verwenden.. 
> 
> Gruß 
> Fabian 
> 

 
-------------- nächster Teil --------------
Ein Dateianhang mit HTML-Daten wurde abgetrennt...
URL: <https://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190820/d338eb0d/attachment.html>
-------------- 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/20190820/d338eb0d/attachment.sig>