[PATCH v2 0/7] Nodewatcher fixes und style fixes

[Tobias Klaus]

Reviewed-by: Tobias Klaus <tk+ff at meskal.net>

und mit 2 tabs weniger und einem typo weniger eingespielt.

Danke fürs überarbeiten, wie schon besprochen, sollten wir uns einfach nochmal 
zusammensetzen.

Grüße
Tobias

Am Samstag, 11. Juni 2016, 17:20:45 CEST schrieb Tim Niemeyer:
> Auf dem WDR3900 gab es Probleme mit dem Nodewatcher, die mit diesem
> Patchset behoben sein sollte. Weiter habe ich mir den Nodewatcher
> mit Shellcheck angeschaut und weitere potentielle Fehler behoben.
> 
> Für die L3 Router ist es wichtig, dass der Nodewatcher nicht crashed,
> nur weil kein fastd installiert ist.
> 
> Eigentlich wollte ich das Patchset nicht auf Spaces umstellen und
> dahingehend überarbeiten, weil ich mir immer noch sicher bin, dass es
> keine Einigung auf spaces gab. Weiter finde ich es nicht i.O.  Patches
> aufgrund von Befindlichkeiten bzw persönlich, optischen Vorlieben zu
> blockieren. Letztlich hält das die Entwicklung auf, obliegt aber keiner
> Sachlichen Grundlage. Nun ja. Jetzt hat mich das wieder ne Stunde
> gekostet, aber ich hoffe wir kommen damit jetzt weiter.
> 
> Tim Niemeyer (7):
>   fff-nodewatcher: style: remove unused code
>   fff-nodewatcher: style: unify to spaces
>   fff-nodewatcher: style: fix some long lines
>   fff-nodewatcher: add platform as chipset
>   fff-nodewatcher: fastd version only if installed
>   fff-nodewatcher: fix shellcheck warnings
>   fff-nodewatcher: bump version
> 
>  src/packages/fff/fff-nodewatcher/Makefile          |   2 +-
>  .../fff/fff-nodewatcher/files/usr/sbin/nodewatcher | 272
> +++++++++++---------- 2 files changed, 145 insertions(+), 129 deletions(-)