[PATCH] Make vpn-select modular

Adrian Schmutzler mail at adrianschmutzler.de
Sa Aug 8 21:09:53 CEST 2020


Hallo,

>>>      else 
>>> -            /usr/sbin/vpn-stop 
>>> +            /usr/sbin/vpn-select stop-VPN 
>> Das finde ich nicht schön. Der erste Parameter ist ein Pfad. 
>> Allerdings könnte man hier einfach den Parameter weglassen und würde effektiv zum gewünschten Ergebnis kommen. 
> Weglassen ist sehr unschön. vpn-select erwartet immer $1 (Usage: 
> vpn-select <path-to-hood-file>). Das ist nicht optional.  vpn-select 
> stop-VPN ist natürlich ein ungültiger Pfad, hier aber sehr sprechend für 
> das was passiert. Darauf reagiert dann vpns-select und legt keine peers 
> an -> vpn-stop (s. oben) 

Naja, wenn du hier mit der "Usage" argumentierst, ist ein falsches Argument jetzt auch nicht richtiger als gar keins.
Funktionieren würde beides. Aber das mit dem stop-VPN stört mich tatsächlich sehr.

Man könnte höchstens überlegen, hier noch einen weiteren Parameter einzuführen, der vorgibt, was vpn-select tun soll, wenn man es ganz ordentlich will.

Grüße

Adrian 
-------------- 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/20200808/d6ffa2cf/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev