From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: [PATCH] speedtest-cli: Fix for bug #12602 - CORE 155 error with
 speedtest-cli
Date: Mon, 12 Apr 2021 11:38:38 +0100
Message-ID: <2B915D1C-532E-40B3-90BA-BEEFE50309E9@ipfire.org>
In-Reply-To: <9289668f-3885-3881-9eec-708729c65ec3@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============1574410979548124292=="
List-Id: <development.lists.ipfire.org>

--===============1574410979548124292==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Hello,

The user who reported this did not really make a case for why the client shou=
ld be changed.

The Python client is working and still maintained, so I do not see any reason=
 to change this.

-Michael

> On 8 Apr 2021, at 22:30, Adolf Belka <adolf.belka(a)ipfire.org> wrote:
>=20
> Hi Jon,
>=20
> On 08/04/2021 23:10, Jon Murphy wrote:
>> I seem to remember a bug report for speedtest (besides #12602 below).  It =
had to do with this post:
>> https://community.ipfire.org/t/replace-speedtest-cli-with-speedtest-for-be=
tter-accuracy/3698 <https://community.ipfire.org/t/replace-speedtest-cli-with=
-speedtest-for-better-accuracy/3698>
>> I could not locate the bugzilla post.
> That is because the poster never raised it as a bug.
>> Can speedtest be updated for this issue?
> I provided links to the addon creation process and how to submit a patch to=
 the development list. He was going to build the patch for himself and then s=
ubmit it to the development list for review. He never came back with a patch =
submission.
>=20
> Regards,
> Adolf
>> Jon
>>> On Apr 8, 2021, at 3:37 PM, Adolf Belka <adolf.belka(a)ipfire.org <mailto=
:adolf.belka(a)ipfire.org>> wrote:
>>>=20
>>> - Update from 2.1.2 to 2.1.3 - fix for bug #12602
>>>   Updated addon installed and fix confirmed
>>> - Update rootfile
>>> - Changelog
>>>    Address issue where a test server may return an HTTP error during uplo=
ad or download
>>>    Address issue where ignore_ids may be empty or have empty values
>>>=20
>>> Signed-off-by: Adolf Belka <adolf.belka(a)ipfire.org <mailto:adolf.belka(=
a)ipfire.org>>
>>> ---
>>> config/rootfiles/packages/speedtest-cli | 13 +++++++------
>>> lfs/speedtest-cli                       |  6 +++---
>>> 2 files changed, 10 insertions(+), 9 deletions(-)
>>>=20
>>> diff --git a/config/rootfiles/packages/speedtest-cli b/config/rootfiles/p=
ackages/speedtest-cli
>>> index 0ef126e23..6632e235c 100644
>>> --- a/config/rootfiles/packages/speedtest-cli
>>> +++ b/config/rootfiles/packages/speedtest-cli
>>> @@ -1,9 +1,10 @@
>>> usr/bin/speedtest
>>> usr/bin/speedtest-cli
>>> usr/lib/python3.8/site-packages/speedtest.py
>>> -usr/lib/python3.8/site-packages/speedtest_cli-2.1.2-py3.8.egg-info
>>> -usr/lib/python3.8/site-packages/speedtest_cli-2.1.2-py3.8.egg-info/PKG-I=
NFO
>>> -usr/lib/python3.8/site-packages/speedtest_cli-2.1.2-py3.8.egg-info/SOURC=
ES.txt
>>> -usr/lib/python3.8/site-packages/speedtest_cli-2.1.2-py3.8.egg-info/depen=
dency_links.txt
>>> -usr/lib/python3.8/site-packages/speedtest_cli-2.1.2-py3.8.egg-info/entry=
_points.txt
>>> -usr/lib/python3.8/site-packages/speedtest_cli-2.1.2-py3.8.egg-info/top_l=
evel.txt
>>> +usr/lib/python3.8/site-packages/speedtest_cli-2.1.3-py3.8.egg-info
>>> +usr/lib/python3.8/site-packages/speedtest_cli-2.1.3-py3.8.egg-info/PKG-I=
NFO
>>> +usr/lib/python3.8/site-packages/speedtest_cli-2.1.3-py3.8.egg-info/SOURC=
ES.txt
>>> +usr/lib/python3.8/site-packages/speedtest_cli-2.1.3-py3.8.egg-info/depen=
dency_links.txt
>>> +usr/lib/python3.8/site-packages/speedtest_cli-2.1.3-py3.8.egg-info/entry=
_points.txt
>>> +usr/lib/python3.8/site-packages/speedtest_cli-2.1.3-py3.8.egg-info/top_l=
evel.txt
>>> +
>>> diff --git a/lfs/speedtest-cli b/lfs/speedtest-cli
>>> index 446844ab3..f6bcf4c96 100644
>>> --- a/lfs/speedtest-cli
>>> +++ b/lfs/speedtest-cli
>>> @@ -24,7 +24,7 @@
>>>=20
>>> include Config
>>>=20
>>> -VER        =3D 2.1.2
>>> +VER        =3D 2.1.3
>>>=20
>>> THISAPP    =3D speedtest-cli-$(VER)
>>> DL_FILE    =3D $(THISAPP).tar.gz
>>> @@ -32,7 +32,7 @@ DL_FROM    =3D $(URL_IPFIRE)
>>> DIR_APP    =3D $(DIR_SRC)/$(THISAPP)
>>> TARGET     =3D $(DIR_INFO)/$(THISAPP)
>>> PROG       =3D speedtest-cli
>>> -PAK_VER    =3D 3
>>> +PAK_VER    =3D 4
>>>=20
>>> DEPS       =3D
>>>=20
>>> @@ -44,7 +44,7 @@ objects =3D $(DL_FILE)
>>>=20
>>> $(DL_FILE) =3D $(DL_FROM)/$(DL_FILE)
>>>=20
>>> -$(DL_FILE)_MD5 =3D a49ad088331810fc9442e8a5c59de713
>>> +$(DL_FILE)_MD5 =3D 08c431f2f398880745c4f0564962b9e2
>>>=20
>>> install : $(TARGET)
>>>=20
>>> --=20
>>> 2.31.1
>>>=20


--===============1574410979548124292==--