2008/2/5, David Roundy <[EMAIL PROTECTED]>: > On Tue, Feb 05, 2008 at 03:30:23AM +0300, Dmitry Kurochkin wrote: > > Some fixes and cleanups for configure checks. > > > > And fix for copyRemotesNormal - if we use copyUrlFirst the first patch > > is downloaded last. So we lock in waitUrl and progress indication stops. > > Thanks for these changes! Applied. (once they pass tests and all that) > > > Tue Feb 5 02:59:06 MSK 2008 Dmitry Kurochkin <[EMAIL PROTECTED]> > > * Append CPPFLAGS to GHCFLAGS in autoconf.mk.in instead of configure.ac. > > > > Tue Feb 5 03:01:09 MSK 2008 Dmitry Kurochkin <[EMAIL PROTECTED]> > > * Fix configure check for curl_multi_timeout. > > Very nice! I'd suspected this would reduce the amount of code considerably, > but I hadn't expected quite *this* much. :)
Me too :) > > > Tue Feb 5 03:07:17 MSK 2008 Dmitry Kurochkin <[EMAIL PROTECTED]> > > * Cleanup curl pipelining configure check. > > > > Tue Feb 5 03:18:41 MSK 2008 Dmitry Kurochkin <[EMAIL PROTECTED]> > > * Cleanup libwww configure check. > > > > Tue Feb 5 03:21:42 MSK 2008 Dmitry Kurochkin <[EMAIL PROTECTED]> > > * Use copyUrl in copyRemotesNormal. > > The only downside of this fix is that it means that we download all > "speculatively" scheduled files before any of those that are needed > strictly. I don't think this is a real problem, because copyRemotes is > rarely used, and I don't think ever follows speculative downloads. But I > do think a more correct approach might be to do something like copyUrlFirst > on the reverse of the list. Although that'll also have issues of getting > the files-already-downloaded count off by five or so... What about doing waitUrl on reverse list? I am not familiar with progress reporting code and I do not understand why counter would be off by five... How speculateFileOrUrl will be used? I do not see it used anywhere now. Regards, Dmitry > -- > David Roundy > Department of Physics > Oregon State University > _______________________________________________ > darcs-devel mailing list > darcs-devel@darcs.net > http://lists.osuosl.org/mailman/listinfo/darcs-devel > _______________________________________________ darcs-devel mailing list darcs-devel@darcs.net http://lists.osuosl.org/mailman/listinfo/darcs-devel
