All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alexey Brodkin <Alexey.Brodkin@synopsys.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] Config.in: Use CDN kernel.org mirror
Date: Mon, 23 May 2016 17:21:24 +0000	[thread overview]
Message-ID: <1464024053.7749.21.camel@synopsys.com> (raw)
In-Reply-To: <20160523191858.6b3e878c@free-electrons.com>

Hi Thomas,

On Mon, 2016-05-23 at 19:18 +0200, Thomas Petazzoni wrote:
> Hello,
> 
> On Mon, 23 May 2016 16:04:05 +0000, Alexey Brodkin wrote:
> 
> > 
> > > 
> > > > 
> > > > ?	?????http://www.XX.kernel.org/pub (XX = country code)
> > > > ?	?????http://mirror.aarnet.edu.au/pub/ftp.kernel.org??
> > > Should this list of mirrors be removed???
> > Well I have no preference here.
> > I mean user still may have a preferred mirror so why would we
> > limit him or her here?
> Well, if we're pointing to a CDN then really there is no longer any
> reason to point users to www.XXX.kernel.org with XX = country code,
> since that's what a CDN is doing automatically.

Indeed but given our previous discussion it makes sense to leave those
help strings for time being.

> > 
> > > 
> > > > 
> > > > +	??Now when kernel.org is mirrored by CDN as well we'll be using
> > > > +	??it by default to speed-up downloads all over the world.??
> > > I'm sorry, but I don't understand this sentence.??
> > If I'm not mistaken CDN allows to use the best server for each user
> > (based on some criteria). Now kerenl.org is mirrored with use of that
> > technology. And in theory users of Buildroot all over the world will
> > benefit from much faster downloads from this CDN mirror (which will
> > be in real life different physical servers for different users).
> Yes, of course. But I still think the wording is wrong. The wording
> feels like something for the commit log, not for the help text. A
> correct wording for the help text could be:
> 
> 	The default value points to kernel.org's CDN, which will
> 	automatically redirect you to a mirror geographically close to
> 	you.

Do you want me to resend the patch or you'll do this fixup before
patch application?

-Alexey

  reply	other threads:[~2016-05-23 17:21 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-05-23 15:36 [Buildroot] [PATCH] Config.in: Use CDN kernel.org mirror Alexey Brodkin
2016-05-23 15:46 ` Thomas Petazzoni
2016-05-23 15:57   ` Peter Korsgaard
2016-05-23 16:04     ` Thomas Petazzoni
2016-05-23 16:11       ` Alexey Brodkin
2016-05-23 19:38       ` Peter Korsgaard
2016-05-23 16:04   ` Alexey Brodkin
2016-05-23 17:18     ` Thomas Petazzoni
2016-05-23 17:21       ` Alexey Brodkin [this message]
2016-05-24 18:44 ` Thomas Petazzoni

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1464024053.7749.21.camel@synopsys.com \
    --to=alexey.brodkin@synopsys.com \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.