This is the mail archive of the
cygwin-apps
mailing list for the Cygwin project.
Re: [PATCH setup 3/3] Add the last element of URL path to site chooser, if interesting.
- From: Jon TURNEY <jon dot turney at dronecode dot org dot uk>
- To: cygwin-apps at cygwin dot com
- Date: Tue, 22 Apr 2014 18:50:24 +0100
- Subject: Re: [PATCH setup 3/3] Add the last element of URL path to site chooser, if interesting.
- Authentication-results: sourceware.org; auth=none
- References: <1398078987-5116-1-git-send-email-jon dot turney at dronecode dot org dot uk> <1398078987-5116-4-git-send-email-jon dot turney at dronecode dot org dot uk> <20140421141028 dot GD661 at ednor dot casa dot cgf dot cx> <53555F7C dot 7070708 at dronecode dot org dot uk> <20140422075315 dot GB5959 at calimero dot vinschen dot de> <20140422144744 dot GA7528 at ednor dot casa dot cgf dot cx>
On 22/04/2014 15:47, Christopher Faylor wrote:
> On Tue, Apr 22, 2014 at 09:53:15AM +0200, Corinna Vinschen wrote:
>> On Apr 21 19:12, Jon TURNEY wrote:
>>> On 21/04/2014 15:10, Christopher Faylor wrote:
>>>> On Mon, Apr 21, 2014 at 12:16:27PM +0100, Jon TURNEY wrote:
>>>>> If I add the site http://mirrors.kernel.org/sources.redhat.com/cygwinports/ to
>>>>> setup's mirror list, using the GUI or --site option, I get two indistinguishable
>>>>> entries named http://mirrors.kernel.org in the mirror list box.
>>>>>
>>>>> So, to make the site chooser list entries more 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)
>>>>>
>>>>> As Corinna pointed out in Nov 2010, there is still a corner case of URLs which
>>>>> share protocol, hostname and the last element of the URL path being
>>>>> indistinguishable. Additionally, it will need updating for any new mirrors which
>>>>> don't use one of the expected strings for the last path element in the URL.
>>>>
>>>> I'd actually be ok with just displaying the whole URL even if it ends with Cygwin.
>>>> Is there a reason not to do that?
>>>
>>> Last time this was discussed [1], I think you felt that the current site list
>>> and dialog box is too small to display the full URL usefully.
>>>
>>> [1] http://cygwin.com/ml/cygwin-apps/2010-11/msg00134.html
>>
>> The dialog has still a lot of space to the left and right of the URL
>> list window. We could resize it to take the full available space even
>> in the smallest dialog size. I think that would be sufficient most
>> of the time. And the dialog can be resized if it doesn't suffice.
>
> I agree. I was going to suggest resizing (maybe I did before too).
> We'd like to get a new version of setup.exe released soon. I think
> these changes would be nice to have in it. Any chance you could do
> that Jon?
Sure, I'll take another look when I can.
per IRC [1/3] and [2/3] applied, [3/3] withdrawn.