This is the mail archive of the
mailing list for the Cygwin project.
Re: [PATCH setup v3 0/6] Distinguish between user URLs and cygwin mirrors in UI
On 12/7/2017 2:08 PM, Jon Turney wrote:
On 07/12/2017 15:07, Ken Brown wrote:
On 12/7/2017 8:55 AM, Jon Turney wrote:
On 06/12/2017 22:15, Brian Inglis wrote:
On 2017-12-06 14:55, Ken Brown wrote:
On 12/6/2017 4:07 PM, Brian Inglis wrote:
On 2017-12-06 13:45, Ken Brown wrote:
This is a followup to
in which Jon suggested splitting site selection into two pages, one
for cygwin.com mirrors and one for additional user URLs. The latter
would be visible only if the user had previously checked a suitable
checkbox. This patch series implements that suggestion.
Thanks very much for looking at this.
This looks pretty good.
My main concern is how this deals with private cygwin mirrors. I
guess you need to not select anything at the "choose a mirror page",
then tick "allow sites that are not cygwin.com mirrors" to get to the
page that lets you enter the URL, which is somewhat confusing.
I'm going to guess that private cygwin mirrors are used more than
repos of additional packages.
I hadn't thought about that. I'll try to think of different
terminology to make it clear that the first page is for mirrors listed
on https://cygwin.com/mirrors.html, and the second page is for private
mirrors and other repos of packages. I'll try to send v4 with these
Hmm... I don't think that's perhaps the best direction to be heading in.
changes later today and you can see if it looks better.
(Another facet to consider: At the moment, I think we do something
like trying all the keys we know to validate the signature of every
setup.ini we download. It would be nice to maintain an idea of which
URLs are supposed to be validated using the cygwin signing key.)
After my patches, I think 'all_mirror_list' is precisely the list of
such URLs. Maybe I should rename it to 'official_mirror_list' or
No, because private cygwin mirrors (by which I just mean a mirror whose
URL isn't published in mirrors.lst) can't be in that list.
OK, I was missing the point. You want cygwin mirrors, whether official
or not, to be treated on the mirror selection page. The second page
should be for package repos that don't claim to be mirrors. Is that
right? It makes a lot more sense than what I was trying to do.
I guess we could achieve this by keeping the 'Add URL' control on the
mirrors page, but then the migration becomes a big problem (as we
don't know if a URL is a mirror or not until we read it)
How about this: We put the 'Add URL' control back on the mirrors page,
but we make it clear that it's intended for cygwin mirrors. When we
later check signatures, we check such mirrors only with the cygwin
signing key. In case the user has turned off signature checking, we
could follow Achim's suggestion of checking for "release: cygwin".
We also stop saving non-mirrors under "last-mirror" in setup.rc. Maybe
we need a new "last-user-site" setting for these.
This whole thing of "remember the mirror list from last time so we can
see if this site came from it", rather than explicitly remembering that
for the site is a bit odd, though.