curl-library
unsubscribe
Date: Tue, 30 Mar 2010 07:59:53 -0400
Ed Anderson
Better days are coming. I have it on good word...
From: curl-library-request_at_cool.haxx.se
Subject: curl-library Digest, Vol 55, Issue 67
To: curl-library_at_cool.haxx.se
Date: Mon, 29 Mar 2010 06:49:43 +0200
Send curl-library mailing list submissions to
curl-library_at_cool.haxx.se
To subscribe or unsubscribe via the World Wide Web, visit
http://cool.haxx.se/cgi-bin/mailman/listinfo/curl-library
or, via email, send a message with subject or body 'help' to
curl-library-request_at_cool.haxx.se
You can reach the person managing the list at
curl-library-owner_at_cool.haxx.se
When replying, please edit your Subject line so it is more specific
than "Re: Contents of curl-library digest..."
--Forwarded Message Attachment--
From: daniel_at_haxx.se
To: curl-library_at_cool.haxx.se
Date: Sun, 28 Mar 2010 22:41:02 +0200
Subject: Re: Getting and setting cookie list from structs instead of netscape formated strings
On Sun, 28 Mar 2010, Claes Jakobsson wrote:
> This gist, http://gist.github.com/346813, is a first patch for it - needs
> some fixing tho and testing.
Great, thanks for working on this! I'm generally in favour of adding this
feature, and I have some feedback on the current patch (in a somewhat random
order):
1. The multi-debugcallback.c change seems unrelated.
2. The bool type should be around by including 'setup.h' which cookie.c
already does. We cannot unconditionally use stdbool.h for that, as that's a C99ism.
3. Regarding the two new public functions I think we need some docs for them
too, to fully be able to evaluate and understand them. For example, the
curl_cookie struct can point to a lot of data, how is that data allocated
and freed? Perhaps libcurl should strdup() the strings passed to it within
the struct?
4. Related to (3), we need some test cases added to verify this new
functionality.
5. There's an $Id$ line reintroduced in getinfo.c that you can skip.
6. Regarding the struct curl_sslist definition. It seems a bit strange to
me to have 'type' in each struct and not in a top-level "header struct"
since surely all nodes within the linked list will have the same type?
Maybe it should work more like the 'struct curl_certinfo' instead and have
some information in a top struct, which itself points to the actual list.
Even for the case when an application creates the data and passes it to
libcurl it makes little sense with a 'type' in each struct.
7. The CURLINFO_SSLIST type isn't necessary. The existing CURLINFO_SLIST is
already (ab)used for generic struct data pointer returns so it would be
better to just create an alias like CURLINFO_STRUCTPOINT and use instead.
-- / daniel.haxx.se --Forwarded Message Attachment-- From: daniel_at_haxx.se To: tetetest_at_rambler.ru; curl-library_at_cool.haxx.se Date: Sun, 28 Mar 2010 22:48:02 +0200 Subject: Re: Some fixes for cmake build. On Mon, 29 Mar 2010, Tetetest wrote: > Recently I tried to build the latest stable version of curl (7.20.0) with > using cmake. To my surprise, the build on windows succeeded without any > issues. Then I tried the cmake build on OpenSolaris, and the build failed. That wasn't any surprise to us since we already knew it was broken but nobody stepped up to fix it - until just a week ago! Can you please see how your changes work and make sense against the current git master repo as Bill Hoffman pushed a bunch of changes that fixed the build, and I'm not into cmake enough to judge or merge! -- / daniel.haxx.se --Forwarded Message Attachment-- From: daniel_at_haxx.se To: curl-library_at_cool.haxx.se Date: Sun, 28 Mar 2010 23:01:47 +0200 Subject: Re: [testcurl.pl] Allow running ./tests/testcurl.pl from within git repo. On Sat, 27 Mar 2010, Ben Greear wrote: > My first instinct was to run the test script within the checked out > repository. This small change to the script allows that to work as > expected. Applied! -- / daniel.haxx.se --Forwarded Message Attachment-- From: daniel_at_haxx.se To: curl-library_at_cool.haxx.se Date: Sun, 28 Mar 2010 23:50:51 +0200 Subject: Re: PATCH: pop3: Get message listing if none specified in URL. On Fri, 26 Mar 2010, Ben Greear wrote: > The attached patch treats empty message ID as a request for LIST (list of > pop3 message IDs). User's code could then parse this and download > individual messages as desired. Applied! -- / daniel.haxx.se --Forwarded Message Attachment-- From: yangsita_at_gmail.com To: tetetest_at_rambler.ru; curl-library_at_cool.haxx.se Date: Mon, 29 Mar 2010 03:28:19 +0200 Subject: Re: Some fixes for cmake build. 2010/3/28, Tetetest wrote: > After some investigation I discovered the following: > 1. CMake build uses 'curlbuild.h' file that comes with the > distribution. This means that, in essence, curl build is just the > same as non-configure builds. CMake build scripts will need to be adjusted to overwrite distributed curlbuild.h, otherwise CMake builds might be a do-nothing as you have discovered. > So I fixed the cmake build so that it generates a correct curlbuild.h. > I also had to fix "curl/curl.h" file, because its original way of > including "curlbuild.h" does not work with out-of-source build > which is used by cmake. > diff -Naur curl-7.20.0-old/include/curl/curl.h curl-7.20.0/include/curl/curl.h > --- curl-7.20.0-old/include/curl/curl.h 2010-03-27 15:26:15.703125000 +0300 > +++ curl-7.20.0/include/curl/curl.h 2010-03-28 00:45:52.531250000 +0300 > @@ -32,7 +32,7 @@ > */ > > #include "curlver.h" /* libcurl version defines */ > -#include "curlbuild.h" /* libcurl build definitions */ > +#include <curl/curlbuild.h> /* libcurl build definitions */ > #include "curlrules.h" /* libcurl rules enforcement */ That change is not acceptable. Some few libcurl versions were released with an include line for curlbuild.h like the one that you propose now. Due to the problem it represented for some platforms it was fixed back to a 'traditional' include which did not have the leading 'curl' path and makefiles for configure and non-configure systems adjusted accordingly. So, CMake build scripts will need to be adjusted to be able to use curl.h 'as is', it is the responsibility of CMake scripts to generate appropriate makefiles which define the proper include paths. -- -=[Yang]=- --Forwarded Message Attachment-- From: julien.chaffraix_at_gmail.com To: curl-library_at_cool.haxx.se Date: Sun, 28 Mar 2010 18:53:42 -0700 Subject: Re: [testcurl.pl] Allow running ./tests/testcurl.pl from within git repo. On Mar 28, 2010 2:04 PM, "Daniel Stenberg" <daniel_at_haxx.se> wrote: On Sat, 27 Mar 2010, Ben Greear wrote: > My first instinct was to run the test script within the ch... Applied! -- / daniel.haxx.se ------------------------------------------------------------------- List admin: http://cool.haxx.se... --Forwarded Message Attachment-- From: bill.hoffman_at_kitware.com To: curl-library_at_cool.haxx.se Date: Sun, 28 Mar 2010 22:13:43 -0400 Subject: Re: Some fixes for cmake build. Yang Tse wrote: > That change is not acceptable. Some few libcurl versions were released > with an include line for curlbuild.h like the one that you propose > now. Due to the problem it represented for some platforms it was fixed > back to a 'traditional' include which did not have the leading 'curl' > path and makefiles for configure and non-configure systems adjusted > accordingly. > > So, CMake build scripts will need to be adjusted to be able to use > curl.h 'as is', it is the responsibility of CMake scripts to generate > appropriate makefiles which define the proper include paths. > > Try the git master of curl and see if it works for you. I have not tested solaris, but I did several fixes for linux, and these should address the problems you are reporting. I did not change any of the code, just CMake files. -Bill -- Bill Hoffman Kitware, Inc. 28 Corporate Drive Clifton Park, NY 12065 bill.hoffman_at_kitware.com http://www.kitware.com 518 881-4905 (Direct) 518 371-3971 x105 Fax (518) 371-4573 --Forwarded Message Attachment-- From: bnagasekhar_at_gmail.com To: curl-library_at_cool.haxx.se; libssh2-devel_at_cool.haxx.se Date: Mon, 29 Mar 2010 09:48:47 +0530 Subject: Character conversion for sftp Hi, I am using libcurl 7.18.1 to do ftp and sftp (libssh2-1.1) on z/OS. By defining the CURL_ICONV_CODESET_OF_HOST and CURL_ICONV_CODESET_OF_NETWORK, ftp is done correctly. But, I still see the problem with sftp. I see that the "ftp_readresp" does data conversion using Curl_convert_from_network and "Curl_nbftpsendf" does it through Curl_convert_to_network (/lib/ftp.c). Similar conversion is not done for sftp's Curl_sftp_recv and Curl_sftp_send (src/curl/curl-7.18.1/lib/ssh.c). Could someone advice me how the character conversion can be handled for sftp? Thanks in advance, Nagasekhar --Forwarded Message Attachment-- From: souvik.datta_at_wipro.com CC: curl-library_at_cool.haxx.se To: kdudka_at_redhat.com Date: Mon, 29 Mar 2010 10:19:29 +0530 Subject: RE: libCURL crash in __libc_res_nquery () I am using following timeouts in my application: CURLOPT_CONNECTTIMEOUT CURLOPT_LOW_SPEED_LIMIT CURLOPT_LOW_SPEED_TIME There is no custom SIGNAL handler in my application for time outs. Pl. find the lib/curl_config.h file as attached. Regards, Souvik -----Original Message----- From: Kamil Dudka [mailto:kdudka_at_redhat.com] Sent: Sunday, March 28, 2010 6:27 PM To: Souvik Datta (WT01 - PES-Mobile-Auto-CE Practice) Cc: curl-library_at_cool.haxx.se Subject: Re: libCURL crash in __libc_res_nquery () On Sunday 28 of March 2010 14:50:49 Kamil Dudka wrote: > On Sunday 28 of March 2010 14:36:38 Kamil Dudka wrote: > > On Sunday 28 of March 2010 10:21:20 souvik.datta_at_wipro.com wrote: > > > Thanks for your reply. I have added the patch as mentioned in the link > > > , but this time I find that it is failing (very frequently ) by > > > throwing the print "Alarm clock" on the console. > > > > AFAIK there is no such message in libcurl sources. You will need to > > check your appliction. > > The message may be printed by the shell. Does your application set its own > handler for SIGALRM? If so, it clashes with the libcurl's handler. > libcurl removes the original handler and the signal comes unhadled, causing > the application to crash. Could you please attach lib/curl_config.h from your build of libcurl? Thanks in advance! > > Kamil Please do not print this email unless it is absolutely necessary. The information contained in this electronic message and any attachments to this message are intended for the exclusive use of the addressee(s) and may contain proprietary, confidential or privileged information. If you are not the intended recipient, you should not disseminate, distribute or copy this e-mail. Please notify the sender immediately and destroy all copies of this message and any attachments. WARNING: Computer viruses can be transmitted via email. The recipient should check this email and any attachments for the presence of viruses. The company accepts no liability for any damage caused by any virus transmitted by this email. www.wipro.com _________________________________________________________________ Hotmail: Trusted email with Microsoft’s powerful SPAM protection. http://clk.atdmt.com/GBL/go/210850552/direct/01/
-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette: http://curl.haxx.se/mail/etiquette.html
Received on 2010-03-30