From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 128743 invoked by alias); 14 Jan 2018 00:00:46 -0000 Mailing-List: contact cygwin-apps-help@cygwin.com; run by ezmlm Precedence: bulk Sender: cygwin-apps-owner@cygwin.com List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Mail-Followup-To: cygwin-apps@cygwin.com Received: (qmail 128456 invoked by uid 89); 14 Jan 2018 00:00:24 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-24.8 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_0,GIT_PATCH_1,GIT_PATCH_2,GIT_PATCH_3,SPF_HELO_PASS,SPF_PASS,T_RP_MATCHES_RCVD autolearn=ham version=3.3.2 spammy=continued, transaction X-HELO: limerock03.mail.cornell.edu Received: from limerock03.mail.cornell.edu (HELO limerock03.mail.cornell.edu) (128.84.13.243) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Sun, 14 Jan 2018 00:00:22 +0000 X-CornellRouted: This message has been Routed already. Received: from authusersmtp.mail.cornell.edu (granite4.serverfarm.cornell.edu [10.16.197.9]) by limerock03.mail.cornell.edu (8.14.4/8.14.4_cu) with ESMTP id w0E00EsQ003996 for ; Sat, 13 Jan 2018 19:00:14 -0500 Received: from [10.13.22.4] (50-192-26-108-static.hfc.comcastbusiness.net [50.192.26.108]) (authenticated bits=0) by authusersmtp.mail.cornell.edu (8.14.4/8.12.10) with ESMTP id w0E00Cv8021487 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NOT) for ; Sat, 13 Jan 2018 19:00:13 -0500 Subject: Re: [PATCH setup 00/14] Use libsolv, solve all our problems... (WIP) To: cygwin-apps@cygwin.com References: <20170531105015.162228-1-jon.turney@dronecode.org.uk> <488ba627-de58-ddc7-7f69-696adae76b8a@cornell.edu> <7a173f99-a2e1-a07c-a9df-5bebcf377957@cornell.edu> <87poau9znx.fsf@Rainer.invalid> <050204e5-0ed3-8e47-3825-58ec6a10f44f@cornell.edu> <87ingltcn0.fsf@Rainer.invalid> <4ed6c549-dddd-fc45-3ed8-f7339548d7cd@cornell.edu> <1ec0f4de-380f-c6d1-59e7-03570f36b80b@cornell.edu> <87609alczj.fsf@Rainer.invalid> <31df6cf0-1abd-9cb0-a5c3-3c2b0a7d069c@cornell.edu> <87e4ba12-ed92-1959-d8ae-ab51986f7036@dronecode.org.uk> <79d71e69-57a9-8303-79c8-ba2bfdd098d8@cornell.edu> <1ecbad19-013e-6cf2-7dd8-019f72f7be3b@cornell.edu> <9ebf0e53-c87e-30f7-114f-a0456004b847@SystematicSw.ab.ca> From: Ken Brown Message-ID: <032b0ecf-744d-b1bf-00c5-d3adaa2a4a02@cornell.edu> Date: Sun, 14 Jan 2018 00:00:00 -0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.5.2 MIME-Version: 1.0 In-Reply-To: Content-Type: multipart/mixed; boundary="------------A8EF4B67D83D3ABD7E0D232D" X-PMX-Cornell-Gauge: Gauge=X X-PMX-CORNELL-AUTH-RESULTS: dkim-out=none; X-IsSubscribed: yes X-SW-Source: 2018-01/txt/msg00044.txt.bz2 This is a multi-part message in MIME format. --------------A8EF4B67D83D3ABD7E0D232D Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 8bit Content-length: 2401 On 1/13/2018 5:55 PM, Ken Brown wrote: > On 1/13/2018 4:29 PM, Brian Inglis wrote: >> On 2018-01-13 12:56, Ken Brown wrote: >>> 2. We should probably remove, or at least reword, the dire warning about >>> accepting the default solutions.  I'm not sure we want to "strongly >>> recommend" >>> the default solution over the other solution(s).  I guess what we >>> really want to >>> say is that we strongly recommend resolving the problems before >>> continuing. >> >> For users who only run setup and use programs, a dire warning and strong >> recommendations are appropriate. >> >> Alternatives are to also remove all packages dependent on the package >> to be >> removed, or lastly, to remove only the requested package, leaving the >> installation inconsistent. Those alternatives would have to be >> presented to the >> user for selection, then executed. >> >> Anything else requiring the user to resolve would require a FAQ entry >> explaining >> what that meant, what diagnosis and actions would be required, and >> that would >> probably generate emails from users asking what they should do. >> >> Better to allow the solver to resolve issues and just let users choose >> straightforward alternatives, with the view of trying to keep the >> installation >> consistent, unless explicitly overridden, such as to test an alternative >> implementation of a dependency installed outside of setup. > > The current situation on the topic/libsolv branch is the following. > Suppose A requires B and the user asks to uninstall B.  They will get a > problem report showing two possible solutions: > > 1. Uninstall A. > 2. (default) Don't uninstall B. > > If they uncheck 'Accept default solutions' and select 'Next', they'll > get a warning that says "We strongly recommend that you accept the > default solutions.  Some packages may not work properly if you don't. > Are you sure you want to proceed?" > > This is misleading insofar as it implies that something bad will happen > if the user prefers to solve the problem by uninstalling A.  What is > true is that some packages may not work properly if the user answers 'Yes'. > > I think we should be able to find wording that is accurate while still > making it clear that we recommend going back and fixing the problem.  I > don't yet have a good candidate for that wording. Something like the attached might do the job. Ken --------------A8EF4B67D83D3ABD7E0D232D Content-Type: text/plain; charset=UTF-8; name="0001-Clarify-the-unsolved-problems-warning.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="0001-Clarify-the-unsolved-problems-warning.patch" Content-length: 1551 >From d15d18dfa4db91416155385034bccf31be88ece3 Mon Sep 17 00:00:00 2001 From: Ken Brown Date: Sat, 13 Jan 2018 18:50:14 -0500 Subject: [PATCH] Clarify the unsolved-problems warning If the user unchecks the 'Accept default solutions' box and selects 'Next', don't imply that choosing a non-default solution would break their system. --- prereq.cc | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/prereq.cc b/prereq.cc index a03e79b..4926c65 100644 --- a/prereq.cc +++ b/prereq.cc @@ -96,10 +96,9 @@ PrereqPage::OnNext () { // breakage imminent! danger, danger int res = MessageBox (h, - "We strongly recommend that you accept the default solutions. " - "Some packages may not work properly if you don't." + "Some packages may not work properly if you continue." "\r\n\r\n" - "Are you sure you want to proceed?", + "Are you sure you want to proceed (NOT RECOMMENDED)?", "WARNING - Unsolved Problems", MB_YESNO | MB_ICONEXCLAMATION | MB_DEFBUTTON2); if (res == IDNO) @@ -107,7 +106,7 @@ PrereqPage::OnNext () else { Log (LOG_PLAIN) << - "NOTE! User refused the default solutions! " + "NOTE! User continued with unsolved problems! " "Expect some packages to give errors or not function at all." << endLog; // Change the solver's transaction list to reflect the user's choices. db.solution.db2trans(); -- 2.15.1 --------------A8EF4B67D83D3ABD7E0D232D--