[PATCH 1/3] bsp: Move package selection to Makefile

mail at adrianschmutzler.de mail at adrianschmutzler.de
Sa Mai 25 19:21:00 CEST 2019


Hallo Fabian,

 

wie soll das dann werden? Vielleicht so:

 

Patch 1:

nls-base in fff/Makefile schieben (würde ich vorsichtshalber behalten).

 

Patch 2:

LED-Zeug weg (direkt aus dem .config)

 

Patch 3:

Traffic-Shaping weg (incl. qos, direct aus dem .config)

 

Patch 4:

Leerzeilen einfügen

 

Wenn wir dann feststellen, dass wir das Traffic-Shaping brauchen, dann fügen wir es da wieder ein, wo was kaputt geht.

 

Einverstanden? Wenn ja, schicke ich das ganze so nochmal neu.

 

Grüße

 

Adrian

 

 

 

From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf Of Fabian Bläse
Sent: Samstag, 25. Mai 2019 19:12
To: Adrian Schmutzler <freifunk at adrianschmutzler.de>; franken-dev at freifunk.net
Subject: Re: [PATCH 1/3] bsp: Move package selection to Makefile

 

Hallo Adrian, 

On 23.05.19 15:10, Adrian Schmutzler wrote: 
> While at it, increase .config readability by introducing blocks 
> of CONFIG lines. 
Mir gefällt das eigentlich gar nicht, das mit in diesen commit rein zu mischen. 
Das macht es schwieriger, Änderungen nachzuvollziehen. 

> diff --git a/src/packages/fff/fff/Makefile b/src/packages/fff/fff/Makefile 
> index 4fec4d39..bbd89a45 100644 
> --- a/src/packages/fff/fff/Makefile 
> +++ b/src/packages/fff/fff/Makefile 
> @@ -1,8 +1,7 @@ 
>  include $(TOPDIR)/rules.mk 
>  
>  PKG_NAME:=fff 
> -PKG_VERSION:=0.0.2 
> -PKG_RELEASE:=5 
> +PKG_RELEASE:=6 
>  
>  PKG_BUILD_DIR:=$(BUILD_DIR)/fff 
>  
> @@ -17,6 +16,13 @@ define Package/fff/base 
>                        +ip6tables \ 
>                        +odhcp6c \ 
>                        +micrond \ 
> +                      +kmod-ifb \ 
> +                      +kmod-ledtrig-heartbeat \ 
> +                      +kmod-ledtrig-netdev \ 
> +                      +kmod-nls-base \ 
> +                      +kmod-sched-connmark \ 
> +                      +qos-scripts \ 
> +                      +tc \ 

Ich bin mir relativ sicher, dass ifb, sched-connmark und tc wenn überhaupt nur für das Traffic Shaping und den macnocker drin sind.

Entweder sollten die zu dem Paket, dass die Sachen braucht, oder wir werfen das ganze Traffic Shaping gleich raus... 

Soweit ich das richtig verstehe, verwenden wir eh simple-tc aus den gluon packages für das Shaping. Das sollte alles was es braucht selbst selektieren.

Der macnocker tut das scheinbar nicht, der braucht auf jeden Fall tc. 

Gruß 
Fabian 

 
-------------- nächster Teil --------------
Ein Dateianhang mit HTML-Daten wurde abgetrennt...
URL: <https://{'listname': 'franken-dev-freifunk.net', 'hostname': 'lists.freifunk.net'}/pipermail/franken-dev-freifunk.net/attachments/20190525/10b7316a/attachment.html>
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : openpgp-digital-signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 834 bytes
Beschreibung: nicht verfügbar
URL         : <https://{'listname': 'franken-dev-freifunk.net', 'hostname': 'lists.freifunk.net'}/pipermail/franken-dev-freifunk.net/attachments/20190525/10b7316a/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev