Re: relative performance

2021-08-24 Thread Ben Greear via curl-library
On 8/24/21 8:20 AM, Daniel Stenberg via curl-library wrote: Hi all, For a long time I've wanted to get something done that allows us to compare curl's relative performance. Ideally something we can run every once in a while to compare that nothing major has turned sour without us being aware of

Re: Problem linking lib-cares

2020-03-07 Thread Ben Greear via curl-library
On 03/07/2020 02:35 PM, Daniel Stenberg wrote: On Sat, 7 Mar 2020, Ben Greear via curl-library wrote: 21517LDFLAGS: -L/usr/lib Usually, when -L/usr/lib appears in LDFLAGS, the reason is that some package that configure picked up has a bad pkg-config that includes that (and

Problem linking lib-cares

2020-03-07 Thread Ben Greear via curl-library
Hello, On older platforms where libcares is old, it seems that my builds are broken because the -L/usr/lib comes before the -L that is generated with the --with-cares [dir] argument: From config.log: 21516CPPFLAGS:-I/home/greearb/git/btbits/l4libs/c-ares.ct/include 21517LDFLAGS

Re: Problem linking on mac-os

2019-11-16 Thread Ben Greear via curl-library
On 11/16/2019 07:25 PM, Ray Satiro via curl-library wrote: On 11/16/2019 10:12 AM, Ben Greear via curl-library wrote: I have a program called l4helper, that links against a libcurl (and lib-cares) that I have compiled myself. The library is in my LD_LIBRARY_PATH, and at any rate was never

Problem linking on mac-os

2019-11-16 Thread Ben Greear via curl-library
I have a program called l4helper, that links against a libcurl (and lib-cares) that I have compiled myself. The library is in my LD_LIBRARY_PATH, and at any rate was never in /usr/local/lib/libcares.2.dylib Anyone know why it is trying to load that specific location and how to make it look in

Re: [PATCH 1/2] urlapi: Fix compiler warning on fedora-17.

2018-12-09 Thread Ben Greear via curl-library
On 12/09/2018 02:31 AM, Daniel Stenberg wrote: On Fri, 30 Nov 2018, Ben Greear wrote: This is a broken compiler that warns about this. I really prefer the existing one-line version to this 4-line replacement. Well yes, it would be great if the compiler wasn't broken, but still nice to all

Re: [PATCH] curl: Support CURLOPT_LOCALADDR to bind to local address.

2018-12-07 Thread Ben Greear via curl-library
On 12/07/2018 06:52 AM, Daniel Stenberg wrote: On Fri, 7 Dec 2018, Ben Greear wrote: I recently rebased this patch and did a new pull request (pull requests are not my specialty, possibly I did that wrong). Are you talking about https://github.com/curl/curl/pull/2177 ? It still has compil

Re: [PATCH] curl: Support CURLOPT_LOCALADDR to bind to local address.

2018-12-07 Thread Ben Greear via curl-library
On 12/13/2017 12:19 AM, Daniel Stenberg wrote: On Thu, 9 Nov 2017, gree...@candelatech.com wrote: Hi Ben! This allows a user to bind to both an interface (with CURLOPT_INTERFACE) and a local IP address. In doing so, it allows the user to work-around some serious performance issues on mach

Re: [PATCH 2/2] openssl: Fix unused variable compiler warning on fedora 17

2018-11-30 Thread Ben Greear via curl-library
On 11/30/2018 03:04 PM, Daniel Stenberg wrote: On Wed, 28 Nov 2018, Ben Greear via curl-library wrote: ctx_options was not used, so the compiler warned. Add a (void)(ctx_options) line to make the compiler happy. What OpenSSL version do you have installed to get this warning? Maybe it

Re: [PATCH 1/2] urlapi: Fix compiler warning on fedora-17.

2018-11-30 Thread Ben Greear via curl-library
On 11/30/2018 03:01 PM, Daniel Stenberg wrote: On Wed, 28 Nov 2018, Ben Greear via curl-library wrote: i = part; -for(o = enc; *i; ++o, ++i) - *o = (*i == ' ') ? '+' : *i; +for(o = enc; *i; ++o, ++i) { + if (*i ==

[PATCH 2/2] openssl: Fix unused variable compiler warning on fedora 17

2018-11-28 Thread Ben Greear via curl-library
From: Ben Greear ctx_options was not used, so the compiler warned. Add a (void)(ctx_options) line to make the compiler happy. Hopefully that is the right fix... Signed-off-by: Ben Greear --- lib/vtls/openssl.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/vtls/openssl.c b/lib/vtls/

[PATCH 1/2] urlapi: Fix compiler warning on fedora-17.

2018-11-28 Thread Ben Greear via curl-library
From: Ben Greear It seems the compiler there gets confused and thinks a char gets converted to an int. This fixes the warning and should be equivalent code. Signed-off-by: Ben Greear --- lib/urlapi.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/lib/urlapi.c b/li