[PATCH setup libsolv, v2] Let the user review added dependencies
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