[PATCH v3 1/3] fff-gateway: add package

[Fabian Bläse]

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/20190424/141b7645/attachment.sig>