Download
Browse source Changelog tiny-curl
Documentation
Project   Bug Bounty   Help us   Known bugs   TODO Protocols   CA bundle   HTTP Cookies   HTTP/2   SSL Certs Releases   Security   Version numbers   Vulnerabilities curl tool   man page   Tutorial   HTTP scripting Who and Why
libcurl
ABI API Competitors Examples Features Mailing list Related libs Using libcurl Tutorial Testimonials
Get Help
curl-library curl-users Mailing lists Book: Everything curl Video presentations Report a bug Paid support
Development
Autobuilds Code review Code style Contribute Dashboard Deprecate Internals Release Notes Release Procedure Roadmap Run Tests Security Process Specifications Test curl
News
Changelog Release table
curl / Mailing Lists / curl-library / Single Mail
Buy commercial curl support from WolfSSL. We help you work out your issues, debug your libcurl applications, use the API, port to new platforms, add new features and more. With a team lead by the curl founder himself.

RE: time to remove Visual Studio project files?

From: Marcel Raad via curl-library <curl-library_at_cool.haxx.se>
Date: Sun, 6 Sep 2020 11:11:35 +0000

Hi,

> -----Original Message-----
> From: curl-library <curl-library-bounces_at_cool.haxx.se> On Behalf Of Daniel
> Stenberg via curl-library
> Sent: Saturday, 5 September 2020 15:16
> To: libcurl hacking <curl-library_at_cool.haxx.se>
> Cc: Daniel Stenberg <daniel_at_haxx.se>
> Subject: time to remove Visual Studio project files?
>
> I'd like to get a better feel for the need to ship the visual studio project files we
> currently host in the project/ directory.
>
> Do you use them? Why isn't generating these files using cmake good enough?

I'm currently using them (slightly patched). nmake/winbuild is currently not an option as it lacks a lot of options. The main reasons for not using CMake are/were, in order of importance:

1) It wasn't possible to set the target Windows for a long time
This is now fixed with https://github.com/curl/curl/commit/295ea5f641cb3d6ca44bb949e63bd1070835a872.

2) HTTP_ONLY leaves SMB enabled
See https://github.com/curl/curl/pull/3770 for the reverted fix. It could probably be revived now that https://github.com/curl/curl/commit/f504f18641ebedde8e8b3805472267efa9a9b528 has landed.

3) Tests passing with VS project-built curl were failing with CMake-built curl in configurations important to me
This should be fixed with https://github.com/curl/curl/commit/f504f18641ebedde8e8b3805472267efa9a9b528. I still have to verify if all tests in all configurations are passing now though.

4) It still outputs a big fat warning on every CMake run that it's poorly maintained, so it just feels like it's experimental and not the recommended way to build

5) It's easy to automatically patch the XML project files to compile with all warnings enabled (/Wall, corresponds to clang's -Weverything) and only the expected warnings disabled as they are per Visual Studio version
I could live with /W4 though.

I'm happy to check if we can migrate to CMake, but I'm not sure if a week will be enough for a final decision. It might take two, or maybe even longer if further fixes to curl's CMake system are needed.

Marcel

-------------------------------------------------------------------
Unsubscribe: https://cool.haxx.se/list/listinfo/curl-library
Etiquette: https://curl.haxx.se/mail/etiquette.html
Received on 2020-09-06