[PATCH v2 1/2] Start fastd only if there are fastd peers left

[robert]

Hi,

die fehlenden Anfuerungszeichen sollte man mit einem extra Patch rein
machen (ich finde, man sollte da auch konsequent sein). Das betrifft
auch noch ein paar andere Dateien. Dann waere das auf einen Schlag
bereinigt. Das stammt noch so aus den alten fastd-skript-zeiten.

Wenn ich mich richtig erinnere, ging doch auch die Meinung um, da nicht
mehr all zu viel Energie rein zu stecken, da sich das mit dem dez.keyX
aendert.? Andererseits ist so ein Patch ja mal schnell gemacht.

Viele Gruesse

Robert


Am 29.05.2017 um 23:15 schrieb Tobias Klaus:
> Hallo,
>
>
> ich finde [ -n "$(ls 2> /dev/null)" ] zwar marginal lesbarer, aber so 
> funktioniert es ja auch.
>
> Etwas problematischer finde ich die fehlenden Anführungszeichen um ${project}. 
> Ich glaube zwar, dass wir nicht so schnell ein $VPN_PROJECT mit Leerzeichen 
> und Sonderzeichen kriegen, aber schöner ist es schon mit Anführungszeichen. 
> Leider fehlen die auch in der ganzen Datei.
>
> Keine Ahnung wie wir damit Umgehen sollen. Da die Datei im Falle eines 
> "kaputten" $project vermutlich eh ziemlichen schaden anrichten würde, das aber 
> nicht bald ins Haus steht, würde ich das wahrscheinlich so nehmen und dann 
> sollte mal[tm] jemand[tm] mit dem shellchecker drüber gehen...
>
> Wie seht du/ihr da?
>
> Viele Grüße
> Tobias
>
>
>
> Am Samstag, 29. April 2017, 18:52:28 CEST schrieb Robert Langhammer:
>> Signed-off-by: Robert Langhammer <rlanghammer at web.de>
>> ---
>>  src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select
>> b/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select index
>> a0878c5..016c940 100755
>> --- a/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select
>> +++ b/src/packages/fff/fff-vpn-select/files/usr/sbin/vpn-select
>> @@ -67,7 +67,8 @@ if ping -w5 -c3 "$test_ipv4_host1" &>/dev/null ||
>>              uci commit fastd
>>          fi
>>          make_config
>> -        /etc/init.d/fastd start
>> +        # start fastd only if there are some peers left
>> +        [ "$(ls /etc/fastd/${project}/peers/* 2>/dev/null)" ] &&
>> /etc/init.d/fastd start /etc/init.d/tunneldigger start
>>      else
>>          # check if new tunneldigger conf is different
>