From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 64150 invoked by alias); 11 Sep 2017 20:40: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 64139 invoked by uid 89); 11 Sep 2017 20:40:45 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.9 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.2 spammy=Turney, turney, solver, clicking X-HELO: limerock02.mail.cornell.edu Received: from limerock02.mail.cornell.edu (HELO limerock02.mail.cornell.edu) (128.84.13.242) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 11 Sep 2017 20:40:44 +0000 X-CornellRouted: This message has been Routed already. Received: from authusersmtp.mail.cornell.edu (granite3.serverfarm.cornell.edu [10.16.197.8]) by limerock02.mail.cornell.edu (8.14.4/8.14.4_cu) with ESMTP id v8BKegP7025233 for ; Mon, 11 Sep 2017 16:40:42 -0400 Received: from [192.168.0.4] (mta-68-175-129-7.twcny.rr.com [68.175.129.7] (may be forged)) (authenticated bits=0) by authusersmtp.mail.cornell.edu (8.14.4/8.12.10) with ESMTP id v8BKeeVh009513 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NOT) for ; Mon, 11 Sep 2017 16:40:41 -0400 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> From: Ken Brown Message-ID: <7a173f99-a2e1-a07c-a9df-5bebcf377957@cornell.edu> Date: Mon, 11 Sep 2017 20:40:00 -0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: <488ba627-de58-ddc7-7f69-696adae76b8a@cornell.edu> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit X-PMX-Cornell-Gauge: Gauge=XXXXX X-PMX-CORNELL-AUTH-RESULTS: dkim-out=none; X-IsSubscribed: yes X-SW-Source: 2017-09/txt/msg00013.txt.bz2 On 9/8/2017 2:54 PM, Ken Brown wrote: > On 5/31/2017 6:50 AM, Jon Turney wrote: >> ... solve some problems, perhaps add some new ones, I guess.  I'm not >> 100% >> sure this is the right approach to take, but I wrote it, so here it is. > > I've now fixed all the bugs I've noticed, and I think I've gotten the > libsolv branch pretty close to the point where it is usable (barring > further bugs). > >> - I've dodged a lot of the UI issues: If the solver reports problems, >> all that >> can be done is accept the default solution or cancel. > > I've changed this so that the Back button takes the user back to the > chooser page, with all packages reset to show the solver's default > solution.  The user can then see what will be done and, if desired, try > to solve the problems in other ways. > >> - We had a very poor UI for showing what will actually be done >> (combine in >> your head the "Pending" view with packages listed in the text on the >> PrereChecker page), and this removes part of that > > This is fixed by the above. > >> - As implemented, selecting "Current" overrides "Keep".  This is >> wrong, and a >> change from current behaviour, > > Fixed. > > Here are some other things I've done, aside from fixing bugs: > > - As discussed earlier in the thread, I've made it possible for the user > to install test packages without clicking the Test button. > > - I've artificially created a conflict if the user tries to uninstall a > Base package.  There's probably a better way to deal with this. > > - I've made the solver check dependencies of installed packages.  (It > doesn't do this by default, which seems strange to me.) > > There's still one UI issue that I haven't dealt with: If the solver > finds missing dependencies, setup simply installs them silently without > informing the user (except in the log).  In particular, the user can no > longer refuse to install them.  I'm not sure how to best deal with this. I've changed this so that the user can now review the packages that will be installed. But I still haven't found a good way to enable the user to refuse dependencies. Ken