Firmware Patch: removed unused tunneldigger package from NanoStation-Loco-M2 bsp

Tim Niemeyer tim.niemeyer at mastersword.de
Sa Jun 27 18:55:09 CEST 2015


Hi

Am Samstag, den 27.06.2015, 17:37 +0200 schrieb Tobias Klaus:
> ich hab mir das mit diesen Mail-Patches jetzt auch mal angeschaut. Kmail2 und 
> git scheinen sich da nicht sooo gut zu verstehen.... Aber na gut.
Versuch doch git send-email zu verwenden. Das wird z.B. auch von patman
verwendet.

> Zum Patch: ich finde ihn gut und kaputt machen tut er sicherlich auch nix, aber 
> was ich mich schon öfter bei ".config" Änderungen gefragt habe: Sollte da noch 
> ein "./buildscript config openwrt"  hinterhergeschickt werden und dann das 
> Ergebnis ins Repository rein, oder einfach nur so?
> 
> Ich denke, man sollte das eigentlich gleich durchs menuconfig, also 
> "./buildscript config openwrt" abwählen und dann dessen Ergebnis nehmen. Dann 
> ist quasi alles erledigt.
Ja, das muss dann aber eben auch immer passieren, wenn etwas anderes an
openWRT geändert wird. Früher oder später läuft das eh auseinander.

>  Anderseits hat das Bearbeiten der reinen Textdatei (so kam dieser Patch 
> zustande?) auch sehr viel Charm. Allein schon der Übersichtlichkeit halber. Um 
> das mal zu verdeutlichen, habe ich diesen Patch eingespielt und einfach nur 
> mal das openwrt config aufgerufen und gespeichert. Der resultierend Patch hängt 
> an dieser Mail.
Ich stimme dir zu, am saubersten wäre es, wenn man die Änderung über das
Tool macht. Dazu habe ich das ja auch damals ins buildscript gebaut.
Aber so lange die Datei schon out-of-sync ist, würde die gemachte
Änderung in diesem clean-up Sumpf auseinanderlaufen. So ne richtig
saubere Lösung gibts da einfach nicht. Ich denke man müsste es so wie
bei der bielefelder umbauen, dass in der config wirklich nur das
nötigste drin ist und der Rest über das defconfig eingebaut wird.

Tim

> Grüße
> Tobias
> 
> Am Donnerstag, 25. Juni 2015, 21:46:29 schrieb Steffen Pankratz:
> > Hi
> > 
> > wie besprochen (1), hier ein Patch, welcher Tunneldigger aus dem
> > NanoStation-Loco-M2 BSP entfernt.
> > 
> > 
> > Gruss
> > -Steffen
> > 
> > (1) https://github.com/kratz00/firmware/issues/4
> -- franken-dev mailing list franken-dev at freifunk.net http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net





Mehr Informationen über die Mailingliste franken-dev