[PATCH] wget: Remove -t from calls

[mail at adrianschmutzler.de]

Hallo,

bei mir steht es da. Mein OpenWrt ist aber ein bisschen neuer.

root at FF-BTL-Bodytalk-Fichtelberg:~# wget
BusyBox v1.28.3 () multi-call binary.

Usage: wget [-cq] [-O FILE] [-Y on/off] [-P DIR] [-S] [-U AGENT] [-T SEC] URL...

Retrieve files via HTTP or FTP

        -c              Continue retrieval of aborted transfer
        -q              Quiet
        -P DIR          Save to DIR (default .)
        -S              Show server response
        -T SEC          Network read timeout is SEC seconds
        -O FILE         Save to FILE ('-' for stdout)
        -U STR          Use STR for User-Agent header
        -Y on/off       Use proxy

Interessanterweise sogar die gleiche Busybox-Version, also war das wohl ein Bug.

Grüße

Adrian

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf
> Of Christian Dresel
> Sent: Sonntag, 27. Januar 2019 15:52
> To: Adrian Schmutzler <freifunk at adrianschmutzler.de>; franken-
> dev at freifunk.net
> Subject: Re: [PATCH] wget: Remove -t from calls
> 
> Hallo Adrian
> 
> ich hab gerade nur mal schnell auf der GW Firmware geguckt (nicht
> getestet) und da scheint es auch kein -T zu geben:
> 
> root at fff-gw-cdhome:~# wget --help
> BusyBox v1.28.3 () multi-call binary.
> 
> Usage: wget [-cq] [-O FILE] [-Y on/off] [-P DIR] [-S] [-U AGENT] URL...
> 
> Retrieve files via HTTP or FTP
> 
>     -c        Continue retrieval of aborted transfer
>     -q        Quiet
>     -P DIR        Save to DIR (default .)
>     -S            Show server response
>     -O FILE        Save to FILE ('-' for stdout)
>     -U STR        Use STR for User-Agent header
>     -Y on/off    Use proxy
> 
> Wenn du das bestätigen kannst und wir eh aufräumen sollten wir den
> Parameter vllt. gleich mit verwerfen auch wenn er (aktuell noch) keine
> Probleme macht?
> 
> Gruß
> 
> Christian
> 
> On 27.01.19 15:47, Adrian Schmutzler wrote:
> > The wget from busybox we are using ignores the -t option, although it
> > does not throw errors in the currently used version.
> >
> > However, specifying the -t option in calls where it is not used is
> > misleading, so it should be removed.
> >
> > Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> > ---
> >  .../fff/fff-hoods/files/usr/lib/functions/fff/hoodfile         | 10 +++++-----
> >  1 file changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git
> > a/src/packages/fff/fff-hoods/files/usr/lib/functions/fff/hoodfile
> > b/src/packages/fff/fff-hoods/files/usr/lib/functions/fff/hoodfile
> > index 89af84c8..d4a9ecb2 100644
> > --- a/src/packages/fff/fff-hoods/files/usr/lib/functions/fff/hoodfile
> > +++ b/src/packages/fff/fff-hoods/files/usr/lib/functions/fff/hoodfile
> > @@ -44,9 +44,9 @@ getWirelessHoodfile() {
> >  	# wait a moment to start the interface
> >  	sleep 10;
> >
> > -	if wget -T15 -t5 -O "$file"
> "http://[fe80::1%w2sta]:2342/keyxchangev2data"; then
> > +	if wget -T15 -O "$file"
> > +"http://[fe80::1%w2sta]:2342/keyxchangev2data"; then
> >  		return 0
> > -	elif wget -T15 -t5 -O "$file"
> "http://[fe80::1%w5sta]:2342/keyxchangev2data"; then
> > +	elif wget -T15 -O "$file"
> > +"http://[fe80::1%w5sta]:2342/keyxchangev2data"; then
> >  		return 0
> >  	else
> >  		return 1
> > @@ -70,7 +70,7 @@ getEthernetHoodfile() {
> >  	neighbor_addrs=$(ping6 -c2 ff02::1%"${ethmesh_dev}" | grep seq |
> > grep DUP | cut -d " " -f4 | sed s/:$//g | sort -u)
> >
> >  	for addr in $neighbor_addrs; do
> > -		wget -T2 -t1 -qO "$file"
> "http://[${addr}%${ethmesh_dev}]:2342/keyxchangev2data" && return 0
> > +		wget -T2 -qO "$file"
> > +"http://[${addr}%${ethmesh_dev}]:2342/keyxchangev2data" && return 0
> >  	done
> >
> >  	# got no hoodfile
> > @@ -86,7 +86,7 @@ getGatewayHoodfile() {
> >
> >  	echo "Trying to get hoodfile from gateway..."
> >
> > -	if wget -T15 -t5 -O "$file" "http://[fe80::1%br-
> mesh]:2342/keyxchangev2data"; then
> > +	if wget -T15 -O "$file"
> > +"http://[fe80::1%br-mesh]:2342/keyxchangev2data"; then
> >  		return 0
> >  	else
> >  		return 1
> > @@ -108,7 +108,7 @@ getKeyserverHoodfile() {
> >
> >  	echo "Getting hoodfile from Keyserver"
> >
> > -	if wget -T15 -t5 -O "$file" "http://keyserver.freifunk-
> franken.de/v2/?lat=$lat&long=$long"; then
> > +	if wget -T15 -O "$file"
> > +"http://keyserver.freifunk-franken.de/v2/?lat=$lat&long=$long"; then
> >  		return 0
> >  	else
> >  		return 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         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20190127/abf205ff/attachment.sig>