Guenter in gmane.comp.web.curl.library (Thu, 01 Nov 2012 01:29:57
+0100):
>Hi Jan,
>Am 30.10.2012 23:18, schrieb Jan Ehrhardt:
>> Vanished once again under the radar? Or did I miss a patch?
>no and no :-)
>I was just otherwise busy, but good you pinged again! Just pushed a
>simple patch:
>https://github.com/bagder/curl/commit/c81eb7e226be6c8ead3865592f3f2487fab4caeb
>I hope that fixes the issue; I did also remove the debug suffix since
>none of the other depndent libs have this, so not sure why libssh2
>should be with debug code; it seems that the whole makefile lacks of any
>debug settings for the dependencies, and probably it would make sense to
>support this, but I'm currently not up add this - patches welcome if
>someone needs it!
>Would be cool if you could test that now everthing's fine with the
>makefile - either with next daily snapshot or directly from git.

Tested with the November 1 daily snapshot. Works flawlesly with a
/winbuild commandline like:

nmake /f makefile.vc mode=static VC=9 MACHINE=x86
        WITH_DEVEL=../../win32build USE_IDN=yes WITH_WINSSL=static
        WITH_SSH2=static WITH_ZLIB=static

ldap.obj was created with this commandline and ldap support was present.

C:\php-sdk\curl-src\builds\libcurl-release-static-zlib-static-ssh2-static-ipv6-sspi-spnego-winssl\bin>curl
--version
curl 7.28.1-20121101 (i386-pc-win32) libcurl/7.28.1-20121101 WinSSL
        zlib/1.2.7 WinIDN libssh2/1.4.2
Protocols: dict file ftp ftps gopher http https imap imaps ldap pop3
        pop3s rtsp scp sftp smtp smtps telnet tftp
Features: AsynchDNS GSS-Negotiate IDN IPv6 Largefile NTLM SPNEGO SSL
        SSPI libz

>Then I recall that there was some issue with the winssl code too which
>you mentioned before - is that still an issue? I think there was
>something where we should block something in the makefile so that its
>not possible to specify more than one ssl lib ...?

In previous release there was a conflict between SSL and SSH2. You
couldn't load them both, but this was apparently solved.

>Finally two other things:
>1) we have also nmake makefiles in lib and src (which are use from main
>makefile); would be cool if you could take a look at these too (but in
>git are only Makefile.vc6, the other *.vc8 and *.vc9 are generated by
>the release script, so you have to test this with a daily snapshot if
>you have VC8 or 9); also one line in Makefile.vc6 I just found very strange:
>CFLAGSSSH2 = /DUSE_LIBSSH2 /DCURL_DISABLE_LDAP /DHAVE_LIBSSH2
>/DHAVE_LIBSSH2_H /DLIBSSH2_WIN32 /DLIBSSH2_LIBRARY /I
>"$(LIBSSH2_PATH)/include"

Tried to compile with makefile.vc9. I had to change inc32 in the
makefile to include to make openssl to be included.
C:\php-sdk\curl-src\lib>nmake /f makefile.vc9 CFG=release-ssl-ssh2-zlib
generated the libcurl.lib

But then I couldn't get src/makefile.vc9 to compile. Linker errors and
the like. On inspection of the makefile.vc9 I did not even find any
reference to SSH2 !?

>why the heck do we disable LDAP support when we build with ssh2 support???

Don't know.

Jan

-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette: http://curl.haxx.se/mail/etiquette.html
Received on 2012-11-02