[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