[PATCH v4 1/2] fff-boardname: introduce new ubnt boards
mail at adrianschmutzler.de
mail at adrianschmutzler.de
So Mär 4 00:51:20 CET 2018
Hallo Tim,
siehe unten.
> -----Original Message-----
> From: Tim Niemeyer [mailto:tim at tn-x.org]
> Sent: Samstag, 3. März 2018 21:31
> To: mail at adrianschmutzler.de; franken-dev at freifunk.net
> Subject: Re: [PATCH v4 1/2] fff-boardname: introduce new ubnt boards
>
> Am Samstag, den 03.03.2018, 20:44 +0100 schrieb
> mail at adrianschmutzler.de:
> > Hallo Tim,
> >
> > > -----Original Message-----
> > > From: Tim Niemeyer [mailto:tim at tn-x.org]
> > > Sent: Samstag, 3. März 2018 20:34
> > > To: mail at adrianschmutzler.de; franken-dev at freifunk.net
> > > Subject: RE: [PATCH v4 1/2] fff-boardname: introduce new ubnt boards
> > >
> > > Hi
> > >
> > > Am 3. März 2018 20:27:18 MEZ schrieb mail at adrianschmutzler.de:
> > > > Hallo,
> > > >
> > > > > -----Original Message-----
> > > > > From: Tim Niemeyer [mailto:tim at tn-x.org]
> > > > > Sent: Samstag, 3. März 2018 20:24
> > > > > To: mail at adrianschmutzler.de; franken-dev at freifunk.net
> > > > > Subject: RE: [PATCH v4 1/2] fff-boardname: introduce new ubnt
> > > > > boards
> > > > >
> > > > > Hi
> > > > >
> > > > > Am 3. März 2018 20:06:29 MEZ schrieb mail at adrianschmutzler.de:
> > > > > > Hallo Tim,
> > > > > >
> > > > > > die neuen Boards musst du dann für das sysupgrade wieder
> > > > > > rückgängig machen, da wir ja nur die vier Images haben, die
> > > > > > OpenWRT
> > > > > > vorgibt:
> > > > > >
> > > > >
> > > > > https://github.com/FreifunkFranken/firmware/blob/master/src/pac
> > > > > kages
> > >
> > > /
> > > > > ff
> > > > > > f/fff
> > > > > > -sysupgrade/files/etc/sysupgrade.sh#L28
> > > > >
> > > > > Das können wir auch am update Server machen.
> > > > > Dann können wir auch die entsprechenden Downloads anbieten.
> > > >
> > > > Ist halt dann inkonsistent im Vergleich zur jetzigen Lösung mit
> > > > den CPEs.
> > > >
> > > > Wenn du das unbedingt so willst, dann sollte man das der Ordnung
> > > > halber auch am Updateserver machen.
> > >
> > > Vielleicht sollten wir die images einfach auch anlegen. Dann ist es
> > > einfacher das richtige image zu wählen.
> >
> > wenn man dein Konzept weiterdenkt, wäre das die Konsequenz: Sich
> > komplett von OpenWRT zu lösen und eine eigene Benennung der ubnt
> > Images zu wählen.
> Genau. Das ist aktuell ja sowieso durcheinander, z.B. wr1043 heißt in
> OpenWRT "tl-wr1043nd" das Image heißt aber "...tl-wr1043nd-v1...".
>
> Man kann das auch über ein "hübsches Download UI" machen. Irgendwas
> auf der Webseite oder so..
>
> Wegen mir kann ich auch die ubnt Image Namen ins sysupgrade.sh rein
> frickeln, aber irgendwie sieht es eher so aus, als ob das CPE Zeugs dort
> eigentlich eher ins fff-boardname gehört.
Der Imagename ist cpe-210-220. Der boardname ist cpe-210.
In meinen Augen ist der Boardname also sinnig. Ich würde also konsistent entweder für CPE und ubnt alles ins sysupgrade schreiben oder die cpe aus dem sysupgrade rausschmeißen und das für alle serverseitig lösen.
Grüße
Adrian
>
> Tim
>
> > Da kann man dann überlegen, ob man das im OpenWRT rumpatcht oder
> die
> > Images am Schluss einfach kopiert/verschiebt.
> >
> > Ich sehe schon den Charme deiner Lösung, ich bin nur grundsätzlich
> > eher dafür, eben nicht sieben Images anzubieten, wenn vier reichen.
> > Ich hatte ja auch schon Mal einen Patch geschickt, der die *.network
> > files für ubnt zusammenfassen sollte, weil ich vier identische Files
> > nicht sinnstiftend finde. Jetzt haben wir sieben oder acht
> > ubnt*.network Dateien mit gleichem Inhalt.
> >
> > Werde mir beide Patches im Laufe des abends mal in Ruhe ansehen.
> >
> > Grüße
> >
> > Adrian
> >
> >
> > >
> > > Tim
> > >
> > >
> > > > Grüße
> > > >
> > > > Adrian
> > > >
> > > >
> > > > >
> > > > > Tim
> > > > >
> > > > >
> > > > > > Das ist einer der Gründe, warum ich kein Fan dieser Lösung
> > > > > > war/bin
> > > >
> > > > und
> > > > > > lieber den Image-Namen von OpenWRT folgen wollte.
> > > > > >
> > > > > > Wenn du das hier noch einbauen willst, kann ich aber auch mit
> > > > > > deiner Lösung leben (und werde sie reviewen), ich finde es
> > > > > > halt nur unnötig kompliziert.
> > > > > >
> > > > > > Grüße
> > > > > >
> > > > > > Adrian
> > > > > >
> > > > > >
> > > > > >
> > > > > > > -----Original Message-----
> > > > > > > From: franken-dev [mailto:franken-dev-bounces at freifunk.net]
> > > > > > > On
> > > > >
> > > > > Behalf
> > > > > > > Of Tim Niemeyer
> > > > > > > Sent: Samstag, 3. März 2018 19:41
> > > > > > > To: franken-dev at freifunk.net
> > > > > > > Subject: [PATCH v4 1/2] fff-boardname: introduce new ubnt
> > > > > > > boards
> > > > > > >
> > > > > > > Signed-off-by: Tim Niemeyer <tim at tn-x.org>
> > > > > > > ---
> > > > > > >
> > > > > > > Changes in v4:
> > > > > > > - New patch
> > > > > > >
> > > > > > > src/packages/fff/fff-
> > > > > > > boardname/Makefile |
> > > >
> > > > 2
> > > > > > +-
> > > > > > > .../fff/fff-boardname/files/etc/uci-defaults/50-fff-
> > > > > > > boardname |
> > > >
> > > > 3
> > > > > > +++
> > > > > > > src/packages/fff/fff-network/ar71xx/network.ubnt-loco-
> > > > > > > m |
> > > >
> > > > 10
> > > > > > > ++++++++++
> > > > > > > src/packages/fff/fff-network/ar71xx/network.ubnt-pico-
> > > > > > > m |
> > > >
> > > > 10
> > > > > > > ++++++++++
> > > > > > > src/packages/fff/fff-network/ar71xx/network.ubnt-power-m-
> > > > > > > xw |
> > > >
> > > > 10
> > > > > > > ++++++++++
> > > > > > > 5 files changed, 34 insertions(+), 1 deletion(-) create
> > > > > > > mode
> > > >
> > > > 100644
> > > > > > > src/packages/fff/fff-network/ar71xx/network.ubnt-loco-m
> > > > > > > create mode 100644
> > > >
> > > > src/packages/fff/fff-network/ar71xx/network.ubnt-
> > > > > > > pico-m
> > > > > > > create mode 100644
> > > >
> > > > src/packages/fff/fff-network/ar71xx/network.ubnt-
> > > > > > > power-m-xw
> > > > > > >
> > > > > > > diff --git a/src/packages/fff/fff-boardname/Makefile
> > > > > >
> > > > > > b/src/packages/fff/fff-
> > > > > > > boardname/Makefile
> > > > > > > index 9848f80..d7ccd00 100644
> > > > > > > --- a/src/packages/fff/fff-boardname/Makefile
> > > > > > > +++ b/src/packages/fff/fff-boardname/Makefile
> > > > > > > @@ -1,7 +1,7 @@
> > > > > > > include $(TOPDIR)/rules.mk
> > > > > > >
> > > > > > > PKG_NAME:=fff-boardname
> > > > > > > -PKG_VERSION:=4
> > > > > > > +PKG_VERSION:=5
> > > > > > > PKG_RELEASE:=1
> > > > > > >
> > > > > > > PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME)
> > > > > > > diff --git
> > > > > >
> > > > > > a/src/packages/fff/fff-boardname/files/etc/uci-defaults/50-
> > > > > > fff-
> > > > > > > boardname
> > > > > >
> > > > > > b/src/packages/fff/fff-boardname/files/etc/uci-defaults/50-
> > > > > > fff-
> > > > > > > boardname
> > > > > > > index 7d4cdb6..db2f253 100644
> > > > > > > ---
> > > >
> > > > a/src/packages/fff/fff-boardname/files/etc/uci-defaults/50-fff-
> > > > > > > boardname
> > > > > > > +++
> > > > > >
> > > > > > b/src/packages/fff/fff-boardname/files/etc/uci-defaults/50-
> > > > > > fff-board
> > > > > > > +++ name
> > > > > > > @@ -34,12 +34,15 @@ case "$BOARD" in
> > > > > > > ;;
> > > > > > > bullet-m)
> > > > > > > BOARD=ubnt-bullet-m
> > > > > > > + grep -i "Pico" /var/sysinfo/model && BOARD=ubnt-
> > > > > > > pico-m
> > > > > > > + grep -i "Loco" /var/sysinfo/model && BOARD=ubnt-
> > > > > > > loco-m
> > > > > > > ;;
> > > > > > > unifi)
> > > > > > > BOARD=ubnt-unifi
> > > > > > > ;;
> > > > > > > loco-m-xw)
> > > > > > > BOARD=ubnt-loco-m-xw
> > > > > > > + grep -i "Power" /var/sysinfo/model &&
> > > >
> > > > BOARD=ubnt-power-m-xw
> > > > > > > ;;
> > > > > > > unifiac-lite)
> > > > > > > BOARD=ubnt-unifiac-lite diff --git
> > > >
> > > > a/src/packages/fff/fff-network/ar71xx/network.ubnt-loco-m
> > > > > > > b/src/packages/fff/fff-network/ar71xx/network.ubnt-loco-m
> > > > > > > new file mode 100644
> > > > > > > index 0000000..372750c
> > > > > > > --- /dev/null
> > > > > > > +++ b/src/packages/fff/fff-network/ar71xx/network.ubnt-
> > > > > > > loco-m
> > > > > > > @@ -0,0 +1,10 @@
> > > > > > > +WANDEV=eth0
> > > > > > > +SWITCHDEV=eth0
> > > > > > > +CLIENT_PORTS=""
> > > > > > > +WAN_PORTS=""
> > > > > > > +BATMAN_PORTS=""
> > > > > > > +ONE_PORT="YES"
> > > > > > > +
> > > > > > > +. /etc/network.mode
> > > > > > > +
> > > > > > > +ROUTERMAC=$(cat /sys/class/ieee80211/phy0/macaddress)
> > > > > > > diff --git
> > > >
> > > > a/src/packages/fff/fff-network/ar71xx/network.ubnt-pico-m
> > > > > > > b/src/packages/fff/fff-network/ar71xx/network.ubnt-pico-m
> > > > > > > new file mode 100644
> > > > > > > index 0000000..372750c
> > > > > > > --- /dev/null
> > > > > > > +++ b/src/packages/fff/fff-network/ar71xx/network.ubnt-
> > > > > > > pico-m
> > > > > > > @@ -0,0 +1,10 @@
> > > > > > > +WANDEV=eth0
> > > > > > > +SWITCHDEV=eth0
> > > > > > > +CLIENT_PORTS=""
> > > > > > > +WAN_PORTS=""
> > > > > > > +BATMAN_PORTS=""
> > > > > > > +ONE_PORT="YES"
> > > > > > > +
> > > > > > > +. /etc/network.mode
> > > > > > > +
> > > > > > > +ROUTERMAC=$(cat /sys/class/ieee80211/phy0/macaddress)
> > > > > > > diff --git
> > > > > >
> > > > > > a/src/packages/fff/fff-network/ar71xx/network.ubnt-power-m-xw
> > > > > > > b/src/packages/fff/fff-network/ar71xx/network.ubnt-power-m-
> > > > > > > xw
> > > > > > > new file mode 100644
> > > > > > > index 0000000..372750c
> > > > > > > --- /dev/null
> > > > > > > +++ b/src/packages/fff/fff-network/ar71xx/network.ubnt-
> > > > > > > power-m-
> > >
> > > xw
> > > > > > > @@ -0,0 +1,10 @@
> > > > > > > +WANDEV=eth0
> > > > > > > +SWITCHDEV=eth0
> > > > > > > +CLIENT_PORTS=""
> > > > > > > +WAN_PORTS=""
> > > > > > > +BATMAN_PORTS=""
> > > > > > > +ONE_PORT="YES"
> > > > > > > +
> > > > > > > +. /etc/network.mode
> > > > > > > +
> > > > > > > +ROUTERMAC=$(cat /sys/class/ieee80211/phy0/macaddress)
> > > > > > > --
> > > > > > > 2.11.0
> > > > > > >
> > > > > > > --
> > > > > > > franken-dev mailing list
> > > > > > > franken-dev at freifunk.net
> > > > > > >
> > > >
> > > > http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.n
> > > > et
> >
> >
Mehr Informationen über die Mailingliste franken-dev