diff mbox series

[2/2] oeqa/sdk/cases/autotools.py: use gnu mirror instead of main server

Message ID 20251009023123.245604-2-steve@sakoman.com
State Accepted, archived
Commit f3046716c9c565e6f65d5169694cd3da126cfaeb
Headers show
Series [1/2] oeqa/selftest/cases/meta_ide.py: use gnu mirror instead of main server | expand

Commit Message

Steve Sakoman Oct. 9, 2025, 2:31 a.m. UTC
ftp.gnu.org is the main server of the GNU project, however download speed
can vary greatly based on one's location.

Using ftpmirror.gnu.org should redirect the request to the closest up-to-date mirror,
which should result sometimes in significantly faster download speed, depending
on one's location. This should also distribute the traffic more across the mirrors.

This information was sourced from https://www.gnu.org/prep/ftp.html

Signed-off-by: Steve Sakoman <steve@sakoman.com>
---
 meta/lib/oeqa/sdk/cases/autotools.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Richard Purdie Oct. 9, 2025, 8:43 a.m. UTC | #1
On Wed, 2025-10-08 at 19:31 -0700, Steve Sakoman via lists.openembedded.org wrote:
> ftp.gnu.org is the main server of the GNU project, however download speed
> can vary greatly based on one's location.
> 
> Using ftpmirror.gnu.org should redirect the request to the closest up-to-date mirror,
> which should result sometimes in significantly faster download speed, depending
> on one's location. This should also distribute the traffic more across the mirrors.
> 
> This information was sourced from https://www.gnu.org/prep/ftp.html
> 
> Signed-off-by: Steve Sakoman <steve@sakoman.com>
> ---
>  meta/lib/oeqa/sdk/cases/autotools.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/meta/lib/oeqa/sdk/cases/autotools.py b/meta/lib/oeqa/sdk/cases/autotools.py
> index 3f51854e3d..ecafafa7d6 100644
> --- a/meta/lib/oeqa/sdk/cases/autotools.py
> +++ b/meta/lib/oeqa/sdk/cases/autotools.py
> @@ -27,7 +27,7 @@ class AutotoolsTest(OESDKTestCase):
>          pmv = parallel_make_value((self.td.get('PARALLEL_MAKE') or '').split())
>  
>          with tempfile.TemporaryDirectory(prefix="cpio-", dir=self.tc.sdk_dir) as testdir:
> -            tarball = self.fetch(testdir, self.td["DL_DIR"], "https://ftp.gnu.org/gnu/cpio/cpio-2.15.tar.gz")
> +            tarball = self.fetch(testdir, self.td["DL_DIR"], "https://ftpmirror.gnu.org/gnu/cpio/cpio-2.15.tar.gz")
>  
>              opts = {}
>              opts["source"] = os.path.join(testdir, "cpio-2.15")
> 

This patch is good, thanks.

This code is supposed to be using a cached copy from DL_DIR. If it is
failing on older releases on the autobuilder, it might mean the version
of the tarball is missing from the mirror for that release. Might be
worth checking...

Cheers,

Richard
diff mbox series

Patch

diff --git a/meta/lib/oeqa/sdk/cases/autotools.py b/meta/lib/oeqa/sdk/cases/autotools.py
index 3f51854e3d..ecafafa7d6 100644
--- a/meta/lib/oeqa/sdk/cases/autotools.py
+++ b/meta/lib/oeqa/sdk/cases/autotools.py
@@ -27,7 +27,7 @@  class AutotoolsTest(OESDKTestCase):
         pmv = parallel_make_value((self.td.get('PARALLEL_MAKE') or '').split())
 
         with tempfile.TemporaryDirectory(prefix="cpio-", dir=self.tc.sdk_dir) as testdir:
-            tarball = self.fetch(testdir, self.td["DL_DIR"], "https://ftp.gnu.org/gnu/cpio/cpio-2.15.tar.gz")
+            tarball = self.fetch(testdir, self.td["DL_DIR"], "https://ftpmirror.gnu.org/gnu/cpio/cpio-2.15.tar.gz")
 
             opts = {}
             opts["source"] = os.path.join(testdir, "cpio-2.15")