[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