Menu ▾ ▴

#1064 libcurl CMake Windows

closed
Daniel Stenberg
compile or build problem (191)
5
2013-11-08
2011-11-16
Anonymous
No

When building libcurl 7.23.0 on Windows with Visual Studio 10 (SP1) with CMake generated solution files and ENABLE_IPV6 on, I get the following linking error:
hostip6.obj : error LNK2019: unresolved external symbol Curl_getaddrinfo_ex referenced in function Curl_getaddrinfo
By manually enabling HAVE_GETADDRINFO in curl_config.h, I can work around the linker error, but this should be detected by the cmake scripts. I think the proper fix would be to remove the set(HAVE_GETADDRINFO 0) line from the WindowsCache.cmake and update the HAVE_GETADDRINFO test in CurlTests.c to #include <ws2tcpip.h> on Windows. Thanks.

Discussion

  • Daniel Stenberg

    Daniel Stenberg - 2011-11-16

    I don't mean to be rude, but "you think" so? Can't you just try it out, make it work and then present us with the necessary patch?

    I personally have no windows machines to develop on so I'd rather avoid doing blind changes someone just thinks might be right.

     

  • Daniel Stenberg

    Daniel Stenberg - 2011-12-05

    Thanks for reporting this issue and helping us improve curl and libcurl.

    We're awaiting feedback in this issue. Due to this, I have set the state of this issue to pending and it will automatically get closed later on unless we get further info.

    Please consider answering the outstanding questions or providing the missing info so that we can proceed to resolve this issue!

     

  • Daniel Stenberg

    Daniel Stenberg - 2011-12-05
    • milestone: --> bad_behaviour
    • status: open --> pending
     

  • Daniel Stenberg

    Daniel Stenberg - 2012-02-23
    • status: pending --> closed
     

  • Z98

    Z98 - 2013-11-08

    Patch to fix linking problem when building libcurl on Windows with IPv6 support.

     
    windowsbuild.diff

  • Daniel Stenberg

    Daniel Stenberg - 2013-11-08

    pretty please do not submit things in bugs that have been closed and dead for years. I've merged this now but PLEASE take patches to the mailing list and if is a bug, submit a new bug report!