[PATCH v3 1/3] fff-gateway: add package
Fabian Bläse
fabian at blaese.de
Di Mai 7 10:57:55 CEST 2019
Gibt es hierzu noch Kommentare?
Ansonsten würde ich die von Adrian gewünschten Änderungen beim Applien einbauen.
Gruß
Fabian
On 24.04.19 10:22, Fabian Bläse wrote:
> Hallo Adrian
>
> On 24.04.19 00:00, Adrian Schmutzler wrote:
>>> +
>>> +configure() {
>>> + echo "This script might remove existing vlans, interfaces, addresses,
>>> etc."
>>> + read -r -p "Do you really want to continue? (y/n) " response
>>> + if ! [ "$response" == "y" ] || [ "$response" == "Y" ]; then
>>
>> Da fehlt ne Klammer? Oder habe ich die Operator-Reihenfolge falsch im Kopf?
>>
>> + if ! ( [ "$response" == "y" ] || [ "$response" == "Y" ] ); then
>>
>> Selbst dann würde ich eine hinmachen zur Klarheit.
> Ja. So funktioniert ein "Y" nicht.
> Die Klammern stimmen so, wie du geschrieben hast.
>
>>> + sleep 5
>>> + echo "Configuration reloaded. Changes will be reverted in 200s."
>>> + echo "Kill this script to keep changes."
>>> + sleep 200
>>> + echo "Reverting changes.."
>>
>> Falls ich das applye, mache ich hier den Punkt weg.
> Von mir aus.
>
>>> +
>>> + revert_changes
>>> + restart_services
>>
>> reload_services
> Sachen gibts.. Hast hier natürlich auch Recht.
>
>>> +}
>>> +
>>> +
>>> +usage() {
>>> + echo Usage: $0 [OPTION];
>>
>> Ich bin echt fasziniert, dass das ohne Anführungszeichen geht. Ich würde trotzdem gerne welche rum machen (dann kann das Semikolon weg?).
> Ja, Klammern rum.
> Das Semikolon hat damit nichts zu tun, das kann auch so weg, genauso in der nächsten Zeile.
>
> Gruß
> Fabian
>
>
>
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname : signature.asc
Dateityp : application/pgp-signature
Dateigröße : 833 bytes
Beschreibung: OpenPGP digital signature
URL : <https://{'listname': 'franken-dev-freifunk.net', 'hostname': 'lists.freifunk.net'}/pipermail/franken-dev-freifunk.net/attachments/20190507/c016e005/attachment.sig>
Mehr Informationen über die Mailingliste franken-dev