RFC: setup package in Base
Ken Brown
kbrown@cornell.edu
Mon Jul 9 12:32:00 GMT 2018
On 7/7/2018 8:40 AM, Jon Turney wrote:
> On 06/07/2018 15:05, Jon Turney wrote:
>> On 06/07/2018 14:51, Jon Turney wrote:
>>> Thanks for the patch.
>>>
>>> The version is checked (again), at ini.cc:404
>>>
>>> I've never understood why we have this twice.
>>>
>>> (I think the idea might be that first we are checking the setup
>>> version as a proxy for the setup.ini format version, to warn if there
>>> might be problems parsing it. The second time we are checking the
>>> setup version to see if an upgrade of setup is possible)
>>
>> Ah, so now we have setup-minimum-version, checking setup-version: in
>> the ini parser should probably be removed (or disabled when
>> setup-minimum-version: is present?)
>
> Like so:
Yes, this is better than my attempt. I hadn't noticed that
setup-version: was checked again in ini.cc.
Ken
More information about the Cygwin-apps
mailing list