[v2] tests/avocado: Update yamon-bin-02.22.zip URL

[PATCH-for-8.2 v2] tests/avocado: Update yamon-bin-02.22.zip URL

Posted by Philippe Mathieu-Daudé 2 years, 2 months ago

http://www.imgtec.com/tools/mips-tools/downloads/ redirects
to https://mips.com/downloads/yamon-version-02-22/ then points
to an invalid path to a s3 bucket. Use the correct path. The
site will eventually be fixed.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
Supersedes: <20231201142139.39816-1-philmd@linaro.org>
Cc: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Cc: BALATON Zoltan <balaton@eik.bme.hu>
---
 tests/avocado/machine_mips_malta.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/tests/avocado/machine_mips_malta.py b/tests/avocado/machine_mips_malta.py
index 9bd54518bf..8e1ee120f4 100644
--- a/tests/avocado/machine_mips_malta.py
+++ b/tests/avocado/machine_mips_malta.py
@@ -124,8 +124,9 @@ def test_mips_malta_i6400_framebuffer_logo_8cores(self):
 class MaltaMachine(QemuSystemTest):
 
     def do_test_yamon(self):
-        rom_url = ('http://www.imgtec.com/tools/mips-tools/downloads/'
-                   'yamon/yamon-bin-02.22.zip')
+        rom_url = ('https://s3-eu-west-1.amazonaws.com/'
+                   'downloads-mips/mips-downloads/'
+                   'YAMON/yamon-bin-02.22.zip')
         rom_hash = '8da7ecddbc5312704b8b324341ee238189bde480'
         zip_path = self.fetch_asset(rom_url, asset_hash=rom_hash)
 
-- 
2.41.0

Re: [PATCH-for-8.2 v2] tests/avocado: Update yamon-bin-02.22.zip URL

Posted by Stefan Hajnoczi 2 years, 2 months ago

On Fri, 1 Dec 2023 at 15:56, Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
>
> http://www.imgtec.com/tools/mips-tools/downloads/ redirects
> to https://mips.com/downloads/yamon-version-02-22/ then points
> to an invalid path to a s3 bucket. Use the correct path. The
> site will eventually be fixed.
>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
> Supersedes: <20231201142139.39816-1-philmd@linaro.org>
> Cc: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> Cc: BALATON Zoltan <balaton@eik.bme.hu>
> ---
>  tests/avocado/machine_mips_malta.py | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)

Applied, thanks!

Stefan

>
> diff --git a/tests/avocado/machine_mips_malta.py b/tests/avocado/machine_mips_malta.py
> index 9bd54518bf..8e1ee120f4 100644
> --- a/tests/avocado/machine_mips_malta.py
> +++ b/tests/avocado/machine_mips_malta.py
> @@ -124,8 +124,9 @@ def test_mips_malta_i6400_framebuffer_logo_8cores(self):
>  class MaltaMachine(QemuSystemTest):
>
>      def do_test_yamon(self):
> -        rom_url = ('http://www.imgtec.com/tools/mips-tools/downloads/'
> -                   'yamon/yamon-bin-02.22.zip')
> +        rom_url = ('https://s3-eu-west-1.amazonaws.com/'
> +                   'downloads-mips/mips-downloads/'
> +                   'YAMON/yamon-bin-02.22.zip')
>          rom_hash = '8da7ecddbc5312704b8b324341ee238189bde480'
>          zip_path = self.fetch_asset(rom_url, asset_hash=rom_hash)
>
> --
> 2.41.0
>

Re: [PATCH-for-8.2 v2] tests/avocado: Update yamon-bin-02.22.zip URL

Posted by Thomas Huth 2 years, 2 months ago

On 01/12/2023 21.56, Philippe Mathieu-Daudé wrote:
> http://www.imgtec.com/tools/mips-tools/downloads/ redirects
> to https://mips.com/downloads/yamon-version-02-22/ then points
> to an invalid path to a s3 bucket. Use the correct path. The
> site will eventually be fixed.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
> Supersedes: <20231201142139.39816-1-philmd@linaro.org>
> Cc: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
> Cc: BALATON Zoltan <balaton@eik.bme.hu>
> ---
>   tests/avocado/machine_mips_malta.py | 5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/avocado/machine_mips_malta.py b/tests/avocado/machine_mips_malta.py
> index 9bd54518bf..8e1ee120f4 100644
> --- a/tests/avocado/machine_mips_malta.py
> +++ b/tests/avocado/machine_mips_malta.py
> @@ -124,8 +124,9 @@ def test_mips_malta_i6400_framebuffer_logo_8cores(self):
>   class MaltaMachine(QemuSystemTest):
>   
>       def do_test_yamon(self):
> -        rom_url = ('http://www.imgtec.com/tools/mips-tools/downloads/'
> -                   'yamon/yamon-bin-02.22.zip')
> +        rom_url = ('https://s3-eu-west-1.amazonaws.com/'
> +                   'downloads-mips/mips-downloads/'
> +                   'YAMON/yamon-bin-02.22.zip')
>           rom_hash = '8da7ecddbc5312704b8b324341ee238189bde480'
>           zip_path = self.fetch_asset(rom_url, asset_hash=rom_hash)
>   

Reviewed-by: Thomas Huth <thuth@redhat.com>