[darcs-devel] darcs patch - remove libcurl dependency

[Eric Y. Kow]

Ok, let's take our time with this one :-)
I'm in no hurry, just gave my first ooh-less-code-yay reaction.

On Thu, Jun 14, 2007 at 07:04:27 -0700, David Roundy wrote:
> This sounds like a serious reversion to me.  It adds a new build dependency
> that we previously didn't have, and also removes features (pull via ftp).

Oops! I hadn't considered the loss of FTP functionality.  That's a bit
of a non-starter I suppose.

Sorry, Justin, I'm going to have to ask you to keep working on this
until you get FTP stuff back.  I might also consider a patch that added
this as an alternative to libcurl (thus giving us 4 options)

> I haven't looked at the actual code, but why not leave the existing
> download mechanisms in place? Removing code is great if the code isn't
> useful, but I suspect many of the same people who are unable/unwilling to
> install libcurl would be unable/unwilling to install the HTTP package.

Well, for some reason I had convinced myself that installing get another
Haskell-related package would be less painful for people, given that
they already to install various Haskell packages anyway.  Then again,
I suppose the stuff they would need to install tends to be standard
(for example, shipped with the vanilla GHC on MacPorts), whereas the
HTTP library is not.  So I guess my thinking was off.

> Why replace three more powerful options with a single less powerful option
> that also complicates the build process (not the configure script, but the
> sequence that user must go through to compile darcs)?

Well, I think Justin had some motivating examples, and there was
issue377 where Tim mentioned replacing libcurl with Network.HTTP
and a few forkIOs

> I do hope that at a minimum the new patch allows us to compile
> without HTTP support?

Justin, this would be a good idea.

-- 
Eric Kow                     http://www.loria.fr/~kow
PGP Key ID: 08AC04F9         Merci de corriger mon français.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 186 bytes
Desc: not available
Url : http://lists.osuosl.org/pipermail/darcs-devel/attachments/20070614/0575a1b8/attachment.pgp