[PATCH 4/4] BUSYBOX_CONFIG_FEATURE_WGET_TIMEOUT

Adrian Schmutzler mail at adrianschmutzler.de
Di Jun 25 15:00:56 CEST 2019


Hallo Robert,

hier würde ich jetzt tatsächlich meinen Patch lieber nehmen.

Zu deinem:

Ich würde den Commit-Namen etwas klarer machen ;-)

Die Dependency in fff-config kann noch mit weg.

Rest unten:

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf Of
> Robert Langhammer
> Sent: Dienstag, 25. Juni 2019 14:34
> To: franken-dev at freifunk.net
> Subject: [PATCH 4/4] BUSYBOX_CONFIG_FEATURE_WGET_TIMEOUT
> 
> fff-vpn-select depends no longer on
> BUSYBOX_CONFIG_FEATURE_WGET_TIMEOUT. It can be removed but this is
> the only one dependence to ..WGET_TIMEOUT. The timeout is still used by fff-
> hoods, fff-sysupgrade and fff-web.
> 
> Signed-off-by: Robert Langhammer <rlanghammer at web.de>
> ---
>  src/packages/fff/fff-hoods/Makefile      |  7 ++++++-
>  src/packages/fff/fff-sysupgrade/Makefile |  6 +++++-
>  src/packages/fff/fff-vpn-select/Makefile |  3 +--
>  src/packages/fff/fff-web/Makefile        | 12 +++++++++++-
>  4 files changed, 23 insertions(+), 5 deletions(-)
> 
> diff --git a/src/packages/fff/fff-hoods/Makefile b/src/packages/fff/fff-
> hoods/Makefile
> index f1a6289..245fe33 100644
> --- a/src/packages/fff/fff-hoods/Makefile
> +++ b/src/packages/fff/fff-hoods/Makefile
> @@ -13,7 +13,12 @@ define Package/$(PKG_NAME)
>  	CATEGORY:=Freifunk
>  	TITLE:= Freifunk-Franken hoods
>  	URL:=http://www.freifunk-franken.de
> -	DEPENDS:=+fff-hoodutils +fff-macnock +fff-vpn-select +fff-firewall
> +	DEPENDS:=+fff-hoodutils \
> +		+fff-macnock \
> +		+fff-vpn-select \
> +		+fff-firewall \
> +		+ at BUSYBOX_CONFIG_WGET \
> +		+ at BUSYBOX_CONFIG_FEATURE_WGET_TIMEOUT

Das sieht schöner aus als bei mir.

>  endef
> 
>  define Package/$(PKG_NAME)/description
> diff --git a/src/packages/fff/fff-sysupgrade/Makefile b/src/packages/fff/fff-
> sysupgrade/Makefile
> index 90ef66a..a0973fc 100644
> --- a/src/packages/fff/fff-sysupgrade/Makefile
> +++ b/src/packages/fff/fff-sysupgrade/Makefile
> @@ -13,7 +13,11 @@ define Package/$(PKG_NAME)
>  	CATEGORY:=Freifunk
>  	TITLE:=Freifunk-Franken sysupgrade
>  	URL:=http://www.freifunk-franken.de
> -	DEPENDS:=+fff-boardname +fff-config +fff-hoodutils
> + at BUSYBOX_CONFIG_WGET
> +	DEPENDS:=+fff-boardname \
> +		+fff-config \
> +		+fff-hoodutils \
> +		+ at BUSYBOX_CONFIG_WGET \
> +		+ at BUSYBOX_CONFIG_FEATURE_WGET_TIMEOUT

Ich glaube, fff-sysupgrade nutzt das timeout nicht. Daher habe ich hier nichts verändert. Ist aber eine philosophische Diskussion.

>  endef
> 
>  define Package/$(PKG_NAME)/description
> diff --git a/src/packages/fff/fff-vpn-select/Makefile b/src/packages/fff/fff-vpn-
> select/Makefile
> index 09dd7ea..7c66c57 100644
> --- a/src/packages/fff/fff-vpn-select/Makefile
> +++ b/src/packages/fff/fff-vpn-select/Makefile
> @@ -13,8 +13,7 @@ define Package/$(PKG_NAME)
>  	CATEGORY:=Freifunk
>  	TITLE:= Freifunk-Franken vpn-select
>  	URL:=http://www.freifunk-franken.de
> -	DEPENDS:=+ at BUSYBOX_CONFIG_FEATURE_WGET_TIMEOUT \
> -			 +fff-fastd
> +	DEPENDS:=+fff-fastd

Hier müsste meiner rebased werden.

Grüße

Adrian

>  endef
> 
>  define Package/$(PKG_NAME)/description
> diff --git a/src/packages/fff/fff-web/Makefile b/src/packages/fff/fff-
> web/Makefile
> index 819644a..ba46ba7 100644
> --- a/src/packages/fff/fff-web/Makefile
> +++ b/src/packages/fff/fff-web/Makefile
> @@ -12,7 +12,17 @@ define Package/$(PKG_NAME)
>  	CATEGORY:=Freifunk
>  	TITLE:= Freifunk-Franken Webinterface
>  	URL:=http://www.freifunk-franken.de
> -	DEPENDS:=+uhttpd +libustream-mbedtls +haserl +px5g +simple-tc +fff-
> boardname +fff-config +fff-network +fff-hoodutils
> +	DEPENDS:=+uhttpd \
> +		+libustream-mbedtls \
> +		+haserl \
> +		+px5g \
> +		+simple-tc \
> +		+fff-boardname \
> +		+fff-config \
> +		+fff-network \
> +		+fff-hoodutils \
> +		+ at BUSYBOX_CONFIG_WGET \
> +		+ at BUSYBOX_CONFIG_FEATURE_WGET_TIMEOUT
>  endef
> 
>  define Package/$(PKG_NAME)/description
> --
> 2.20.1
-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : nicht verfügbar
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/20190625/1f2d7cc5/attachment.sig>


Mehr Informationen über die Mailingliste franken-dev