This is the mail archive of the cygwin-apps mailing list for the Cygwin project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH setup 0/2] Improve behavior after download error

On 11/9/2017 11:42 AM, Ken Brown wrote:
On 11/9/2017 8:21 AM, Jon Turney wrote:
On 08/11/2017 18:52, Brian Inglis wrote:
On 2017-11-08 07:35, Ken Brown wrote:
On 11/7/2017 1:56 PM, Jon Turney wrote:
On 07/11/2017 04:28, Brian Inglis wrote:
On 2017-11-06 14:49, Ken Brown wrote:
This is a followup to ; The
focus of that thread was a crash that occurs on the topic/libsolv
branch.  Here I'm more interested in a UI issue.  Namely, I don't
think it's reasonable that setup goes back to the site page if the
user clicks Yes in response to "Download Incomplete. Try again?".
This is not what the message says will happen, and I'm not convinced
that it even works right if the user changes mirrors after being sent
to the site page.

Would it make more sense to drop to the package chooser page, after issuing the error message and advising the user to: select Back to go to the package chooser
page, select Next to retry the downloads, or select Cancel to exit?

Do we actually report the package name for the failed download so that the
user could make an informed change in the package chooser?

No.  Currently the only way for the user to find out is to finish the setup run and then look at the log.  There's been a FIXME about this at the end of since 2001.  Maybe it's time to fix this. We could simply keep a list of packages (or files?) for which the download failed, and
then report this in the "Download incomplete" dialog.

Note that in the pathological case of a mirror which only has a setup.ini, the list of failed packages could be very large.

I guess we should limit the number of failed packages that we report.

I'm about to send a patch series that implements Brian's suggestion and adds a (limited) list of failed packages.

Jon, these patches are to be applied to the libsolv branch, on top of your recent series of 5 patches.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]