[W32] Testsuite does not build in 7.28.1 anymore
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 `make check' log is attached. It worked in 7.28.0 (or, at least, 7.27.0). Tried rebuilding everything with --disable-shared, didn't help. -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.11 (MingW32) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQEcBAEBAgAGBQJQ+jTvAAoJEOs4Jb6SI2CwKbEH/16oLLXPGpgeCcJiHOXgnAqK yR4numMCwDX9fegKV9hjD5AcI1Jw8VOH7OUuEZV6NMoHx26PPkXUHGCAQyi6HyDi Si7MfEzrholUtj2C0w3MZ0B5EA0W4ey/NPZMayPFlCPoKgH8M8+qw30I7mKw3jUA HgumQF1X5FVt3ryTrhDxWlqwVcNbVQJcaVfNyI9Q1d2OGE+z/ZyVlS1n1rJ7PfVh 4aqq7pz61DbwV4RiBQ7ETM26voUYVTrBYsxNU7G6eb4Vkp43QYUQc3dT3tMKovza gbf8NcdPvkNkowvP5I+AE/TIOckUhSJVLdSg0JXJZ07fLN4JWHYoaQIGh7DxuJU= =KGez -END PGP SIGNATURE- checklog.xz Description: Binary data --- List admin: http://cool.haxx.se/list/listinfo/curl-library Etiquette: http://curl.haxx.se/mail/etiquette.html
Re: [PATCH] Addition of trailer headers in HTTP requests generated by libcurl
On 17 Jan 2013, at 12:35, Chrysovaladis Datsios wrote: Hi, this is a patch for the feature request as described in http://sourceforge.net/p/curl/feature-requests/67/ the feature is a standard in http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.40 I welcome this improvement but I agree with Dan Fandrich that it would be more useful if implemented as a callback. My use cases for request trailers involve signing operations on the message body where it's expensive to buffer the whole request before starting to send the body… especially as parsing the trailer would be optional. Example case: Chunked PUT request with Trailer headers. PUT /myfile HTTP/1.1 Host: 10.8.60.209 Accept: / Transfer-Encoding: chunked Trailer: mytrailerheader Expect: 100-continue 53 first chunk of data.. 53 .second chunk of data. 0 mytrailerheader: myvalue Why is the client sending Expect: 100-continue here? As an aside, the Accept: header also looks invalid. -- Tim Bannister – is...@jellybaby.net smime.p7s Description: S/MIME cryptographic signature --- List admin: http://cool.haxx.se/list/listinfo/curl-library Etiquette: http://curl.haxx.se/mail/etiquette.html
Re: [HTTP][POST] file upload speed is too low.
On Fri, 18 Jan 2013, Sang Young, Ryan, Lee wrote: Now i'm try to upload the big size file(about 85MB) from embedded system to cloud via HTTP Post. which libcurl version are you using? What tool/library are you comparing with that does a similar operation so that you can tell that it truly is libcurl that's the culprit here? Is my code not optimized?? If your media or file system is slow, then nope. If they're not, then the problem should be elsewhere... -- / daniel.haxx.se --- List admin: http://cool.haxx.se/list/listinfo/curl-library Etiquette: http://curl.haxx.se/mail/etiquette.html
Re: curl_easy_recv and disconnections
On Fri, 18 Jan 2013, Craig Davison wrote: I had a look at the libcurl code. curl_easy_recv() calls easy_connection(). easy_connection() calls Curl_getconnectinfo(). If Curl_getconnectinfo() sees that the server has shut down the connection, it returns CURL_SOCKET_BAD, and easy_connection() returns CURLE_UNSUPPORTED_PROTOCOL with the error message Failed to get recent socket. curl_easy_recv() in turn returns the error code from easy_connection(). Ick. You're completely right and I bet I even wrote that code myself but have just completely forgotten about that detail since. Thanks for digging it up, I'm adding this detail to the man pages for curl_easy_send() and curl_easy_recv() right now! CURLE_OK/n=0 seems to happen when my connection does not use SSL or a proxy, and the function curl_easy_recv() uses to read is Curl_recv_plain. In that case, the n=0 comes directly from recv(). curl_easy_recv will return CURLE_AGAIN if reading would block. This is documented and working properly. So it seems that both CURLE_UNSUPPORTED_PROTOCOL and CURLE_OK/n=0 indicate the connection has been shut down. I think this needs to be mentioned in the documentation. Indeed. I'm adding that too now. Will push in a short while. But (IMO) I think ideally curl_easy_recv should never return 0, and in any case return a new error code like CURLE_PEER_DISCONNECTED if the connection is closed. The same would go for curl_easy_send. Probably, yes. But changing that now will introduce a _third_ way for applications that want to work with existing libraries too! I think I then prefer to simply document the existing behavior... -- / daniel.haxx.se --- List admin: http://cool.haxx.se/list/listinfo/curl-library Etiquette: http://curl.haxx.se/mail/etiquette.html
Re: [PATCH] Addition of trailer headers in HTTP requests generated by libcurl
On Sat, 19 Jan 2013, Tim Bannister wrote: I welcome this improvement but I agree with Dan Fandrich that it would be more useful if implemented as a callback. My use cases for request trailers involve signing operations on the message body where it's expensive to buffer the whole request before starting to send the body… especially as parsing the trailer would be optional. I too agree with the idea of having this as a callback instead to allow applications to provide the data during or even post transfer. Seems to match the protocol idea better! -- / daniel.haxx.se--- List admin: http://cool.haxx.se/list/listinfo/curl-library Etiquette: http://curl.haxx.se/mail/etiquette.html
Re: [curl:bugs] #1171 [PATCH] support upload file with ',' or '; ' in the filename
On Sat, 19 Jan 2013, Ulion wrote: the double-quote in filename is not correctly supported by libcurl. Yes, you're right that's a separate bug. But is the presence of that bug really a good reason to not do the right thing in the curl application? If (when?) we fix the problem in libcurl we shouldn't have to change curl again, should we? I'm just seeing (and liking) this opportunity to fix the file name thing in curl for formposting and while we're going at it I think fixing it all the way seems like a good idea! -- / daniel.haxx.se --- List admin: http://cool.haxx.se/list/listinfo/curl-library Etiquette: http://curl.haxx.se/mail/etiquette.html
[vms] Fixes to get vms builds working again.
I finally got curl to build correctly on ALPHA, VAX, IA64, using both GNV (GNU on VMS tool chain) and using the native VMS tools. curl/packages/vms/config_h.com is a new file that generates a config.h file based on the curl_config.h.in file and a quick scan of the configure script. This is actually a generic procedure that is shared with other VMS packages. The existing pre-built config-vms.h had over 100 entries that were not correct and in some cases conflicted with the build options available in the build_vms.com. curl/package/vms/generate_config_vms_h_curl.com is a helper procedure to the config_h.com. It covers the cases that the generic config_h.com is not able to figure out, and accepts input from the build_vms.com procedure. curl/packages/vms/build_curlbuild_h.com is a new file to generate the curlbuild.h file that Curl is now using when it is using a curl_config.h file. curl/lib/post-config-vms.h is a new file that is needed to provide VMS specific definitions, and most of them need to be set before the system header files are included. curl_setup_h.gdiff is a patch to add the post-config-vms.h header to the build. curl_setup_once_h.gdiff needed some VMS specific code removed that conflicted with curl/lib/post-config-vms.h build_vms_com.gdiff.gz is a patch to the VMS build procedure so that it works. 1. Fixed to link in the correct HP ssl library. 2. Fixed to detect if HP Kerberos is installed. 3. Fixed to detect if HP LDAP is installed. 4. Fixed to detect if gnv$libzshr is installed. 5. Simplified the input parameter parsing to not use a loop. 6. Warn that 64 bit pointer option support is not complete in comments. 7. Default to IEEE floating if platform supports it so resulting libcurl will be compatible with other open source projects on VMS. 8. Default to LARGEFILE if platform supports it. 9. Default to enable SSL, LDAP, Kerberos, libz if the libraries are present. 10. Build with exact case global symbols for libcurl. 11. Generate linker option file needed. 12. Compiler list option only commonly needed items. 13. fulllist option for those who really want it. 14. Create debug symbol file on Alpha, IA64. getenv_c.gdiff is a patch to remove VMS specific code that is not needed as it is handled in post-config-vms.h netrc_c.gdiff is a patch to remove VMS specific code that is not needed as it is handled in post-config-vms.h tool_homedir_c.gdiff is a patch to remove VMS specific code that is not needed as it is handled in post-config-vms.h tool_vms_h.gdiff is a patch to make vms_show a global variable. tool_setup_h.gdiff is a patch to add tool_vms_h to it. tool_operate_c.gdiff is a patch to set the vms_show global variable. tool_main_c.gdiff needs to use the vms_show global variable. After these changes are accepted, the following files are no longer used by VMS: curl/lib/config-vms.h in curl/packages/vms/ : hpssl_alpha.opt hpssl_ia64.opthpssl_vax.opt ldap.opt openssl_alpha.optopenssl_ia64.opt openssl_ssl_alpha.opt openssl_ssl_ia64.opt openssl_ssl_vax.opt openssl_vax.opt After this is resolved, I can start submitting the files for VMS to use the GNV based build. Regards, -John wb8...@qsl.net Personal Opinion Only build_curlbuild_h.com.gz Description: application/gzip build_vms_com.gdiff.gz Description: application/gzip config_h.com.gz Description: application/gzip --- /src_root/curl/lib/curl_setup.h Tue Jan 8 21:00:07 2013 +++ /vms_root/curl/lib/curl_setup.h Tue Jan 8 07:09:31 2013 @@ -40,6 +40,11 @@ #include curl_config.h +/* VMS needs some helper routines/definitions */ +#ifdef __VMS +# include post-config-vms.h +#endif + #else /* HAVE_CONFIG_H */ #ifdef _WIN32_WCE @@ -645,7 +650,9 @@ * Include macros and defines that should only be processed once. */ +#ifndef HEADER_CURL_SETUP_ONCE_H #include curl_setup_once.h +#endif /* * Definition of our NOP statement Object-like macro --- /src_root/curl/lib/getenv.c Tue Jan 8 21:00:07 2013 +++ /vms_root/curl/lib/getenv.c Sat Jan 19 14:39:04 2013 @@ -22,10 +22,6 @@ #include curl_setup.h -#ifdef __VMS -#include unixlib.h -#endif - #include curl/curl.h #include curl_memory.h @@ -46,10 +42,6 @@ return (env[0] != '\0')?strdup(env):NULL; #else char *env = getenv(variable); -#ifdef __VMS - if(env strcmp(HOME,variable) == 0) -env = decc_translate_vms(env); -#endif return (env env[0])?strdup(env):NULL; #endif #endif --- /src_root/curl/lib/netrc.c Tue Jan 8 21:00:07 2013 +++ /vms_root/curl/lib/netrc.c Sat Jan 19 14:41:21 2013 @@ -25,9 +25,6 @@ #ifdef HAVE_PWD_H #include pwd.h #endif -#ifdef __VMS -#include unixlib.h -#endif #include curl/curl.h #include netrc.h @@ -83,11 +80,7 @@ struct passwd *pw; pw= getpwuid(geteuid()); if(pw) { -#ifdef __VMS -home = decc_translate_vms(pw-pw_dir); -#else home =