From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 81043 invoked by alias); 16 Dec 2017 01:50:44 -0000 Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: cygwin-owner@cygwin.com Mail-Followup-To: cygwin@cygwin.com Received: (qmail 81011 invoked by uid 89); 16 Dec 2017 01:50:43 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.2 required=5.0 tests=AWL,BAYES_40,KAM_LAZY_DOMAIN_SECURITY,RCVD_IN_DNSWL_LOW autolearn=no version=3.3.2 spammy=calgary, Calgary, alberta, Alberta 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; Sat, 16 Dec 2017 01:50:42 +0000 Received: from [192.168.1.100] ([24.64.240.204]) by shaw.ca with ESMTP id Q1cdecOfqZ8gBQ1ceeK29A; Fri, 15 Dec 2017 18:50:40 -0700 X-Authority-Analysis: v=2.2 cv=M/g9E24s c=1 sm=1 tr=0 a=MVEHjbUiAHxQW0jfcDq5EA==:117 a=MVEHjbUiAHxQW0jfcDq5EA==:17 a=N659UExz7-8A:10 a=w_pzkKWiAAAA:8 a=4Rt9iLsgBowfNyw5XZMA:9 a=pILNOxqGKmIA:10 a=RcyqAG5v6fMA:10 a=sRI3_1zDfAgwuvI8zelB:22 Reply-To: Brian.Inglis@SystematicSw.ab.ca Subject: Re: setup's response to a "corrupt local copy" To: cygwin@cygwin.com References: <116333312.20171214224616@yandex.ru> From: Brian Inglis Message-ID: Date: Sat, 16 Dec 2017 04:00:00 -0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit X-CMAE-Envelope: MS4wfBEsGixhCgdlpLlF90HH8qp5rWShcM33K+bcasZlh2n+5fJnYNRlbDLfdCELnB3Ied5KFMySWHav5X8giKwW3PMQnqaaKyIS23cYXHZkQXWYw/Bn0lbJ mjjHmVwY3X7ADd2VjtCQ/VpbdY48ep7KSE9mL2VnocElrQF9BB3M+s/2IitAMOqdMyLLOQqyHRKgMg== X-IsSubscribed: yes X-SW-Source: 2017-12/txt/msg00152.txt.bz2 On 2017-12-14 12:57, Ken Brown wrote: > On 12/14/2017 2:46 PM, Andrey Repin wrote: >>> This is a followup to the discussion started here: >>>     https://cygwin.com/ml/cygwin/2017-12/msg00088.html >>> When setup is preparing to download files and it finds a corrupt copy in >>> the local cache, it issues a fatal error message telling the user to >>> remove the corrupt file and retry.  Steven said that setup should >>> silently delete the corrupt file, while I argued in favor of the current >>> behavior, on the grounds that setup shouldn't be deleting user files if >>> it doesn't know where they came from. >> The point being, if this is a "Download" Setup mode, the files are NOT "User" >> files, but a local setup cache. And all files therein SHOULD be valid package >> archives. >> There's of course situations, when setup.ini on server become corrupted or >> otherwise out of sync. But being rare, they should not interfere too much. >>> There is a middle ground: setup could query the user.  Additionally, as >>> suggested by cyg Simple, there could be an option that directs setup to >>> silently remove corrupt files. >> Make it mode dependent. >> If it's a "download[ and install]" mode, cleanup and redownload. >> If redownloaded file still does not match the setup.ini hash or if it's an >> "install from local cache" mode, leave file alone for investigation and notify >> the user. > You've misunderstood the context.  The error is only shown in download or > download/install mode.  And, as I said, it happens when setup is *preparing* to > download files and finds a corrupt copy already present in the local cache.  In > that context, setup has no idea where the file came from. Setup knows the file came from .../%encoded-mirror-url/arch/release/package/.../pvr....tar.xz so it could warn the user of the validation failure, rename the file failing validation if the cache is (one of) the currently selected mirror(s) and inform the user of the rename, then add the package for redo of download/install from (one of) the currently selected mirror(s), similar to a package prereq. This would DTRT in most cases and not blow away the questionable cached file. -- Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada -- Problem reports: http://cygwin.com/problems.html FAQ: http://cygwin.com/faq/ Documentation: http://cygwin.com/docs.html Unsubscribe info: http://cygwin.com/ml/#unsubscribe-simple