[PATCH setup libsolv, v2] Let the user review added dependencies

Jon Turney jon.turney@dronecode.org.uk
Sun Jan 28 18:11:00 GMT 2018

On 25/01/2018 03:57, Ken Brown wrote:
> On 1/24/2018 10:25 PM, Ken Brown wrote:
>> On 1/24/2018 3:30 PM, Jon Turney wrote:
>>> On 22/01/2018 17:37, Ken Brown wrote:
>>>> If the solver found no problems but added packages to resolve
>>>> dependencies, ask the user whether they want to review the added
>>>> packages before proceeding.
>>>> If they answer Yes, go back to the chooser with the 'Pending' view
>>>> selected.  The implementation adds several new members to the
>>>> PrereqChecker class so that the latter can communicate with the
>>>> chooser page.
>>> As discussed, this approach could be confusing.
>>> Attached is a slightly different approach, which adds a new page to 
>>> review and confirm what actions we're going to take.
>>> For the moment, this just contains a simple text report, but I guess 
>>> this could be extended e.g. to use a grid control, or give reasons 
>>> for why packages are being installed.
>> I like this a lot.  It's *much* better than my approach.
>> My only suggestion is that you rethink what the confirm page should do 
>> in "download only" mode.  Maybe that page could even be skipped.  But 
>> if you think the user should confirm the download, there's no need to 
>> mention "uninstall" actions, and "install" actions should be reported 
>> as "download".
> One other minor detail: What should happen if nothing is being done?  I 
> don't think we necessarily want to skip the confirm page, because maybe 
> the user was expecting something to happen.  But it looks a little 
> strange to just display an empty report.  Maybe the report should say 
> something like "No changes will be made" or, in download mode, "Nothing 
> will be downloaded".

Good points. I adjusted the output as you suggest.

More information about the Cygwin-apps mailing list