[Berlin-wireless] olsrd 0.5.5 in Kamikaze generalueberholt... :)

[Felix Fietkau]

Alina Friedrichsen wrote:
> Hi,
> 
> so hier entlich der olsrd 0.5.5 mit foellig neuen init.d-Script und ner
> menge Bug-Fixes. Das init.d-Script leuft jetzt wesentlich schneller
> durch und konfiguriert den olsrd jetzt weitgehend automatisch, so das Du
> z.B. Deinen Hostname fuer das Nameservice-Plugin nicht mehr doppelt
> angeben musst. (Achja, die Geo-Koordinaten des Nodes lassen sich jetzt
> auch systemweit mit uci set system.cfg1.latlon='12.123 12.123'
> einstellen) Des weiteren ist so jetzt auch die uci-Konfiguration
> wesentlich uebersichtlicher. Aber keine Sorge, jede
> Konfigurations-Eingenschaft laesst sich natuerlich auch weiterhin voll
> und ganz ueberschreiben, so das die Autoconfig Dir dabei nicht im Wege
> steht. Einzieges Manko: Das Script erfordert eine robuste und sichere
> uci-Implementation, wie ich sie einglich mit Felix geplant hatte. Es
> lauft aber auch noch auf der jetziegen Shell-Implementaion. Es empfielt
> sich aber nach einer Konfigurationsaenderung ein uci commit zu machen.
> Des weiteren hab ich jetzt auch so einiege Bugs im Makefile von OpenWrt
> gefixt. Alles im allem sollte jetzt OLSR mit Kamikaze wesentlich mehr
> Spass machen. Sollte es doch noch einen Bug geben, bitte ne E-Mail an
> mich. Fragen zur neuen uci-Config beantworte ich auch gerne. :)
Hi Alina,

die von dir erwähnte Shell-Implementation wird in trunk nicht mehr
verwendet und ist durch meine C Implementation ersetzt worden. Die Dateien
in /var/state sind im gleichen Format, wie die noch nicht committeten
Änderungen in /tmp/.uci, welche bei uci set gespeichert werden.
Das heißt, dass ein config_load /var/state/network nicht mehr gemacht
werden sollte, da config_load network auch /var/state berücksichtigt (aber
nur beim Laden, nicht beim Commit).
Ansonsten kann man den Code etwas übersichtlicher machen (finde ich
zumindest), indem man die append() funktion aus /etc/functions.sh nimmt.
Beispiel:

> [ -z "$value" ] && current_interface="$current_interface${N}${TAB}HelloInterval${TAB}6.0"
wird zu
> [ -z "$value" ] && append current_interface "${TAB}HelloInterval${TAB}6.0" "$N"

Ansonsten sieht's gut aus. Ich werd's demnächst mal testen und mergen.

- Felix