From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 64246 invoked by alias); 31 Aug 2016 10:48:56 -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 64211 invoked by uid 89); 31 Aug 2016 10:48:55 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.4 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW autolearn=ham version=3.3.2 spammy=broad, 1251, 1241, ambiguous X-HELO: out4-smtp.messagingengine.com Received: from out4-smtp.messagingengine.com (HELO out4-smtp.messagingengine.com) (66.111.4.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 31 Aug 2016 10:48:45 +0000 Received: from compute5.internal (compute5.nyi.internal [10.202.2.45]) by mailout.nyi.internal (Postfix) with ESMTP id A0A53205A2 for ; Wed, 31 Aug 2016 06:48:43 -0400 (EDT) Received: from frontend1 ([10.202.2.160]) by compute5.internal (MEProxy); Wed, 31 Aug 2016 06:48:43 -0400 Received: from [192.168.1.102] (host31-51-206-108.range31-51.btcentralplus.com [31.51.206.108]) by mail.messagingengine.com (Postfix) with ESMTPA id 4BEEEF29CF for ; Wed, 31 Aug 2016 06:48:43 -0400 (EDT) Subject: Re: [PATCH setup] Allow setup to parse more than 3 versions from the setup.ini file To: cygwin-apps@cygwin.com References: <1433349024-9776-1-git-send-email-jon.turney@dronecode.org.uk> <20150608134318.GO3416@calimero.vinschen.de> From: Jon Turney Message-ID: <05b7266e-6a83-b860-9079-f8fd8c310175@dronecode.org.uk> Date: Wed, 31 Aug 2016 10:48:00 -0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: <20150608134318.GO3416@calimero.vinschen.de> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2016-08/txt/msg00114.txt.bz2 On 08/06/2015 14:43, Corinna Vinschen wrote: > On Jun 3 17:30, Jon TURNEY wrote: >> Reminded by a recent request as to how to install xorg-server-1.17.1-2, which >> has disappeared beyond setup's ken (in order to determine if there was a >> regression in the curent version), this is a re-send of a patch I originally >> submitted back in 2011 [1], which received an ambiguous response then. >> >> [1] https://cygwin.com/ml/cygwin-apps/2011-04/msg00053.html >> >> This recognizes any "[foo]" line as introducing the information for another >> version, which doesn't have one of the trust levels [curr], [prev] or [test], >> and so isn't automatically selected when setup is told to install all packages >> at that trust level (by default, [curr]). >> >> Setup already does all the neccessary sorting in version order etc. to use these >> additional versions. >> >> Since the value of carries no meaning, it might make sense to update the >> setup.ini specification to mandate the use of specific strings like "[also]" or >> "[other]", or perhaps "[prev-1]", "[prev-2]", etc. >> >> I have written a corresponding patch to genini. >> >> I'm not sure what expiry policy is currently used by upset for old packages, but >> presumably that would need to be made more sophisticated, along with the changes >> needed to generate setup.ini entries for other versions. > > Upset does not handle expiry of packages at all. Versions are mentioned > in setup.hint as test, curr, prev, or exp (yes, really) and those are > handled, everything else throws an error message. Package versions not > mentioned in setup.hint are simply ignored. Yes, upset doesn't (didn't) explicitly handle expiry, but the fact that a package version is not mentioned in setup.ini causes it to be removed by stalepkgs, when that is next run. After the corresponding change to setup.ini generation, it will list all versions, so none would ever be eligible for expiry under that policy. Anyhow, improving that is close to the top of my hit-list for calm. > I'm not against adding some functionality along these lines (provided > you also fix upset), but I'm not so sure about the broad definition of > the version state pattern. It feels as generating problems down the > road. Think setup.hint. Your patch would requite to recognize more or > less any string as version state: > > category: [...] > requires: [...] > sdesc: [...] > ldesc: [...] > prev: 1.2.3-1 > curr: 1.2.4-1 > test: 1.2.5-1 > blub: 1.2.6-1 > fwexf3efx24x: 1.2.7-1 I wasn't envisioning these labels appearing in setup.hint at all, since that only gives the information 'this version exists', which is already apparent from the existence of the package file. (In fact, even 'prev' has no real meaning any more since the removal of the 'prev' stability selector in setup [1], apart from 'don't expire me') Nevertheless, you are correct, so (by specification) let's restrict the label to 'ver'. I should also have mentioned that existing versions of setup cannot parse setup.ini with these extra labels (reporting an error and asking 'do you have the latest setup?'), so it would be nice to get this into setup sooner rather than later, if possible. [1] https://sourceware.org/git/?p=cygwin-setup.git;a=commit;h=ec9c1d708418e05c4ba02b58a9869f1e232ad381