cURL / Mailing Lists / curl-library / Single Mail

curl-library

RE: [PATCH] MSVC: remove old makefiles, refer to winbuild/ only

From: Steve Holme <steve_holme_at_hotmail.com>
Date: Thu, 3 Apr 2014 09:55:37 +0100

On Thu, 3 Apr 2014, Daniel Stenberg wrote:

> They do. Makefile.dist has a bunch of makefile targets that all do
> things like this:

Ah yes, sorry - for some reason I was thinking the dependency was the other way round and that somehow makefile.vc* needed makefile.dist rather than maefile.dist invoking makefile.vc* :(

> We can thus change the relevant targets in Makefile.dist to instead
> invoke the makefile in winbuild/. I'll offer that change _before_ I
> remove any files as a sort of backup to break less things in one go!
> =)

Fair enough.

> Then when that works, we can probably insert the winbuild/ logic into
> Makefile.dist but that's then just a technical detail since it'll still
> work the same.

It sounds like a plan to me - I'm not sure I have the necessary makefile skills myself to make the changes but its something I would like to be involved with from an autobuild point of view.

> I want us to deprecate the cmake build system since I believe it
> mostly confuses users and distracts them into thinking they can
> actually use that to build a curl/libcurl version in similar style
> that configure or the makefile based builds can do but it is just
> so inferior in comparison.

We use cmake here for our product's various build targets so I believe it has it's place but whether or not it has a place in (lib)curl is not really something I think I can comment on.

> But I'll take that in a separate thread later on.

No worries.

BTW, Please feel free to send over any patches if you want me to test them on my autobuild machines.

Kind Regards

Steve
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette: http://curl.haxx.se/mail/etiquette.html
Received on 2014-04-03