Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix sage-download-file --print-fastest-mirror #18457

Closed
jhpalmieri opened this issue May 19, 2015 · 7 comments
Closed

Fix sage-download-file --print-fastest-mirror #18457

jhpalmieri opened this issue May 19, 2015 · 7 comments

Comments

@jhpalmieri
Copy link
Member

Trac #15642 broke the command sage-download-file --print-fastest-mirror: that command returns

Downloading the Sage mirror list
Searching fastest mirror
  162ms: http://echidna.maths.usyd.edu.au/sage/
  293ms: http://ftp.leg.uct.ac.za/pub/packages/sage/
  137ms: http://ftp.riken.jp/sagemath/
  150ms: http://ftp.tsukuba.wide.ad.jp/software/sage/
  142ms: http://ftp.yz.yamagata-u.ac.jp/pub/math/sage/
  199ms: http://linorg.usp.br/sage/
  162ms: http://mirror.aarnet.edu.au/pub/sage/
  175ms: http://mirror.switch.ch/mirror/sagemath/
  370ms: http://mirror.ufs.ac.za/sagemath/
  210ms: http://mirror.yandex.ru/mirrors/sage.math.washington.edu/
  193ms: http://mirrors.fe.up.pt/pub/sage/
   74ms: http://mirrors.mit.edu/sage/
  216ms: http://mirrors.ustc.edu.cn/sagemath/
  205ms: http://sagemath.c3sl.ufpr.br/
  673ms: http://sagemath.mirror.ac.za/
  370ms: http://sagemath.polytechnic.edu.na/
  171ms: http://www-ftp.lip6.fr/pub/math/sagemath/
    8ms: http://www.cecm.sfu.ca/sage/
  171ms: http://www.mirrorservice.org/sites/www.sagemath.org/
Fastest mirror: http://www.cecm.sfu.ca/sage/
http://www.cecm.sfu.ca/sage/

rather than just the URL for a single server (i.e., the last line in this output).

Component: packages: optional

Author: John Palmieri

Branch/Commit: cff0472

Reviewer: Volker Braun

Issue created by migration from https://trac.sagemath.org/ticket/18457

@jhpalmieri
Copy link
Member Author

Branch: u/jhpalmieri/fastest-mirror

@jhpalmieri
Copy link
Member Author

New commits:

cff0472#18457: fix sage-download-file --print-fastest-mirror

@jhpalmieri
Copy link
Member Author

Commit: cff0472

@vbraun
Copy link
Member

vbraun commented May 19, 2015

Reviewer: Volker Braun

@vbraun
Copy link
Member

vbraun commented May 19, 2015

comment:3

I see, fails if the mirror list hasn't been downloaded yet. Passing verbose=bool flags around is kind of fugly but we can refactor it later

@jhpalmieri
Copy link
Member Author

comment:4

Replying to @vbraun:

I see, fails if the mirror list hasn't been downloaded yet.

Right.

Passing verbose=bool flags around is kind of fugly but we can refactor it later

Yes, I thought briefly about some other choices, like redirecting output temporarily, but this was easy to do, and it's clear what's going on.

@vbraun
Copy link
Member

vbraun commented May 20, 2015

Changed branch from u/jhpalmieri/fastest-mirror to cff0472

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants