This is the mail archive of the
cygwin-apps
mailing list for the Cygwin project.
Re: [PATCH 3/3] Add the last element of URL path to site chooser, if interesting.
On Sat, Nov 06, 2010 at 05:05:15PM +0000, Jon TURNEY wrote:
>Currently, if, for example, I manually add the site http://mirrors.kernel.org/sources.redhat.com/cygwinports/
>to setup's mirror list, I get two indistinguishable entries named http://mirrors.kernel.org.
>
>Furthermore, because the code to ensure the site just added is selected uses the string inside the list control
>to locate elements, we end up with a random one of those two indistinguishable entries selected (usually the
>previously existing one). This problem also prevents the selected sites being correctly saved and restored
>for the next setup run.
>
>So, to make the site chooser list entries unique and distinguishable, add the last element of the URL path to
>the site chooser, if it exists and isn't 'cygwin' (or some other alternatives used by current mirrors)
>
>(Also fix the logic for identifying protocol and site name part of the URL to find the first '/' after a '//',
>rather than the first '/' after a '.', to handle sitenames which aren't FQDNs correctly)
This is great, please check in. I suspect that you're just trying a
local mirror or somethin but if there are actually sites in the mirror
list which are not proper FQDNs, please report them so that they can be
fixed in the mirror database.
cgf