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] Stop automatic dependency selection on setup.exe chooser screen


On 7 August 2010 09:56, Corinna Vinschen wrote:
> On Aug Â7 06:39, Andy Koppe wrote:
>> On 7 August 2010 05:29, Christopher Faylor wrote:
>> > On Fri, Aug 06, 2010 at 10:29:43PM +0100, Andy Koppe wrote:
>> >>FWIW, attached is my take on a fix for this. ÂGetting the same
>> >>dependencies, but additional "Required bys". Â(Actually libcurl4 and
>> >>libssh2_1 are extra dependencies, but I assume you already have them
>> >>installed.
>> >
>> > This seems to do the same job as Corinna's and it seems to be a smaller
>> > patch. ÂIs there a possibility of an endless loop here at all? ÂI don't
>> > think there is but I want to be sure.
>>
>> Attached is a simpler version, after I realised that the 'known' set
>> isn't actually needed. Added a few more comments as well.
>>
>> I don't think an endless loop is a possibility, because only
>> dependencies that aren't in 'unmet' already are added to the todo
>> queue, so each package can make it into the queue only once.
>
> Your code looks much better than mine. ÂPlease go ahead and check it
> in, after testing.

Thanks, will do.

Andy


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