From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 105491 invoked by alias); 8 Nov 2017 18:46:03 -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 105394 invoked by uid 89); 8 Nov 2017 18:46:03 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=AWL,BAYES_00,KAM_LAZY_DOMAIN_SECURITY,RCVD_IN_DNSWL_LOW autolearn=no version=3.3.2 spammy=utterly, awhile, H*Ad:D*ca, convinced X-HELO: smtp-out-no.shaw.ca Received: from smtp-out-no.shaw.ca (HELO smtp-out-no.shaw.ca) (64.59.134.9) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 08 Nov 2017 18:46:01 +0000 Received: from [192.168.1.100] ([24.64.240.204]) by shaw.ca with SMTP id CVMNeYvLpI8mCCVMOeadKS; Wed, 08 Nov 2017 11:46:00 -0700 X-Authority-Analysis: v=2.2 cv=HahkdmM8 c=1 sm=1 tr=0 a=MVEHjbUiAHxQW0jfcDq5EA==:117 a=MVEHjbUiAHxQW0jfcDq5EA==:17 a=N659UExz7-8A:10 a=CCpqsmhAAAAA:8 a=8PirogmhM8N_zB6vXoAA:9 a=pILNOxqGKmIA:10 a=wiOExPc6FJcA:10 a=ul9cdbp4aOFLsgKbc677:22 Reply-To: Brian.Inglis@SystematicSw.ab.ca Subject: Re: [PATCH setup 0/2] Improve behavior after download error To: cygwin-apps@cygwin.com References: <20171106214907.7120-1-kbrown@cornell.edu> From: Brian Inglis Message-ID: <2b0378a3-d539-4b2e-2bed-04943e5a9798@SystematicSw.ab.ca> Date: Wed, 08 Nov 2017 18:46:00 -0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit X-CMAE-Envelope: MS4wfM0L2H6PYGtJSw8zNN4j7p7bal4Epi8o5GuwcVSyKnc/4wpPJLX6Q66q6xF+4ptgn8HapJdV/lTVZ2yx97kNocIuTwbQUXZkfaCp+LyxBnapZCWF5laC esouSJs3pvfmm/jJIA72oTwVD2jwLrgDkLaa1hiihERTVvqdR1qgRc5EWV7Zk4gNiYGE1c1lLHfJWg== X-IsSubscribed: yes X-SW-Source: 2017-11/txt/msg00028.txt.bz2 On 2017-11-07 11:56, 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 >>> https://sourceware.org/ml/cygwin-apps/2017-11/msg00003.html.  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? Don't remember ever having a package download failure, normally mirror download problems are at the setup.ini... stage, possibly accessing during an rsync, then the packages download okay. > In any case, this is an incremental improvement, especially over the utterly mad > thing we are currently doing in unattended mode. I like to retry the download on the current mirror, then either pick another mirror, or quit and wait awhile. In (semi-)unattended mode, you should either decide to quit as if unattended, or possibly go into full interactive mode and allow Back, Next, Cancel without limits? -- Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada