Re: speedtest-cli - ValueError: invalid literal for int() with base 10

2021-04-09 Thread Kevin Lo
On Fri, Apr 09, 2021 at 08:57:42AM +0200, Theo Buehler wrote:
> 
> On Fri, Apr 09, 2021 at 06:45:40AM +, Mikolaj Kucharski wrote:
> > Hi,
> > 
> > This should go upsteam, but I just noticed this on multiple -current
> > machines. I see that v2.1.3 was released yesterday and at first glance
> > last commit in repo looks like could fix below issue. Sorry, didn't dive
> > deeper and I'm not sure will I be able until next weekend :/
> 
> Indeed, this seems to work for me:

Works for me, thanks.
ok kevlo@

> Index: Makefile
> ===
> RCS file: /cvs/ports/benchmarks/speedtest-cli/Makefile,v
> retrieving revision 1.23
> diff -u -p -r1.23 Makefile
> --- Makefile  23 Feb 2021 19:39:10 -  1.23
> +++ Makefile  9 Apr 2021 06:54:59 -
> @@ -2,10 +2,9 @@
>  
>  COMMENT =command-line client for speedtest.net bandwidth tester
>  
> -MODPY_EGG_VERSION =  2.1.2
> +MODPY_EGG_VERSION =  2.1.3
>  DISTNAME =   speedtest-cli-${MODPY_EGG_VERSION}
>  CATEGORIES = benchmarks net
> -REVISION =   0
>  
>  HOMEPAGE =   https://github.com/sivel/speedtest-cli
>  
> Index: distinfo
> ===
> RCS file: /cvs/ports/benchmarks/speedtest-cli/distinfo,v
> retrieving revision 1.16
> diff -u -p -r1.16 distinfo
> --- distinfo  23 Aug 2019 15:49:58 -  1.16
> +++ distinfo  9 Apr 2021 06:55:05 -
> @@ -1,2 +1,2 @@
> -SHA256 (speedtest-cli-2.1.2.tar.gz) = 
> zx04YiL5TDJOMSW6mg0YfkbUoT3KCMAjvbmiMJa+LlQ=
> -SIZE (speedtest-cli-2.1.2.tar.gz) = 24365
> +SHA256 (speedtest-cli-2.1.3.tar.gz) = 
> XidzIzzttfo9gSDrf5e8xJdLUiGyVNM/8W4vHUE9kPA=
> +SIZE (speedtest-cli-2.1.3.tar.gz) = 24721
> 



Re: speedtest-cli - ValueError: invalid literal for int() with base 10

2021-04-09 Thread Theo Buehler
On Fri, Apr 09, 2021 at 06:45:40AM +, Mikolaj Kucharski wrote:
> Hi,
> 
> This should go upsteam, but I just noticed this on multiple -current
> machines. I see that v2.1.3 was released yesterday and at first glance
> last commit in repo looks like could fix below issue. Sorry, didn't dive
> deeper and I'm not sure will I be able until next weekend :/

Indeed, this seems to work for me:

Index: Makefile
===
RCS file: /cvs/ports/benchmarks/speedtest-cli/Makefile,v
retrieving revision 1.23
diff -u -p -r1.23 Makefile
--- Makefile23 Feb 2021 19:39:10 -  1.23
+++ Makefile9 Apr 2021 06:54:59 -
@@ -2,10 +2,9 @@
 
 COMMENT =  command-line client for speedtest.net bandwidth tester
 
-MODPY_EGG_VERSION =2.1.2
+MODPY_EGG_VERSION =2.1.3
 DISTNAME = speedtest-cli-${MODPY_EGG_VERSION}
 CATEGORIES =   benchmarks net
-REVISION = 0
 
 HOMEPAGE = https://github.com/sivel/speedtest-cli
 
Index: distinfo
===
RCS file: /cvs/ports/benchmarks/speedtest-cli/distinfo,v
retrieving revision 1.16
diff -u -p -r1.16 distinfo
--- distinfo23 Aug 2019 15:49:58 -  1.16
+++ distinfo9 Apr 2021 06:55:05 -
@@ -1,2 +1,2 @@
-SHA256 (speedtest-cli-2.1.2.tar.gz) = 
zx04YiL5TDJOMSW6mg0YfkbUoT3KCMAjvbmiMJa+LlQ=
-SIZE (speedtest-cli-2.1.2.tar.gz) = 24365
+SHA256 (speedtest-cli-2.1.3.tar.gz) = 
XidzIzzttfo9gSDrf5e8xJdLUiGyVNM/8W4vHUE9kPA=
+SIZE (speedtest-cli-2.1.3.tar.gz) = 24721



speedtest-cli - ValueError: invalid literal for int() with base 10

2021-04-09 Thread Mikolaj Kucharski
Hi,

This should go upsteam, but I just noticed this on multiple -current
machines. I see that v2.1.3 was released yesterday and at first glance
last commit in repo looks like could fix below issue. Sorry, didn't dive
deeper and I'm not sure will I be able until next weekend :/

$ speedtest-cli
Retrieving speedtest.net configuration...
Traceback (most recent call last):
  File "/usr/local/bin/speedtest-cli", line 11, in 
load_entry_point('speedtest-cli==2.1.2', 'console_scripts', 
'speedtest-cli')()
  File "/usr/local/lib/python3.8/site-packages/speedtest.py", line 1986, in main
shell()
  File "/usr/local/lib/python3.8/site-packages/speedtest.py", line 1872, in 
shell
speedtest = Speedtest(
  File "/usr/local/lib/python3.8/site-packages/speedtest.py", line 1091, in 
__init__
self.get_config()
  File "/usr/local/lib/python3.8/site-packages/speedtest.py", line 1173, in 
get_config
ignore_servers = list(
ValueError: invalid literal for int() with base 10: ''

-- 
Regards,
 Mikolaj