From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 42154 invoked by alias); 24 Mar 2019 18:26:17 -0000 Mailing-List: contact cygwin-apps-cvs-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: cygwin-apps-cvs-owner@sourceware.org Received: (qmail 42087 invoked by uid 9795); 24 Mar 2019 18:26:17 -0000 Date: Sun, 24 Mar 2019 18:26:00 -0000 Message-ID: <20190324182617.42075.qmail@sourceware.org> From: jturney@sourceware.org To: cygwin-apps-cvs@sourceware.org Subject: [setup - the official Cygwin setup program] branch master, updated. release_2.896 X-Git-Refname: refs/heads/master X-Git-Reftype: branch X-Git-Oldrev: 3969f3ea7bf6c9de04d6cec6e1ae2f1dd6b7e99d X-Git-Newrev: 76dc99c1a15cc2f612432fde71170cb4b010ad2e X-SW-Source: 2019-q1/txt/msg00003.txt.bz2 https://sourceware.org/git/gitweb.cgi?p=cygwin-apps/setup.git;h=76dc99c1a15cc2f612432fde71170cb4b010ad2e commit 76dc99c1a15cc2f612432fde71170cb4b010ad2e Author: Jon Turney Date: Wed Mar 20 19:56:44 2019 +0000 Add double-click for a 'default action' to ListView Add support for a double-click in the ListView to invoke a 'default action'. Because we receive both NM_CLICK and then NM_DBCLK, reduce the area of a pop-up column which is sensitive to a click to the drop-down button (which opens a focus-stealing pop-up menu), so the rest of the area can receive a click (which does nothing) or a double-click (which does the default action). For a package, this default action is to toggle between the currently installed version (or uninstalled, if not installed), and the highest non-test version. There is no keyboard acceleration for this action currently. https://sourceware.org/git/gitweb.cgi?p=cygwin-apps/setup.git;h=b5f54d354d4270eec0b703950ce3288ca63ac7eb commit b5f54d354d4270eec0b703950ce3288ca63ac7eb Author: Jon Turney Date: Wed Mar 20 18:08:23 2019 +0000 Don't propagate actions down category tree into obsolete categories Choosing the 'Install' action on the 'All' category shouldn't propagate down into the '_obsolete' category, because that will just result in dependency conflicts due to trying to install both obsolete packages and their replacements. https://sourceware.org/git/gitweb.cgi?p=cygwin-apps/setup.git;h=df05f2348600a72ebefdc16f57dc8afe7e4b8a0e commit df05f2348600a72ebefdc16f57dc8afe7e4b8a0e Author: Jon Turney Date: Wed Mar 20 17:12:08 2019 +0000 Don't show FTP 550 'file not found' errors in a MessageBox Don't show FTP 550 'file not found' errors in a MessageBox. Also log InternetGetLastResponseInfo() when fetching a URL. WinInet documentation seems to indicate this kind of extended error information only exists when WinInet has an FTP error code to report. Log this error (and the associated URL) rather than just showing a MessageBox. Don't bother showing a MessageBox for 550, since we can tolerate certain missing files (e.g. setup.zst), and we'll go on to report that couldn't fetch needed files in a more generic way. https://sourceware.org/git/gitweb.cgi?p=cygwin-apps/setup.git;h=6b82fcbb70db715c859ab553f19755091bfbf276 commit 6b82fcbb70db715c859ab553f19755091bfbf276 Author: Jon Turney Date: Wed Mar 20 16:23:36 2019 +0000 Use 'kill -W' when killing processes We only have access to the Windows PID, which (since Cygwin 3.0.0) are completely decoupled from Cygwin PIDs. Use the new '-W' flag to indicate to kill that we are providing a Windows PID. (If the Cygwin install is too old to support kill -W, kill will fail, and we'll fall back to using TerminateProcess()) Diff: --- ListView.cc | 38 +++++++++++++++++++++++++++++++++++++- ListView.h | 1 + PickCategoryLine.cc | 6 ++++++ PickCategoryLine.h | 1 + PickPackageLine.cc | 11 +++++++++++ PickPackageLine.h | 1 + PickView.h | 5 ++++- nio-ie5.cc | 23 ++++++++++++++++------- package_meta.cc | 21 +++++++++++++++++++++ package_meta.h | 1 + processlist.cc | 2 +- 11 files changed, 100 insertions(+), 10 deletions(-) diff --git a/ListView.cc b/ListView.cc index d58780b..a6dc56d 100644 --- a/ListView.cc +++ b/ListView.cc @@ -345,9 +345,19 @@ ListView::OnNotify (NMHDR *pNmHdr, LRESULT *pResult) if (headers[iCol].type == ListView::ControlType::popup) { POINT p; - // position pop-up menu at the location of the click GetCursorPos(&p); + RECT r; + ListView_GetSubItemRect(hWndListView, iRow, iCol, LVIR_BOUNDS, &r); + POINT cp = p; + ::ScreenToClient(hWndListView, &cp); + + // if the click isn't over the pop-up button, do nothing yet (but this + // might be followed by a NM_DBLCLK) + if (cp.x < r.right - GetSystemMetrics(SM_CXVSCROLL)) + return true; + + // position pop-up menu at the location of the click update = popup_menu(iRow, iCol, p); } else @@ -366,6 +376,32 @@ ListView::OnNotify (NMHDR *pNmHdr, LRESULT *pResult) } break; + case NM_DBLCLK: + { + NMITEMACTIVATE *pNmItemAct = (NMITEMACTIVATE *) pNmHdr; +#if DEBUG + Log (LOG_BABBLE) << "NM_DBLCLICK: pnmitem->iItem " << pNmItemAct->iItem << " pNmItemAct->iSubItem " << pNmItemAct->iSubItem << endLog; +#endif + int iRow = pNmItemAct->iItem; + int iCol = pNmItemAct->iSubItem; + if (iRow < 0) + return false; + + int update = 0; + + // Inform the item of the double-click + update = (*contents)[iRow]->do_default_action(iCol ); + + // Update items, if needed + if (update > 0) + { + ListView_RedrawItems(hWndListView, iRow, iRow + update -1); + } + + return true; + } + break; + case NM_CUSTOMDRAW: { NMLVCUSTOMDRAW *pNmLvCustomDraw = (NMLVCUSTOMDRAW *)pNmHdr; diff --git a/ListView.h b/ListView.h index 97b138a..519b61a 100644 --- a/ListView.h +++ b/ListView.h @@ -37,6 +37,7 @@ class ListViewLine virtual int get_indent() const = 0; virtual ActionList *get_actions(int col) const = 0; virtual int do_action(int col, int id) = 0; + virtual int do_default_action(int col) = 0; virtual bool map_key_to_action(WORD vkey, int *col_num, int *action_id) const = 0; }; diff --git a/PickCategoryLine.cc b/PickCategoryLine.cc index c088cd4..7aa1f28 100644 --- a/PickCategoryLine.cc +++ b/PickCategoryLine.cc @@ -56,6 +56,12 @@ PickCategoryLine::do_action(int col_num, int action_id) return 1; } +int +PickCategoryLine::do_default_action(int col_num) +{ + return 0; +} + ActionList * PickCategoryLine::get_actions(int col) const { diff --git a/PickCategoryLine.h b/PickCategoryLine.h index 9486904..35b90e6 100644 --- a/PickCategoryLine.h +++ b/PickCategoryLine.h @@ -40,6 +40,7 @@ public: int get_indent() const; ActionList *get_actions(int col) const; int do_action(int col, int action_id); + int do_default_action(int col); bool map_key_to_action(WORD vkey, int *col_num, int *action_id) const; private: diff --git a/PickPackageLine.cc b/PickPackageLine.cc index 1f884be..685d632 100644 --- a/PickPackageLine.cc +++ b/PickPackageLine.cc @@ -144,6 +144,17 @@ PickPackageLine::do_action(int col_num, int action_id) return 0; } +int +PickPackageLine::do_default_action(int col_num) +{ + if (col_num == new_col) + { + pkg.toggle_action(); + return 1; + } + return 0; +} + ActionList * PickPackageLine::get_actions(int col_num) const { diff --git a/PickPackageLine.h b/PickPackageLine.h index a35f399..3877a4b 100644 --- a/PickPackageLine.h +++ b/PickPackageLine.h @@ -36,6 +36,7 @@ public: int get_indent() const; ActionList *get_actions(int col_num) const; int do_action(int col, int action_id); + int do_default_action(int col); bool map_key_to_action(WORD vkey, int *col_num, int *action_id) const; private: packagemeta & pkg; diff --git a/PickView.h b/PickView.h index 3a6c602..3715d93 100644 --- a/PickView.h +++ b/PickView.h @@ -136,7 +136,10 @@ public: i != _bucket.end(); i++) { - // recurse for all contained categories + // recurse for all contained non-obsolete categories + if (isObsolete((*i)->_cat.first)) + continue; + int l = (*i)->do_action(action_id, deftrust); if (!_collapsed) diff --git a/nio-ie5.cc b/nio-ie5.cc index 8786774..f5ad020 100644 --- a/nio-ie5.cc +++ b/nio-ie5.cc @@ -216,15 +216,24 @@ try_again: { DWORD e = GetLastError (); if (e == ERROR_INTERNET_EXTENDED_ERROR) - { - char buf[2000]; - DWORD e, l = sizeof (buf); - InternetGetLastResponseInfo (&e, buf, &l); - mbox (0, buf, "Internet Error", MB_OK); - } + { + char buf[2000]; + DWORD e, l = sizeof (buf); + InternetGetLastResponseInfo (&e, buf, &l); + + // show errors apart from file-not-found (e doesn't contain the + // response code so we have to resort to looking at the message) + if (strncmp("550", buf, 3) != 0) + mbox (0, buf, "Internet Error", MB_OK); + + for (unsigned int i = 0; i < l; i++) + if (buf[i] == '\n' or buf[i] == '\r') + buf[i] = ' '; + Log (LOG_PLAIN) << "connection error: " << buf << " fetching " << url << endLog; + } else { - Log (LOG_PLAIN) << "connection error: " << e << endLog; + Log (LOG_PLAIN) << "connection error: " << e << " fetching " << url << endLog; } } diff --git a/package_meta.cc b/package_meta.cc index cbb7388..3509f05 100644 --- a/package_meta.cc +++ b/package_meta.cc @@ -465,6 +465,27 @@ packagemeta::select_action (int id, trusts const deftrust) user_picked = true; } +// toggle between the currently installed version (or uninstalled, if not +// installed), and the naively preferred version (the highest non-test version) +void +packagemeta::toggle_action () +{ + if (desired != installed) + { + set_action(Default_action, installed); + } + else + { + packageversion naively_preferred; + std::set::iterator i = versions.begin (); + for (i = versions.begin (); i != versions.end (); ++i) + if (!packagedb::solver.is_test_package(*i)) + naively_preferred = *i; + + set_action(Install_action, naively_preferred); + } +} + ActionList * packagemeta::list_actions(trusts const trust) { diff --git a/package_meta.h b/package_meta.h index 0eff8d0..c9bfe7c 100644 --- a/package_meta.h +++ b/package_meta.h @@ -63,6 +63,7 @@ public: void set_action (_actions, packageversion const & default_version); ActionList *list_actions(trusts const trust); void select_action (int id, trusts const deftrust); + void toggle_action (); void set_message (const std::string& message_id, const std::string& message_string) { diff --git a/processlist.cc b/processlist.cc index 70bc2d7..4e8518a 100644 --- a/processlist.cc +++ b/processlist.cc @@ -91,7 +91,7 @@ Process::kill (int force) break; } - std::string kill_cmd = backslash (cygpath ("/bin/kill.exe")) + " " + signame + " " + stringify (processID); + std::string kill_cmd = backslash (cygpath ("/bin/kill.exe")) + " " + signame + " -W " + stringify (processID); ::run (kill_cmd.c_str ()); }