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. Corinna -- Corinna Vinschen Please, send mails regarding Cygwin to Cygwin Project Co-Leader cygwin AT cygwin DOT com Red Hat
