public inbox for cygwin-apps@cygwin.com
 help / color / mirror / Atom feed
From: Jon Turney <jon.turney@dronecode.org.uk>
To: cygwin-apps@cygwin.com
Cc: Ken Brown <kbrown@cornell.edu>
Subject: Re: [PATCH setup 00/14] Use libsolv, solve all our problems... (WIP)
Date: Tue, 05 Sep 2017 13:34:00 -0000	[thread overview]
Message-ID: <2d099e44-fb17-4bf3-f72c-e6019a6259b3@dronecode.org.uk> (raw)
In-Reply-To: <470d4572-4d63-81c2-7e26-6a642cce1e9f@cornell.edu>

On 02/09/2017 17:57, Ken Brown wrote:
> On 9/1/2017 11:00 AM, Ken Brown wrote:
>> On 5/31/2017 6:50 AM, Jon Turney wrote:
>>> What remains to be done:
>>> - 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.  This possibly 
>>> isn't a
>>> big problem until we have a package set which can contain problems...
>>> - 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
>>> - As implemented, selecting "Current" overrides "Keep".  This is 
>>> wrong, and a
>>> change from current behaviour, but is probably a symptom of some deeper
>>> confusion in the picker UI I'm not sure how to address

Thanks very much for taking the time to look at these changes and test them.

It was my intention to come back and take another look at this, but that 
hasn't happened yet :S

>> There are also some issues involving the treatment of test releases:
>>
>> - It's not possible to install a test release of a package without 
>> clicking the global Test button.  But then you have to manually choose 
>> Keep for all the package where you don't want the test release.

Yeah, I'm not sure if putting the test packages into a separate repo 
which is disabled unless explicitly enabled is the right approach.

(Instead, perhaps it is possible to tell the solver that certain 
repositories are disfavoured)

>> - Once a test release is installed, selecting Reinstall will downgrade 
>> the package if the global Test button is not pressed.  Presumably this 
>> is because the test release has become invisible, since its repo has 
>> been disabled.  But I haven't checked carefully to see exactly what's 
>> going on.

>> - I found some glitches involving SHA512 sums of test releases.  It's 
>> probably not worth pursuing this until the handling of test releases 
>> is redone.
> 
> I fixed the last two problems, which turned out to have nothing to do 
> with the way test releases are handle.  So all that's left of my three 
> issues is the first one, which is just a UI annoyance.
> 
> I don't think it makes sense for me to send further patches here. 
> Instead, I'll fork Jon's GitHub repo and continue to try to make 
> improvements.

Awesome.

  reply	other threads:[~2017-09-05 13:34 UTC|newest]

Thread overview: 75+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-05-31 10:53 Jon Turney
2017-05-31 10:53 ` [PATCH setup 03/14] Hoist addScript() etc. up from packageversion to packagemeta Jon Turney
2017-05-31 10:53 ` [PATCH setup 01/14] Opaque how PackageDepends is stored Jon Turney
2017-05-31 10:53 ` [PATCH setup 02/14] Factor out reading installed.db Jon Turney
2017-05-31 10:53 ` [PATCH setup 04/14] Hoist pick() up to packagemeta Jon Turney
2017-05-31 10:53 ` [PATCH setup 05/14] Hoist uninstall up to Installer::uninstallOne() Jon Turney
2017-05-31 10:57 ` [PATCH setup 07/14] Store package stability in class packageversion Jon Turney
2017-05-31 10:57 ` [PATCH setup 10/14] Remove packageversion class Jon Turney
2017-05-31 10:57 ` [PATCH setup 08/14] Change to using a libsolv pool for storing package information Jon Turney
2017-05-31 10:57 ` [PATCH setup 09/14] Remove cygpackage class Jon Turney
2017-05-31 10:57 ` [PATCH setup 06/14] Hoist scan() up from packageversion to packagemeta Jon Turney
2017-05-31 11:05 ` [PATCH setup 11/14] Drop in SolvableVersion as a replacement for packageversion Jon Turney
2017-05-31 11:05   ` [PATCH setup 14/14] Add obsoletes: support Jon Turney
2017-05-31 11:05   ` [PATCH setup 13/14] Download/checksum/install/uninstall what transaction wants Jon Turney
2017-05-31 11:05   ` [PATCH setup 12/14] Use solver to check for problems and produce a list of package transactions Jon Turney
2017-08-29 13:37 ` [PATCH setup 00/14] Use libsolv, solve all our problems... (WIP) Ken Brown
2017-08-30 21:47   ` Ken Brown
2017-09-01 15:01 ` Ken Brown
2017-09-02 16:57   ` Ken Brown
2017-09-05 13:34     ` Jon Turney [this message]
2017-09-05 18:40       ` Achim Gratz
2017-09-06  2:52         ` Ken Brown
2017-11-23 18:10           ` Jon Turney
2017-11-23 20:32             ` Ken Brown
2017-11-23 20:54             ` Achim Gratz
2017-09-08 18:54 ` Ken Brown
2017-09-11 20:40   ` Ken Brown
2017-09-13 19:17     ` Achim Gratz
2017-09-13 21:16       ` Ken Brown
2017-09-14 17:26         ` Achim Gratz
2017-09-14 20:46           ` Ken Brown
2017-09-15 19:24             ` Jon Turney
2017-09-16 16:21               ` Ken Brown
2017-09-19 12:24                 ` Ken Brown
2017-09-19 16:46                   ` Jon Turney
2017-09-19 16:58                     ` Ken Brown
2017-12-05 14:32             ` Jon Turney
2017-12-05 17:36               ` Ken Brown
2017-12-13 17:31               ` Ken Brown
2017-12-13 18:06                 ` Achim Gratz
2017-12-13 22:31                   ` Ken Brown
2017-12-14 14:12                     ` Ken Brown
2017-12-24 15:00                     ` Ken Brown
2018-01-09 13:25                       ` Jon Turney
2018-01-09 15:37                         ` Ken Brown
2018-01-09 15:49                           ` Ken Brown
2018-01-13 14:14                             ` Jon Turney
2018-01-13 19:56                               ` Ken Brown
2018-01-13 21:29                                 ` Brian Inglis
2018-01-13 22:55                                   ` Ken Brown
2018-01-14  0:00                                     ` Ken Brown
2018-01-14  1:52                                       ` Brian Inglis
2018-01-14  2:37                                         ` Ken Brown
2018-01-15 19:02                                 ` Jon Turney
2018-01-15 21:50                                   ` Ken Brown
2018-01-18 19:14                                     ` Jon Turney
2017-09-15 15:15   ` Jon Turney
2017-09-15 16:53     ` Ken Brown
2017-09-15 20:56       ` cyg Simple
2017-09-17 16:02         ` Ken Brown
2017-09-26 14:50       ` Jon Turney
2017-09-26 16:07         ` Ken Brown
2017-09-27 19:14           ` Jon Turney
2017-09-27 20:33             ` Ken Brown
2017-09-29 17:38               ` Jon Turney
2017-09-29 20:34                 ` Ken Brown
2017-10-02 14:07                   ` Jon Turney
2017-10-02 15:17                     ` Marco Atzeri
2017-10-04 14:43                       ` Jon Turney
2017-10-10 11:18                   ` Ken Brown
2017-10-10 15:49                     ` Jon Turney
2017-10-17 12:45                     ` Ken Brown
2017-10-17 18:47                       ` Jon Turney
2017-10-18 15:28                         ` Ken Brown
2017-10-18 15:57                           ` Ken Brown

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2d099e44-fb17-4bf3-f72c-e6019a6259b3@dronecode.org.uk \
    --to=jon.turney@dronecode.org.uk \
    --cc=cygwin-apps@cygwin.com \
    --cc=kbrown@cornell.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).