Hmm, subshells. Ob das wieder ich war? Das sollte man auch mal ordentlich machen, aber auch hier separat.
Der Ordnung halber könnte man hier noch das json_select ".." # back to root
wieder einfügen, funktional dürfte es egal sein.
Statt prüfen, Ausgabe unterdrücken, und dann echo könnte man auch einfach den grep direkt das richtige ausgeben lassen (-o). Den Fehler kann man dann auch gleich mit (-q) unterdrücken:
Bitte aber nur als Diskussionsanregung verstehen. Sowas würde ich wenn überhaupt nur in einem separaten Patch anfassen wollen ...
Ich frage mich gerade, ob es klug ist, hier zeilenweise zu schreiben. Da die Dateien aber auf /tmp im RAM liegen, ist es an der Stelle wahrscheinlich wurscht. Keine Ahnung ob die Buffer da Effekte haben...
Hmm, gibts es da theoretische Vor- oder Nachteile im Vergleich zu mkdir -p /tmp/fastd_fff_peers ohne test?
Wenn man voraussetzt, das jedes Protokoll in protocol_order eingetragen werden muss, könnte man das etwas vereinfachen:
Hier würde ggf. auch z.B. "fastd" mit "fastd2" gematcht. Ggf. sollte man word-boundaries in den grep mit einführen.
Zur Info wer selbst bauen will: Ich hab vergessen das package irgendwo zu selektieren, nachdem alle Abhängigkeiten aus der commitmsg und dieses Patch angewahnt wurden, muss noch:
…
password.html macht auch nichts anderes, als passwd aufzurufen ...
Eigenlich unterstützen wir noch kein vxlan, insofern würde ich das hier rausnehmen, bis es offiziell unterstützt wird.
Ich habe mal alle Commits außer dem mit den Depencies gepickt und gemergt.