[RFC 0/4] FastD durch Tunneldigger ersetzt

[Tim Niemeyer]

Am Montag, den 14.03.2016, 22:47 +0100 schrieb mayosemmel:
> Hallo Tim,
> 
> nach den 3 oder 4 Antworten zu dem Thema fasse ich meine Stellungnahme
> einfach mal zusammen:
> 
> Dieses Patchset sollte für nicht mehr gut sein, als den Leuten, die das
> Testen wollen eine Grundlage zu geben und transparent zu machen, was in
> den binaries drin ist.
Ah ok, dann hab ich das falsch verstanden. Aber ist ja nicht
schlimm. :-)

> Weiterhin gab es in der Zwischenzeit auch noch einen Patch von asdil12,
> welcher hier eigentlich mit reingehört.
Du meinst den Kernel Patch?

> Das du zu dem Thema schonmal nen Patch verfasst hast, war mir nicht
> bekannt. Kannst du den ggf. nochmal posten?
http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/2016-February/002859.html
http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/2016-February/002858.html
http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/2016-February/002857.html

> Mit den Packages hast du durchaus recht. Ich hab das wie gesagt einfach
> mal Hart reingepatcht und es ist nur für Testzwecke. Wie soll man sowas
> in Zukunft betiteln, wenn nicht als RFC, um Missverständnisse zu
> vermeiden?
Ich finde das ist eigentlich alles ganz gut gelaufen. RFC steht für mich
für: "Schaut mal hier, gebt mir für den groben Aufbau bitte Feedback
(oder anders in dem Cover-Letter spezifiziert). Da es nicht fertig ist,
bitte nicht applien."
Gern kannst du auch ans Patchset schreiben: Kein review nötig, soll hier
nur ins Archiv. Oder irgendwie sowas. Aber selbst dann schaden ja
Kommentare nicht. ;)

So in etwa lief es ja jetzt auch. Nehm meine Kommentare einfach als
zusätzlichen Input, wenn du irgendwann an dem Patchset weiter arbeitest.
Ansonsten kannst du es auch einfach ignorieren.

Tim

> Hoffe ich hab keinen deiner Kommentare übersehen.
> 
> Grüße Jan
> 
> Am Dienstag, den 08.03.2016, 15:51 +0100 schrieb Jan Kraus:
> > Hallo zusammen,
> > 
> > wie versprochen habe ich das ganze Zeug bezüglich Tunneldigger mal in eine Patchserie gepackt.
> > Wenn ihr das nutzen wollt und nicht in Nürnberg unterwegs seid, bitte unbedingt in /etc/config/tunneldigger den Server an die entsprechende Hood anpassen!
> > 
> > Fertig durchgebaut findet ihr das ganze auf meinem GW:
> > öffentlich: http://fff-gw-m1.fff.community/firmware/tunneldigger/
> > FF-intern: http://fff-gw-m1.gw.fff.community/firmware/tunneldigger/
> > 
> > Viele Grüße
> > Jan
> > 
> > Jan Kraus (4):
> >   Added tunneldigger
> >   Added tunneldigger into config
> >   Removed fastd
> >   Added configuration for tunneldigger     Be careful, Server for
> >     Nürnberg is hardcoded
> > 
> >  .gitignore                                         |  1 +
> >  bsp/ar71xx/.config                                 | 17 ++--
> >  bsp/default/root_file_system/etc/bat-hosts         |  1 -
> >  .../root_file_system/etc/config/tunneldigger       |  8 ++
> >  bsp/default/root_file_system/etc/fastdstart.sh.tpl | 95 ----------------------
> >  .../etc/hotplug.d/iface/50-fastdstart              |  5 --
> >  bsp/default/root_file_system/etc/tunneldigger.hook | 19 +++++
> >  .../root_file_system/usr/lib/micron.d/default      |  1 -
> >  buildscript                                        | 10 ++-
> >  9 files changed, 42 insertions(+), 115 deletions(-)
> >  delete mode 120000 bsp/default/root_file_system/etc/bat-hosts
> >  create mode 100644 bsp/default/root_file_system/etc/config/tunneldigger
> >  delete mode 100755 bsp/default/root_file_system/etc/fastdstart.sh.tpl
> >  delete mode 100755 bsp/default/root_file_system/etc/hotplug.d/iface/50-fastdstart
> >  create mode 100755 bsp/default/root_file_system/etc/tunneldigger.hook
> > 
> 

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 473 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20160315/8071600a/attachment-0002.sig>