Re: O: pylint, python-logilab-*, python-paramiko, spambayes (python package collisions)

2017-12-06 Thread Yaakov Selkowitz
On 2017-11-19 19:10, Yaakov Selkowitz wrote: > While we're at it, your two remaining Python module packages > (python-crypto and python-feedparser) are both due for updates and split > wheel builds for Python 2 and 3; the former also needs some patches from > Fedora for a CVE and Python 3

[PATCH setup v3 2/6] Use SitePage for cygwin.com mirrors only

2017-12-06 Thread Ken Brown
Keep separate lists of mirrors and user URLs. Rename several SiteList variables to indicate that they now contain only mirrors. Introduce a helper function remove_user_urls() to move non-mirrors from the initial list of sites to the list of user URLs. Set the initial state of allow_user_url

[PATCH setup v3 0/6] Distinguish between user URLs and cygwin mirrors in UI

2017-12-06 Thread Ken Brown
This is a followup to https://cygwin.com/ml/cygwin-apps/2011-04/msg00014.html, 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.

[PATCH setup v3 1/6] Use the IDD_SITE dialog for cygwin.com mirrors only

2017-12-06 Thread Ken Brown
Remove controls related to adding a user URL. Add a checkbox IDC_ALLOW_USER_URL; checking this sets a new boolean variable allow_user_url. --- res.rc | 14 ++ resource.h | 1 + site.cc| 39 --- 3 files changed, 11 insertions(+), 43

[PATCH setup v3 5/6] Keep the mirror list sorted properly

2017-12-06 Thread Ken Brown
When site.cc:load_site_list() added a mirror whose URL was already in the list, it put the new entry in the same position as the old. Remove the old entry and merge the new one instead. --- site.cc | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/site.cc b/site.cc index

[PATCH setup v3 6/6] Display area and location of mirrors, and add these to the sort key

2017-12-06 Thread Ken Brown
From: Brian Inglis Also change site_list_type::operator== to compare URLs rather than keys, since the key no longer depends only on the URL. --- site.cc | 12 +--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/site.cc b/site.cc index

[PATCH setup v3 3/6] Create new page UserSitePage for user URLs

2017-12-06 Thread Ken Brown
This is done in two new files, usersite.h and usersite.cc, based on site.h and site.cc. The new page is activated after the mirror selection page if IDC_ALLOW_USER_URL is checked. --- Makefile.am | 2 + main.cc | 4 + res.rc | 25 +++ resource.h | 2 + site.cc | 24

[PATCH setup v3 4/6] Display full URLs in the user site list

2017-12-06 Thread Ken Brown
--- site.cc | 6 ++ 1 file changed, 6 insertions(+) diff --git a/site.cc b/site.cc index accc264..98dd756 100644 --- a/site.cc +++ b/site.cc @@ -187,6 +187,12 @@ site_list_type::site_list_type (const string &_url, idx = 0; } while (idx > 0); key += url; + /* Display the full

Re: [PATCH setup v3 0/6] Distinguish between user URLs and cygwin mirrors in UI

2017-12-06 Thread Brian Inglis
On 2017-12-06 13:45, Ken Brown wrote: > This is a followup to > https://cygwin.com/ml/cygwin-apps/2011-04/msg00014.html, > 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

Re: [PATCH setup v3 0/6] Distinguish between user URLs and cygwin mirrors in UI

2017-12-06 Thread Ken Brown
On 12/6/2017 4:07 PM, Brian Inglis wrote: On 2017-12-06 13:45, Ken Brown wrote: This is a followup to https://cygwin.com/ml/cygwin-apps/2011-04/msg00014.html, in which Jon suggested splitting site selection into two pages, one for cygwin.com mirrors and one for additional user URLs. The

Re: [PATCH setup v3 0/6] Distinguish between user URLs and cygwin mirrors in UI

2017-12-06 Thread Brian Inglis
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 >>>    https://cygwin.com/ml/cygwin-apps/2011-04/msg00014.html, >>> in which Jon suggested splitting site selection into two pages, one >>> for