Re: [patch] Files for building VMS PCSI kits.

2013-08-08 Thread Yang Tse
On 8/8/13, John E. Malmberg wb8...@qsl.net wrote: I modified make_pcsi_curl_kit_name.com and compare_curl_source.com to remove the references to CVS. The first patch has the missing and changed files. The second patch fixes the file protections back to what they were before the first

Re: [bagder/curl] 6b2770: configure: don't error out on variable confusions, ...

2013-08-05 Thread Yang Tse
On Mon, Aug 5, 2013, Daniel Stenberg wrote: I don't think we should be strict pedants that point out _possible_ flaws and errors out on them. I'd rather have us point out the possible problems, clearly warn about them and then let the script continue and possibly still work. Or not. The user

Re: [bagder/curl] 6b2770: configure: don't error out on variable confusions, ...

2013-08-04 Thread Yang Tse
On Sun, Aug 4, 2013 at 10:28 AM, GitHub nore...@github.com wrote: Commit: 6b27703b5f525eccdc0a8409f51de8595c75132a https://github.com/bagder/curl/commit/6b27703b5f525eccdc0a8409f51de8595c75132a Author: Daniel Stenberg dan...@haxx.se Date: 2013-08-04 (Sun, 04 Aug 2013)

curl_xferinfo_callback

2013-08-01 Thread Yang Tse
Hi, The curl progress-bar display frequency limiting introduced with commit 1691a31c and subsequent ones, still lacks a way to ensure that the very last time the progress callback is called it displays updated data of finished transfer. Since this release introduces curl_xferinfo_callback, it

Re: bug #1255 claimed to not be done yet

2013-07-30 Thread Yang Tse
Using... +#elif (defined(__MAC_OS_X_VERSION_MAX_ALLOWED) \ + (__MAC_OS_X_VERSION_MAX_ALLOWED = 1040)) || \ + (defined(__IPHONE_OS_VERSION_MAX_ALLOWED) \ + (__IPHONE_OS_VERSION_MAX_ALLOWED = 2)) Instead of ... +#elif __MAC_OS_X_VERSION_MAX_ALLOWED = 1040 || \

Re: patch - Initial set of files for Curl on VMS using GNV toolkit.

2013-07-25 Thread Yang Tse
On Mon, Jul 15, 2013, John E. Malmberg wrote: [...] set off files for building Curl on OpenVMS using the GNV (GNu on VMS) tool kit, and the port of Bash 4.2.45 for OpenVMS that will be released shortly. Merged as commit e277e20a6d Thanks, -- -=[Yang]=-

Re: Fwd: [bagder/curl] 8ec2cb5544 WIN32 MemoryTracking

2013-07-19 Thread Yang Tse
On Thu, Jul 18, Yang Tse wrote: On Thu, Jul 18, Daniel Stenberg wrote: But since I reverted that due to a problem that was reported with some other windows compiler, is there a chance that this or pieces of this is (also) compiler dependent and not OS dependent? Yes it is possible. I'll

Fwd: [bagder/curl] 8ec2cb5544 WIN32 MemoryTracking

2013-07-18 Thread Yang Tse
Forwarding to the list, given I sent it privately unintentionally... -- Forwarded message -- From: Yang Tse yangs...@gmail.com Date: Thu, Jul 18, 2013 at 8:33 PM Subject: Re: [bagder/curl] 8ec2cb5544 WIN32 MemoryTracking To: Daniel Stenberg dan...@haxx.se On Tue, May 7, 2013

Re: Fwd: [bagder/curl] 8ec2cb5544 WIN32 MemoryTracking

2013-07-18 Thread Yang Tse
On Thu, Jul 18, Daniel Stenberg wrote: On Thu, 18 Jul 2013, Yang Tse wrote: Ok to reinstate? Indeed OK by me. Thanks a lot for the clarifying message and explanation. Ok, I'll push it then now to have it included in next snapshot. But since I reverted that due to a problem

Re: make distclean broken?

2013-07-17 Thread Yang Tse
Hi, Given that we have to remain as backwards compatible as possible while still trying to allow usage of latest autotool versions, commit 26b0cb6ae2 has been pushed in order to fix the 'distclean' and 'maintainer-clean' target issues triggered when using automake 1.14. For those unaware of

Re: make distclean broken?

2013-07-15 Thread Yang Tse
Nick Zitzmann wrote: Yeah, I'm pretty sure something broke with the upgrade to Automake 1.14. I tried building and then running make distclean on another system running an older version (1.12.x), and it worked. Then I upgraded it to 1.14, and it failed with the same error. Yes, currently,

Re: strdup

2013-07-12 Thread Yang Tse
Does anybody has some hint how to make them compilable, still preserving them for platforms implementing strdup() ? Attached (completely untested) patch should fix the issue. At least, as long as libcurl isn't built as a shared library with symbol hiding enabled. Give it a try, -- -=[Yang]=-

Re: strdup

2013-07-12 Thread Yang Tse
On Fri, Jul 12, 2013 at 12:12 PM, Ryan meok...@yahoo.com wrote: On Jul 12, 2013, at 5:58 PM, Yang Tse yangs...@gmail.com wrote: Does anybody has some hint how to make them compilable, still preserving them for platforms implementing strdup() ? Attached (completely untested) patch should fix

Re: strdup

2013-07-12 Thread Yang Tse
Patrick Monnerat wrote: Your patch is OK for lib1900.c, but not for lib509.c because it uses (strdup)(ptr) However by changing your patch to #define strdup curlx_strdup It compiles fine with that. Hi :-) I don't foresee any problem in libtest programs if your #define was used.

Re: New SSL backend GSKit, certinfo for everyone, etc.

2013-07-12 Thread Yang Tse
Your comments are welcome. Patch looks good to me. -- -=[Yang]=- --- List admin: http://cool.haxx.se/list/listinfo/curl-library Etiquette: http://curl.haxx.se/mail/etiquette.html

Re: [PATCH] configure: explicitly use subdir-objects in automake init

2013-07-08 Thread Yang Tse
On Mon, Jul 8, 2013 at 3:13 PM, Dave Reisner wrote: This avoids a large warning on invocation of ./buildconf on automake 1.14: Commits ecf042ff3c and fe7e3229f8 ensure that configure script is generated without using 'subdir-objects' when using automake versions older than 1.14 resulting in a

Re: Name lookup/DNS resolution mechanism of libcurl

2013-04-09 Thread Yang Tse
Hi Gün, On Wed, Apr 10, 2013, Guenter li...@gknw.net wrote: honestly I'm not happy with that hack, and if someone has something better in mind please speak up! Modify Makefile.m32 in order to make it define one out of three possibilities ENABLE_ARES, ENABLE_SYNC_DNS or ENABLE_THREADS_WIN32.

Re: curl mingw compile and shared v/s static libraries?

2013-04-09 Thread Yang Tse
On Tue, Apr 9, 2013, Ben Greear gree...@candelatech.com wrote: I updated to the latest curl recently and now my mingw build is different. [...] Now, it does not. I notice that it appears to only be able to do shared OR static libraries...I am pretty sure it used to do both. Is this on

Re: [bagder/curl] 3dcc1a: http_negotiate.c: fix several SPNEGO memory handli...

2013-04-04 Thread Yang Tse
http_negotiate.c: In function 'Curl_output_negotiate': http_negotiate.c:336:29: error: expected expression before '{' token http_negotiate.c:342:29: error: expected expression before '{' token Should be fixed now with follow-up commit ed35e1fa https://github.com/bagder/curl/commit/ed35e1fa

Re: BUG: free statement in http_negotiate.c giving heap error

2013-04-03 Thread Yang Tse
On Tue, Mar 26, 2013, Arunav Sanyal arunav.sanya...@gmail.com wrote: Ok line number 308 is the malloc call which occurs immediately after the free statement that i talked about. This malloc is never reached, since the free itself causes a breakpoint after which I cannot step forward Daily

Re: [patch]build_vms.com - Use existing curlbuild.h, parsing fix.

2013-03-20 Thread Yang Tse
On Wed, Mar 20, 2013, John E. Malmberg wb8...@qsl.net wrote: This patch removes building curlbuild.h from the build_vms.com procedure and [...] Tested on IA64/VMS 8.4 and VAX/VMS 7.3 Pushed to github now. I can submit a version with tabs converted to spaces as a different patch. There's an

Re: [PATCH 2/3] library init: documentation update

2013-03-15 Thread Yang Tse
On Fri, Mar 15, 2013, Alexander Klauer alexander.kla...@itwm.fraunhofer.de wrote: This commit updates the documentation of ares_library_init() and ares_library_cleanup() with regard to the new recursive behaviour. It might be better to use Its reference counted initialize/deinitialize

Re: curl_easy_perform() fails with Problem with the SSL CA cert (path? access rights?) after first time calling this routine

2013-03-15 Thread Yang Tse
On Fri, Mar 15, 2013 at 7:59 PM, cnm marketing cnn.market...@gmail.com wrote: error:0506706E:Diffie-Hellman routines:GENERATE_KEY:key size too small 1. When libcurl uses Diffie-Hellman, does libcurl hardcode the Diffie-Hellman key and the length? 2. Does libcurl have an option that allow

Re: curl_easy_perform() fails with Problem with the SSL CA cert (path? access rights?) after first time calling this routine

2013-03-15 Thread Yang Tse
On Fri, Mar 15, 2013 at 9:26 PM, cnm marketing cnn.market...@gmail.com wrote: The problem is in the certificate you are using which does not have a long enough Diffie-Hellman key. [cnm] I don't understand what you are refering to, please give a bit more details. Assuming that the certificate

Re: vms_show hack explained

2013-03-15 Thread Yang Tse
On Sat, Mar 16, 2013, John E. Malmberg wrote: The daily tarball that I used earlier did not have a curlbuild.h file in it, so I had to generate it. You might have been unfortunate back then fetching a badly broken daily snapshot. I've just verified that curl-7.30.0-20130314.tar certainly has

Re: vms_show hack explained

2013-03-14 Thread Yang Tse
On Thu, Mar 14, 2013, John E. Malmberg wb8...@qsl.net wrote: I have reasons for all of them from a technical point of view. But in some cases, it is more a case of what I have time to implement and test. I am currently trying to get a number of vms ported projects up to date and also

Re: VMS fixes needed to allow building

2013-03-13 Thread Yang Tse
Tom and John, Issues mentioned here have been addressed with following commits: https://github.com/bagder/curl/commit/17c088fb45 https://github.com/bagder/curl/commit/8ae9ee0cf2 Daily snapshot (http://curl.haxx.se/snapshots/) of March 14 should include these. Thanks, -- -=[Yang]=-

Re: VMS fixes needed to allow building.

2013-03-13 Thread Yang Tse
On Wed, Mar 13, John E. Malmberg wb8...@qsl.net wrote: I have a git clone in one directory tree now and it has the command procedures needed to build the .h files. I have not tested them. I can not run autoconf on VMS to generate the configure and lib/curl_config_in. [...] I pulled down

Re: VMS fixes needed to allow building

2013-03-13 Thread Yang Tse
On Wed, Mar 13, 2013 at 4:13 PM, Tor Arntsen kspt@gmail.com wrote: [...] Usually the latter is achieved by having extern int vms_show; in a header file and including it in the necessary .c files (and in exactly one of them you put the 'int vms_show=0;') Removed redundant 'extern' now with

Re: VMS fixes needed to allow building.

2013-03-12 Thread Yang Tse
On Tue, Mar 12, 2013, Tom Grace thomas.gr...@amdocs.com wrote: Also I see in TOOL_MAIN.C that variable vms_show is a module static yet , the thread update comments suggest it should be extern, via edit submitted in TOOL_MAIN.H , what do you want to do here? (there is compile warning

Re: VMS fixes needed to allow building.

2013-03-12 Thread Yang Tse
On Tue, Mar 12, 2013, John E. Malmberg wb8...@qsl.net wrote: Not sure what happened, either I missed something in the patch submission or something else went wrong. It definitely needs to be extern or the VMS exit code does not work. I will try to find time to check all the files in my

Re: VMS fixes needed to allow building.

2013-03-12 Thread Yang Tse
On Tue, Mar 12, 2013, Tom Grace thomas.gr...@amdocs.com wrote: CC (opts) DISK$USER1:[TGRACE.CURL.CURL-7_29_0.SRC]TOOL_MAIN.C;1 [...] static int vms_show = 0; ...^ %CC-W-MIXLINKAGE, In this declaration, vms_show is declared with both internal and external linkage. The previous

Re: VMS fixes needed to allow building

2013-03-12 Thread Yang Tse
On Tue, Mar 12, 2013, Tom Grace thomas.gr...@amdocs.com wrote: Also note I also had to perform the following to vms_build.com, 1.Missin slash in symbol CC_FULL_LIST 2.Extra unwanted quotes inside symbol tool_main, causing link fail, since this is main module [...] Would it be

Re: VMS fixes needed to allow building

2013-03-12 Thread Yang Tse
On Wed, Mar 13, 2013, Tom Grace thomas.gr...@amdocs.com wrote: Ok, I understand that declaring it with 'globaldef' instead of 'static' in tool_main.c makes it a single instance variabe that is 'visible' and can be modified from any other source file which happens to include tool_vms.h with its

Re: [Patch] lib/polarssl.c

2013-03-08 Thread Yang Tse
On Fri, Mar 8, 2013, Gisle Vanem gva...@broadpark.no wrote: A typo in lib/polarssl.c? lib/polarsslthreadlock.h doesn't exists here. Patch: [...] Fixed now in master.. Thanks, -- -=[Yang]=- --- List admin:

Re: possible problem with curl-config

2013-02-25 Thread Yang Tse
On Mon, Feb 25, Dennis Clarke dcla...@blastwave.org wrote: $ curl-config --configure '--enable-shared' '--enable-static' '--with-libidn=/usr/local' '--enable-tls-srp' '--with-ssl' 'CC=/opt/solarisstudio12.3/bin/cc' 'CFLAGS=-errfmt=error -erroff=%none -errshort=full -xstrconst -xildoff -m64

Re: Problem building libcurl on AIX platforms

2013-02-22 Thread Yang Tse
On Thu, Feb 21, Rozita Mirzadeh rmirza...@beyondtrust.com wrote: I am trying to build libcurl on several platforms, and on AIX 5.1 and AIX 5.2, I get the following error: [...] multi.c:1021: error: structure has no member named `reqevents' multi.c:1022: error: structure has no member named

Re: Ares: socklen_t problem

2013-02-19 Thread Yang Tse
On Mon, Feb 18, Tor Arntsen t...@spacetec.no wrote: c-ares commit cf7ad9758dd792 includes the following change to ares.h: +CARES_EXTERN const char *ares_inet_ntop(int af, const void *src, char *dst, +socklen_t size); socklen_t in a public header file

Re: Ares: socklen_t problem

2013-02-19 Thread Yang Tse
On Mon, Feb 18, Tor Arntsen t...@spacetec.no wrote: c-ares commit cf7ad9758dd792 includes the following change to ares.h: +CARES_EXTERN const char *ares_inet_ntop(int af, const void *src, char *dst, +socklen_t size); socklen_t in a public header file

curl.h and CURL_EXTERN

2013-02-15 Thread Yang Tse
Hi friends, Please read fully before replying. curl.h at line 80 and surrounding has a preprocessor block which takes care of defining CURL_EXTERN symbol depending on wether other symbols are defined or not. Attached file old_logic.txt shows existing preprocessor block and associated truth

Re: tests/libtest/chkhostname.c

2013-02-08 Thread Yang Tse
On Fri, Feb 8, 2013 at 12:15 PM, Gisle Vanem gva...@broadpark.no wrote: This is AFAICS the only test program in this directory that doesn't call curl_global_init(). This function is needed for Curl_gethostname() to work on Windows at least. I have work in progress in this specific area, but

Re: [bagder/curl] 632e50: msvc: move Makefile.msvc.names into winbuild/

2013-02-07 Thread Yang Tse
On Wed, Feb 6, 2013 at 11:16 PM, GitHub nore...@github.com wrote: Commit: 632e50ca8d71a309ea3863acac5f1a7f88633e69 https://github.com/bagder/curl/commit/632e50ca8d71a309ea3863acac5f1a7f88633e69 Author: Daniel Stenberg dan...@haxx.se Date: 2013-02-06 (Wed, 06 Feb 2013)

Re: [bagder/curl] 0e66d5: vc: remove explicit MSVC6 IDE project file and doc...

2013-02-07 Thread Yang Tse
On Wed, Feb 6, 2013 at 11:30 PM, GitHub nore...@github.com wrote: Commit: 0e66d5878edc3d7ffc445116d194b58bbc7504b9 https://github.com/bagder/curl/commit/0e66d5878edc3d7ffc445116d194b58bbc7504b9 Author: Daniel Stenberg dan...@haxx.se Date: 2013-02-06 (Wed, 06 Feb 2013)

Re: [bagder/curl] 632e50: msvc: move Makefile.msvc.names into winbuild/

2013-02-07 Thread Yang Tse
On Thu, Feb 7, 2013, Daniel Stenberg wrote: No, but the name is 'winbuild' and it is windows related. I realize it isn't optimal but then neither is having two build systems for Windows (in different places) - I *really* would like to see them merge to a single one at some point. Two? :-)

Re: [bagder/curl] 0e66d5: vc: remove explicit MSVC6 IDE project file and doc...

2013-02-07 Thread Yang Tse
On Thu, Feb 7, 2013, Daniel Stenberg wrote: Okay. I grepped around for references to vc6curl.dsw in files but I found none. I'm listening and if you really think it should be left where it was then I'll revert. Also, it wouldn't mean that we drop support for building with that environment.

Re: wc test in c7b6e43e fails on sysv

2013-01-30 Thread Yang Tse
On Wed, Jan 30, 2013, Tor Arntsen wrote: The test for the existence of 'wc' fails on at least IRIX and Solaris: [...] IRIX, Solaris: $ echo h h h h|wc -w 4 Commit 38131d41 should fix this issue, as long as leading whitespace are spaces and not tabs. It also sets

Re: wc test in c7b6e43e fails on sysv

2013-01-30 Thread Yang Tse
On Wed, Jan 30, 2013, Yang Tse wrote: Commit 38131d41 should fix this issue, as long as leading whitespace are spaces and not tabs. It also sets internationalization behavior variables to 'C'. I'm pushing a follow-up commit for this to take into account that leading white space may be any

Re: b1826d81f breaks configure

2013-01-25 Thread Yang Tse
On Fri, Jan 25, 2013, Tor Arntsen t...@spacetec.no wrote: b1826d81f breaks configure, see today's autobuilds page. It seems that the problem is a bashism that is not compatible with sh/ksh, I can reproduce it in a shell: [...] and that is the type of error that causes all those

Re: [PATCH] always multi v5

2013-01-17 Thread Yang Tse
Commit https://github.com/bagder/curl/commit/9fd88abb pushed now for autobuild testing, solves the problems shown locally with test cases 1059 and 1316. The patch has been tested without 'always multi' on top of commit 4ed6b07a, with test suite passing ok. Additionally, patch tested with 'always

Re: [PATCH] always multi v5

2013-01-16 Thread Yang Tse
On Tue, Jan 15, 2013, Daniel Stenberg wrote: I run this and compare the traces and that tells Curl_proxyCONNECT() seems to not be working entirely correct for you, although I haven't been able to pinpoint why/how. [...] To me it looks as if in your case it doesn't properly detect the end of

Re: [PATCH] always multi v5

2013-01-16 Thread Yang Tse
Daniel, Attached patch (--no-prefix against current master) fixes the 1059 issue for me I'm now running full 'always multi' test suite. Later on I'll also apply patch to master without 'always multi' and run test suite. The issue doesn't actually seem an 'always multi' one, but simply a bug that

Re: [PATCH] always multi v5

2013-01-16 Thread Yang Tse
On Wed, Jan 16, 2013, Yang Tse wrote: Attached patch (--no-prefix against current master) fixes the 1059 issue for me I'm now running full 'always multi' test suite. Buarghh. NTLM and SFTP tests fail like mad and others sigsegv :-( I've killed the test suite. Something more involved

Re: [bagder/curl] 16abdd: configure: fix automake 1.13 compatibility

2013-01-15 Thread Yang Tse
On Mon, Jan 14, 2013, Daniel Stenberg wrote: I assume then that we can conclude that bug #1182 can be closed? Dave Reisner has confirmed that the issue is fixed for him, and autobuilds with autoconf 2.57 and newer versions have worked ok. So yes it can be closed as fixed for next release. --

Re: Compilation problems with automake 1.13

2013-01-14 Thread Yang Tse
Hi Dave, Could you verify if if current git libcurl version is capable of running buildconf and building libcurl properly with automake 1.13 version and the build ecosystem on which you have reported having the issues? An alternative would be run buildconf from libcurl snapshot

Re: [bagder/curl] 16abdd: configure: fix automake 1.13 compatibility

2013-01-14 Thread Yang Tse
On Mon, Jan 14, 2013, Daniel Stenberg wrote: I assume then that we can conclude that bug #1182 can be closed? I would prefer to await Dave Reisner's confirmation on issue being fixed, and tomorrow's autobuilds which will show if anything goes south when using old autoconf versions, before

Re: [PATCH] always multi v5

2013-01-14 Thread Yang Tse
On Mon, Jan 14, 2013, Daniel Stenberg wrote: On Fri, 11 Jan 2013, Yang Tse wrote: 00:25:55.571476 == Info: Recv failure: Connection reset by peer 00:25:55.571476 == Info: Connection died, retrying a fresh connect ... it doesn't look like this is truly trying a fresh connect, it seems

Re: [PATCH] always multi v5

2013-01-11 Thread Yang Tse
On Fri, Jan 11, 2013, Daniel Stenberg wrote: Can you first check and see if there's a time descripancy between the trace64 log and the http_server.log ? I can't make sense of the times in there: These are the best kind of timestamps we can get on this kind of 'system', not too bad accuracy

Re: [PATCH] always multi v5

2013-01-10 Thread Yang Tse
Attached file logs_7_tests.tar.gz has test traces for test cases 64, 69, 90, 153, 160, 1079 and 1095. These http tests have in common that http test server is instructed to close connection with 'swsclose', while test case is doing more than one request. Above seven tests needed kiil -9 to stop

Re: USE_ARES in config-win32.h

2013-01-10 Thread Yang Tse
Hi Gisle, On Thu, Jan 10, Gisle Vanem wrote: lib/Makefile.m32 has this: ifdef ARES INCLUDES += -I$(LIBCARES_PATH) CFLAGS += -DUSE_ARES Yes that's a problem with makefiles which don't write a curl_config.h file and expect to fully control build behavior even when a config-platform.h

Re: [Patch] packages/DOS/common.dj

2013-01-10 Thread Yang Tse
On Thu, Jan 10, 2013 Gisle Vanem wrote: gcc on DOS hasn't really supported COFF-debug (-gcoff) on djgpp for a long time. Eli Zaretskii wrote in: [...] So lets drop it too: Pushed. https://github.com/bagder/curl/commit/b35a176c Thanks, -- -=[Yang]=-

Re: [PATCH] always multi v5

2013-01-10 Thread Yang Tse
Daniel, Some tests I've done... Forced libcurl to build using select() instead of poll() to rule out any problem in that area. No joy, exhibits same behavior. Used http test server sws.c rolled back to commit d759a70d and also back to commit 4d3fb91f. But it exhibits same behavior. And, given

Re: [PATCH] always multi v5

2013-01-10 Thread Yang Tse
On Thu, Jan 10, Daniel Stenberg wrote: Lets focus on the single test 64 first. What's the exact test target/OS/version etc you're using to trigger this? Does it fail on all runs? cygwin 1.5.25 on w2k. Yes it fails in all runs. Also, can you show us the log/http_server.log after a run where

Re: [PATCH] always multi v5

2013-01-09 Thread Yang Tse
On Wed, Jan 9, Daniel Stenberg wrote: On Wed, 9 Jan 2013, Yang Tse wrote: Nearly all test cases pass except for: 64, 69, 90, 153, 160, 1079 and 1095. I think I would need details on these in order to do something. I've now run these tests MANY times here and none of them failed even once

Re: [PATCH] build: fix circular header inclusion with other packages

2013-01-08 Thread Yang Tse
Ack, Pasting below updated commit message. I hope it is more clear now... build: fix circular header inclusion with other packages This commit renames lib/setup.h to lib/curl_setup.h and renames lib/setup_once.h to lib/curl_setup_once.h. Removes the need and usage of a header inclusion guard

Re: [bagder/curl] 7a6d8b: configure: update req to 2.59

2013-01-08 Thread Yang Tse
On Tue, Jan 8, 2013 at 11:51 AM, GitHub nore...@github.com wrote: Branch: refs/heads/master Home: https://github.com/bagder/curl Commit: 7a6d8b1b1a8fcc184c36d6b6e741e32250b4bacb https://github.com/bagder/curl/commit/7a6d8b1b1a8fcc184c36d6b6e741e32250b4bacb Author: Daniel

Re: [PATCH] always multi v5

2013-01-08 Thread Yang Tse
On Tue, Jan 8, 2013, Daniel Stenberg wrote: I accidentally posted a version that wasn't the latest rebase yesterday, and with Steve Holme's POP3 TLS fix this morning (that required a little extra merge edit) I decided to post a follow-up version. Nothing else changed. Hi Daniel, I'm trying

Re: [PATCH] always multi v5

2013-01-08 Thread Yang Tse
On Tue, Jan 8, 2013, Daniel Stenberg wrote: Those line endings are a bit painful to handle in patches. Try the attached gzip version and see if it makes anything better! Perfect! Building now... -- -=[Yang]=- --- List admin:

Re: [PATCH] always multi v5

2013-01-08 Thread Yang Tse
Daniel, Patch applies cleanly on top of fea3a3d5. Builds successfully with: Protocols: dict file ftp ftps gopher http https imap imaps pop3 pop3s rtsp scp sftp smtp smtps telnet tftp Features: Debug TrackMemory IDN Largefile NTLM NTLM_WB SSL libz TLS-SRP Nearly all test cases pass except for:

Re: [bagder/curl] 13606b: build: make use of 93 lib/*.c renamed files

2013-01-06 Thread Yang Tse
- Don't revert renaming. Steve Holme - Push revertion commit. Yang Tse - Don't revert renaming. Tor Arntsen - Push revertion commit. Gisle Vanem - Push revertion commit. Dan Fandrich - Push revertion commit. Tom Bishop - Declines giving formal recommendation And... Daniel Stenberg - Push revertion

Re: VMS fixes needed to allow building.

2013-01-06 Thread Yang Tse
Hi John, I noticed your VMS proposed patches time ago. I have some comments/adjustments/suggestions for these. I'll do it soonish. Thanks, -- -=[Yang]=- --- List admin: http://cool.haxx.se/list/listinfo/curl-library Etiquette:

[PATCH] build: fix circular header inclusion with other packages

2013-01-06 Thread Yang Tse
Hi, Attached is a compressed patch with the following commit message: build: fix circular header inclusion with other packages This commit renames lib/setup.h to lib/curl_setup.h and renames lib/setup_once.h to lib/curl_setup_once.h. Removes the need and usage of a header

Re: [bagder/curl] 13606b: build: make use of 93 lib/*.c renamed files

2013-01-05 Thread Yang Tse
Hi friends, List of those who have expressed their opinion on reverting the lib/*.[ch] renaming, after I said that the reversion commit is ready for pushing and requested reassurance on pushing it or not: Marc Hoersken - Don't revert renaming. Dave Reisner - Push reversion commit. Oscar Koeroo -

Re: [bagder/curl] 13606b: build: make use of 93 lib/*.c renamed files

2013-01-05 Thread Yang Tse
Koeroo - Don't revert renaming. Steve Holme - Push reversion commit. Yang Tse - Don't revert renaming. Tor Arntsen - Push reversion commit. I would appreciate this list gets populated, at least, with every one's name which have posted in this thread. In case I've misinterpreted the decision

Re: [bagder/curl] 13606b: build: make use of 93 lib/*.c renamed files

2013-01-05 Thread Yang Tse
Koeroo - Don't revert renaming. Steve Holme - Push reversion commit. Yang Tse - Don't revert renaming. Tor Arntsen - Push reversion commit. Gisle Vanem - Push reversion commit. Dan Fandrich - Push reversion commit. I would appreciate this list gets populated, at least, with every one's name which

Re: [bagder/curl] 13606b: build: make use of 93 lib/*.c renamed files

2013-01-04 Thread Yang Tse
Hi, I've re-read all messages in this thread a couple of times, if you wish to reply to this message, please, don't do it until you've read it fully ... In first place. I apologize for not bringing to the list the renaming change before pushing it to the public repo. Thanks to all for the

Re: [bagder/curl] 13606b: build: make use of 93 lib/*.c renamed files

2013-01-03 Thread Yang Tse
Hi Daniel, Gisle, and all, The renaming of lib/*.h which already were not named curl_*.h to curl_*.h, and renaming lib/*.c which already were not named curl_*.c has at least the following intended benefits over the non-renamed sources: Relative to the header files... 1) We are able to guarantee

Re: [bagder/curl] 13606b: build: make use of 93 lib/*.c renamed files

2013-01-03 Thread Yang Tse
spent on backporting fixes the more time we have to help upstream. Please clarify if you oppose the renaming or not. @Gisle... On Thu, Jan 3, 2013, Gisle Vanem wrote: Yang Tse yangs...@gmail.com wrote: 2) When

Re: Compilation problems with automake 1.13

2013-01-01 Thread Yang Tse
On Mon, Dec 31, 2012, Dave Reisner wrote: Automake 1.13 made some changes surrounding tracing of the AC_CONFIG_MACRO_DIR macro. Curl appears to override this macro in m4/curl-override.m4 leading to ./buildconf failing rather strangely: [...] Any ideas? I'm going to push attached patch to

Re: Compilation problems with automake 1.13

2013-01-01 Thread Yang Tse
On Tue, Jan 1, 2013 at 5:57 PM, Dave Reisner d...@falconindy.com wrote: On Tue, Jan 01, 2013 at 05:25:51PM +0100, Yang Tse wrote: On Mon, Dec 31, 2012, Dave Reisner wrote: Automake 1.13 made some changes surrounding tracing of the AC_CONFIG_MACRO_DIR macro. Curl appears to override

Re: [PATCH] configure.ac: Prefer AC_CONFIG_HEADERS over AM_CONFIG_HEADER

2013-01-01 Thread Yang Tse
On Sun, Dec 30, 2012 at 4:11 PM, Dave Reisner dreis...@archlinux.org wrote: automake 1.13 removes the definition for the latter, citing the former as being long since deprecated. Fixed in git repo now. Thanks, -- -=[Yang]=- ---

Re: release planning

2012-12-26 Thread Yang Tse
@Daniel Stenberg, Could you evaluate the possibility of releasing an out-of-normal-release-cycle 7.28.2 version immediately before the always-multi merge that will ship with 7.29.0? Even if we don't ever look back again to the time before the new always-multi kingdom, having done an official

Re: libcurl autobuilds: configure whoes when metalink is enabled

2012-12-20 Thread Yang Tse
Hi Gün, but I examined a bit further, and found that I did originally build libssh2 with both shared and static, and therefore there were the .so libs mentioned in libssh2.la; after I manually edited the libssh2.la file and removed the shared libs there the build now succeeds:

Re: libcurl autobuilds: configure whoes when metalink is enabled

2012-12-19 Thread Yang Tse
Guenter li...@gknw.net wrote: somehow libtool wants to link against libssh2.so with the failing build (which I intentionally dont have - just static libssh2.a) and somehow adds a full path to libssh2.so, with the working build libtool seems to honor the values foudn with pkg-config and just

Re: libcurl internal bool TRUE and FALSE definitions and usage

2012-12-17 Thread Yang Tse
On Sun, Dec 16, 2012 Daniel Stenberg wrote: On Sun, 16 Dec 2012, Yang Tse wrote: The purpose of internally using the bool data type was to detect and avoid unclever assignments to bool variables, attempting to get compiler warnings or errors when such a condition existed. While a good

libcurl internal bool TRUE and FALSE definitions and usage

2012-12-15 Thread Yang Tse
Hi friends, The origin of all this libcurl internal bool TRUE and FALSE definitions and usage I am bringing to your attention was triggered due to a compilation issue on a HP-UX system reported by Frank Chang on the mailing list. Fact 1) HP-UX net/if.h system header file indirectly includes

Re: Order Of Include Files in libcurl-7.28.1 that appears to compile OK

2012-12-14 Thread Yang Tse
On Thu, Dec 13, 2012 at 2:35 PM, Frank Chang frank_chan...@hotmail.com wrote: #ifndef HAVE_NET_IF_H #include net/if.h #endif Anything based on that would completely break libcurl building on systems that lack net/if.h -- -=[Yang]=-

Re: Potential Fix to libcurl 7.28.1 HPUX11.11 rw_lock.h compiler error developed

2012-12-12 Thread Yang Tse
On Wed, Dec 12, 2012 at 7:59 AM, Frank Chang frank_chan...@hotmail.com wrote: Yang Tse and Dan Fandrich, Thank you for your help .I just developed a potential fix to the nasty rw_lock.h problem.on HPUX 11.11. curl 7.28.1 compiles and links successfully if you use the following fix

Re: Potential Fix to libcurl 7.28.1 HPUX11.11 rw_lock.h compiler error developed

2012-12-12 Thread Yang Tse
On Wed, Dec 12, 2012 at 2:25 PM, Michael Wood esiot...@gmail.com wrote: On 12 December 2012 14:58, Yang Tse yangs...@gmail.com wrote: On Wed, Dec 12, 2012 at 7:59 AM, Frank Chang frank_chan...@hotmail.com wrote: Yang Tse and Dan Fandrich, Thank you for your help .I just developed

Re: Potential Fix to libcurl 7.28.1 HPUX11.11 rw_lock.h compiler error developed

2012-12-12 Thread Yang Tse
That's an #ifNdef _KERNEL, not #ifdef. Oops. Michael Wood you are right. _KERNEL not defined actually trigerring the issue. Anyway, commit f254c59dc7 should fix the issue. -- -=[Yang]=- --- List admin:

Re: Potential Fix to libcurl 7.28.1 HPUX11.11 rw_lock.h compiler error developed

2012-12-12 Thread Yang Tse
On Wed, Dec 12, 2012 at 5:53 PM, Dan Fandrich d...@coneharvesters.com wrote: I really don't think HP-UX expects every application to have to define TRUE and FALSE macros in order to compile. That should be true. It's much more likely that there's something borked in the compile environment,

Re: Possible Curl-7.28.1 compiler error message on aCC compiler

2012-12-11 Thread Yang Tse
On Tue, Dec 11, 2012 at 6:56 PM, Frank Chang frank_chan...@hotmail.com wrote: On HPUX B.11.11 using the aCC compiler, we obtain the following compile error shown below, Error 20: /usr/include/sys/rw_lock.h, line 169 # ';' expected before 'probably'. This is probably not a good thing

[RESULT] SSPI/Schannel string literals voting process

2012-07-09 Thread Yang Tse
Hi friends, [STEP 3] of this process has resulted in the following results. Top voted Version_Literal is [Version_Literal=WinSSL]. Top voted Config_Literal is [Config_Literal=winssl]. The implications of this result are: Curl_schannel_version() shall return WinSSL literal. Configure option

Asking for some used hardware

2012-07-09 Thread Yang Tse
Hi all, I'm trying to put into good use a couple of server-grade motherboards I have lying around here. In order to complete this task I'm asking for the following used but in working condition hardware: CPUs: Intel Q6700 or Q6600. Memory: Pairs of 1Gb or 2Gb DIMMs with following specs:

[STEP 3] SSPI/Schannel string literals voting process

2012-07-04 Thread Yang Tse
Hi friends, [STEP 2] of this process has resulted in the following candidate lists. Two candidates for the [STEP 3] Version_Literal voting: [Version_Literal=Schannel] [Version_Literal=WinSSL] Two candidates for the [STEP 3] Config_Literal voting: [Config_Literal=schannel]

Re: Makefile.m32: Run del and copy commands through cmd

2012-07-04 Thread Yang Tse
Hi Gün, On Tue, Jul 3, 2012 at 1:41 PM, Guenter li...@gknw.net wrote: it would sure be possible to create subfolders release_obj / debug_obj for *.o *.res etc, but currently I see no great benefit other than separating *.o for debug and release; *.o was anyway already in .gitignore ... I

Re: Makefile.m32: Run del and copy commands through cmd

2012-07-03 Thread Yang Tse
Hi Gün, It would be very interesting if your changes prevented winbuild from writing into the source subdirs as much as possible. :-p thats usually done, yes - however also usually all created files should be removed after a clean, or maybe vclean (which I have in mind to implement); I have

Re: gitignore: Added files generated by mingw32, eclipse and VC

2012-07-03 Thread Yang Tse
On Tue, Jul 3, 2012 at 10:42 AM, Guenter li...@gknw.net wrote: and I think this could be simplified to: +.project +.cproject When the day comes that someone provides eclipse 'project files' for curl we can simply remove those. +*.a +*.res +*.idb +winbuild/*.inc These should better go

Re: Unicode and NTLM

2012-07-03 Thread Yang Tse
Marc Hoersken i...@marc-hoersken.de wrote: the changes introduced a warning about the unused variable passwd. Attached you will find a patch to remove that variable. Merged and pushed. Thanks, -- -=[Yang]=- --- List admin:

Re: Makefile.m32: Run del and copy commands through cmd

2012-07-02 Thread Yang Tse
On Sun, Jul 1, 2012 at 1:41 PM, Guenter li...@gknw.net wrote: I've in mind to change the makefiles in a way so that they can also run from MSYS or Linux bash, and this will exactly tweak at these commands ... Gün, It would be very interesting if your changes prevented winbuild from writing

Re: curl-compilers.m4: -Wstrict-aliasing=3 for warning enabled ...

2012-06-29 Thread Yang Tse
Hi Gün, sorry, but I cant answer this, and cant provide any more info as I did already; the clang 3.0 which I use on the OpenSuSE 12.1 box seems latest of what OpenSuSE currently offers ... Ok, given that we have no clang known version on which the -Wstrict-aliasing=3 stuff works I'm simply

  1   2   3   4   5   >