[PATCH v3 2/2] Allow initial configuration from Ethernet

[Fabian Bläse]

> On 11. Jun 2018, at 20:57, Tim Niemeyer <tim at tn-x.org> wrote:
> 
> Hi
> 
> Wenn du die drei Sachen einbaust, kannst du mein Review dran packen.
> 
> Am Montag, den 11.06.2018, 19:58 +0200 schrieb Fabian Bläse:
>> 
>> +getEthernetHoodfile() {
>> +	if [ $# != 1 ]; then
>> +			echo "Usage: getEthernetHoodfile <target-file>"
>> +			return 1
> 
> Hier ist die Einrückung kaputt.

Wo du Recht hast, hast du Recht.

>> +
>> +	neighbor_addrs=$(ping6 -c2 ff02::1%${ethmesh_dev} | grep seq | cut -d " " -f4 | sed s/:$//g | sort | uniq)
> 
>                                            ^-- SC2086: Double quote to prevent globbing and word splitting.

Bau ich ein, man kann nie sicher genug sein.

> Außerdem könntest du doch uniq sparen und "sort -u" nehmen.

Ansichtssache, mir aber egal, würde ich auch übernehmen.

Gruß
Fabian
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 833 bytes
Beschreibung: Message signed with OpenPGP
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20180611/7526a169/attachment.sig>