[PATCH 2/2] fff-boardname/fff-network: Reuse network.* files for equal boards
Tim Niemeyer
tim at tn-x.org
Fr Nov 3 13:33:58 CET 2017
Hi
Am Montag, den 23.10.2017, 12:27 +0200 schrieb Adrian Schmutzler:
> This patch introduces a second entry in the board config file
> which is used only for the network setup. This enables us to
> delete a lot of redundant files.
Das kann man machen, aber außer das wir ein paar Files sparen sehe ich
da jetzt keinen Vorteil.
Eigentlich erhöht sich nur die Komplexität etwas, was sogar ein kleiner
Nachteil ist.
Ich würde das hier auf jeden Fall erstmal ganz weit nach hinten stellen
(also erstmal ablehnen), weil mir der Patch mit dem Switch im Web-Ui
sehr gefällt und ich mir nicht sicher bin, ob das PORTORDER auch in
dieser Art und Weise aggregiert werden kann oder ob sich dann doch
wieder spezielle Eigenheiten pro Gerät ergeben.
Selbst wenn nicht, haben wir vielleicht irgendwann mal eine und dann
ist es ein hin und her kopiere usw .. daher wäre die geringere
Flexibilität ein zweiter Nachteil.
Tim
> Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> ---
> .../files/etc/uci-defaults/50-fff-boardname | 21
> +++++++++++++++++++--
> .../fff/fff-network/ar71xx/network.tl-wdr3600-v1 | 8 --------
> .../fff/fff-network/ar71xx/network.tl-wdr4310-v1 | 8 --------
> .../fff/fff-network/ar71xx/network.tl-wr1043nd-v3 | 8 --------
> .../fff/fff-network/ar71xx/network.tl-wr841-v10 | 10 ----------
> .../fff/fff-network/ar71xx/network.tl-wr841-v11 | 10 ----------
> .../fff/fff-network/ar71xx/network.tl-wr841-v12 | 10 ----------
> .../fff/fff-network/ar71xx/network.tl-wr841-v9 | 2 ++
> .../fff/fff-network/ar71xx/network.ubnt-bullet-m | 11 -----------
> .../fff/fff-network/ar71xx/network.ubnt-loco-m-xw | 11 -----------
> .../fff/fff-network/ar71xx/network.ubnt-oneport | 11 +++++++++++
> .../fff/fff-network/ar71xx/network.ubnt-unifi | 11 -----------
> .../fff/fff-network/files/usr/sbin/configurenetwork | 2 +-
> .../fff/fff-support/files/usr/sbin/show_info | 2 +-
> .../fff/fff-web/files/www/ssl/cgi-bin/ports.html | 2 +-
> 15 files changed, 35 insertions(+), 92 deletions(-)
> delete mode 100644 src/packages/fff/fff-network/ar71xx/network.tl-
> wdr3600-v1
> delete mode 100644 src/packages/fff/fff-network/ar71xx/network.tl-
> wdr4310-v1
> delete mode 100644 src/packages/fff/fff-network/ar71xx/network.tl-
> wr1043nd-v3
> delete mode 100644 src/packages/fff/fff-network/ar71xx/network.tl-
> wr841-v10
> delete mode 100644 src/packages/fff/fff-network/ar71xx/network.tl-
> wr841-v11
> delete mode 100644 src/packages/fff/fff-network/ar71xx/network.tl-
> wr841-v12
> delete mode 100644 src/packages/fff/fff-network/ar71xx/network.ubnt-
> bullet-m
> delete mode 100644 src/packages/fff/fff-network/ar71xx/network.ubnt-
> loco-m-xw
> create mode 100644 src/packages/fff/fff-network/ar71xx/network.ubnt-
> oneport
> delete mode 100644 src/packages/fff/fff-network/ar71xx/network.ubnt-
> unifi
>
> 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 5470405..1d87ded 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-
> boardname
> @@ -63,5 +63,22 @@ case "$BOARD" in
> ;;
> esac
>
> -uci set board.model.name=$BOARD
> -uci commit board
> +NETWORK="$BOARD"
> +case "$NETWORK" in
> + tl-wr1043nd-v3)
> + NETWORK=tl-wr1043nd-v2
> + ;;
> + tl-wr841-v10|tl-wr841-v11|tl-wr841-v12)
> + NETWORK=tl-wr841-v9
> + ;;
> + ubnt-bullet-m|ubnt-unifi|ubnt-loco-m-xw)
> + NETWORK=ubnt-oneport
> + ;;
> + tl-wdr3600-v1|tl-wdr4310-v1)
> + NETWORK=tl-wdr4300-v1
> + ;;
> +esac
> +
> +uci -q set "board.model.name=$BOARD"
> +uci -q set "board.model.net=$NETWORK"
> +uci -q commit board
> diff --git a/src/packages/fff/fff-network/ar71xx/network.tl-wdr3600-
> v1 b/src/packages/fff/fff-network/ar71xx/network.tl-wdr3600-v1
> deleted file mode 100644
> index 897152e..0000000
> --- a/src/packages/fff/fff-network/ar71xx/network.tl-wdr3600-v1
> +++ /dev/null
> @@ -1,8 +0,0 @@
> -WANDEV=eth0
> -SWITCHDEV=eth0
> -CLIENT_PORTS="0t 4 5"
> -WAN_PORTS="0t 1"
> -BATMAN_PORTS="0t 2 3"
> -
> -ETHMESHMAC=
> -ROUTERMAC=w5mesh
> diff --git a/src/packages/fff/fff-network/ar71xx/network.tl-wdr4310-
> v1 b/src/packages/fff/fff-network/ar71xx/network.tl-wdr4310-v1
> deleted file mode 100644
> index 897152e..0000000
> --- a/src/packages/fff/fff-network/ar71xx/network.tl-wdr4310-v1
> +++ /dev/null
> @@ -1,8 +0,0 @@
> -WANDEV=eth0
> -SWITCHDEV=eth0
> -CLIENT_PORTS="0t 4 5"
> -WAN_PORTS="0t 1"
> -BATMAN_PORTS="0t 2 3"
> -
> -ETHMESHMAC=
> -ROUTERMAC=w5mesh
> diff --git a/src/packages/fff/fff-network/ar71xx/network.tl-wr1043nd-
> v3 b/src/packages/fff/fff-network/ar71xx/network.tl-wr1043nd-v3
> deleted file mode 100644
> index 6dd4674..0000000
> --- a/src/packages/fff/fff-network/ar71xx/network.tl-wr1043nd-v3
> +++ /dev/null
> @@ -1,8 +0,0 @@
> -WANDEV=eth0
> -SWITCHDEV=eth0
> -CLIENT_PORTS="1 2 6t"
> -WAN_PORTS="5 6t"
> -BATMAN_PORTS="3 4 6t"
> -
> -ETHMESHMAC=
> -ROUTERMAC=eth1
> diff --git a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v10
> b/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v10
> deleted file mode 100644
> index 4d22fde..0000000
> --- a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v10
> +++ /dev/null
> @@ -1,10 +0,0 @@
> -PORTORDER="4 3 2 1"
> -
> -WANDEV=eth1
> -SWITCHDEV=eth0
> -CLIENT_PORTS="1 2 0t"
> -WAN_PORTS=
> -BATMAN_PORTS="3 4 0t"
> -
> -ETHMESHMAC=eth1
> -ROUTERMAC=eth0
> diff --git a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v11
> b/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v11
> deleted file mode 100644
> index 4d22fde..0000000
> --- a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v11
> +++ /dev/null
> @@ -1,10 +0,0 @@
> -PORTORDER="4 3 2 1"
> -
> -WANDEV=eth1
> -SWITCHDEV=eth0
> -CLIENT_PORTS="1 2 0t"
> -WAN_PORTS=
> -BATMAN_PORTS="3 4 0t"
> -
> -ETHMESHMAC=eth1
> -ROUTERMAC=eth0
> diff --git a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v12
> b/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v12
> deleted file mode 100644
> index 4d22fde..0000000
> --- a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v12
> +++ /dev/null
> @@ -1,10 +0,0 @@
> -PORTORDER="4 3 2 1"
> -
> -WANDEV=eth1
> -SWITCHDEV=eth0
> -CLIENT_PORTS="1 2 0t"
> -WAN_PORTS=
> -BATMAN_PORTS="3 4 0t"
> -
> -ETHMESHMAC=eth1
> -ROUTERMAC=eth0
> diff --git a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v9
> b/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v9
> index 1d3f146..4d22fde 100644
> --- a/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v9
> +++ b/src/packages/fff/fff-network/ar71xx/network.tl-wr841-v9
> @@ -1,3 +1,5 @@
> +PORTORDER="4 3 2 1"
> +
> WANDEV=eth1
> SWITCHDEV=eth0
> CLIENT_PORTS="1 2 0t"
> diff --git a/src/packages/fff/fff-network/ar71xx/network.ubnt-bullet-
> m b/src/packages/fff/fff-network/ar71xx/network.ubnt-bullet-m
> deleted file mode 100644
> index 307a201..0000000
> --- a/src/packages/fff/fff-network/ar71xx/network.ubnt-bullet-m
> +++ /dev/null
> @@ -1,11 +0,0 @@
> -WANDEV=eth0
> -SWITCHDEV=eth0
> -CLIENT_PORTS=""
> -WAN_PORTS=""
> -BATMAN_PORTS=""
> -ONE_PORT="YES"
> -
> -. /etc/network.mode
> -
> -ETHMESHMAC=
> -ROUTERMAC=w2mesh
> \ No newline at end of file
> diff --git a/src/packages/fff/fff-network/ar71xx/network.ubnt-loco-m-
> xw b/src/packages/fff/fff-network/ar71xx/network.ubnt-loco-m-xw
> deleted file mode 100644
> index 307a201..0000000
> --- a/src/packages/fff/fff-network/ar71xx/network.ubnt-loco-m-xw
> +++ /dev/null
> @@ -1,11 +0,0 @@
> -WANDEV=eth0
> -SWITCHDEV=eth0
> -CLIENT_PORTS=""
> -WAN_PORTS=""
> -BATMAN_PORTS=""
> -ONE_PORT="YES"
> -
> -. /etc/network.mode
> -
> -ETHMESHMAC=
> -ROUTERMAC=w2mesh
> \ No newline at end of file
> diff --git a/src/packages/fff/fff-network/ar71xx/network.ubnt-oneport
> b/src/packages/fff/fff-network/ar71xx/network.ubnt-oneport
> new file mode 100644
> index 0000000..307a201
> --- /dev/null
> +++ b/src/packages/fff/fff-network/ar71xx/network.ubnt-oneport
> @@ -0,0 +1,11 @@
> +WANDEV=eth0
> +SWITCHDEV=eth0
> +CLIENT_PORTS=""
> +WAN_PORTS=""
> +BATMAN_PORTS=""
> +ONE_PORT="YES"
> +
> +. /etc/network.mode
> +
> +ETHMESHMAC=
> +ROUTERMAC=w2mesh
> \ No newline at end of file
> diff --git a/src/packages/fff/fff-network/ar71xx/network.ubnt-unifi
> b/src/packages/fff/fff-network/ar71xx/network.ubnt-unifi
> deleted file mode 100644
> index 307a201..0000000
> --- a/src/packages/fff/fff-network/ar71xx/network.ubnt-unifi
> +++ /dev/null
> @@ -1,11 +0,0 @@
> -WANDEV=eth0
> -SWITCHDEV=eth0
> -CLIENT_PORTS=""
> -WAN_PORTS=""
> -BATMAN_PORTS=""
> -ONE_PORT="YES"
> -
> -. /etc/network.mode
> -
> -ETHMESHMAC=
> -ROUTERMAC=w2mesh
> \ No newline at end of file
> diff --git a/src/packages/fff/fff-
> network/files/usr/sbin/configurenetwork b/src/packages/fff/fff-
> network/files/usr/sbin/configurenetwork
> index d4866e4..eb4a3ef 100755
> --- a/src/packages/fff/fff-network/files/usr/sbin/configurenetwork
> +++ b/src/packages/fff/fff-network/files/usr/sbin/configurenetwork
> @@ -11,7 +11,7 @@
>
> . /lib/functions/fff/network
>
> -BOARD="$(uci get board.model.name)"
> +BOARD="$(uci get board.model.net)"
> . /etc/network.$BOARD
>
> if [ -s /etc/network.config ] ; then
> diff --git a/src/packages/fff/fff-support/files/usr/sbin/show_info
> b/src/packages/fff/fff-support/files/usr/sbin/show_info
> index 1e7c498..61323f4 100755
> --- a/src/packages/fff/fff-support/files/usr/sbin/show_info
> +++ b/src/packages/fff/fff-support/files/usr/sbin/show_info
> @@ -2,7 +2,7 @@
>
> . /etc/firmware_release
>
> -board_name=$(uci -q get board.model.name)
> +board_name=$(uci -q get board.model.net)
>
> HOSTNAME=$(uci -q get 'system. at system[0].hostname')
>
> diff --git a/src/packages/fff/fff-web/files/www/ssl/cgi-
> bin/ports.html b/src/packages/fff/fff-web/files/www/ssl/cgi-
> bin/ports.html
> index 5912126..e9ae231 100755
> --- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/ports.html
> +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/ports.html
> @@ -1,7 +1,7 @@
> #!/usr/bin/haserl
>
> <%
> -board_name=$(uci -q get board.model.name)
> +board_name=$(uci -q get board.model.net)
> # write
> if [ "$REQUEST_METHOD" == "POST" ] ; then
> if [ "$POST_change_mode" != "" ] ; then
> --
> 2.7.4
>
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname : signature.asc
Dateityp : application/pgp-signature
Dateigröße : 488 bytes
Beschreibung: This is a digitally signed message part
URL : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20171103/c518af8f/attachment.sig>
Mehr Informationen über die Mailingliste franken-dev