← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Rework OpenSSL mutex workaround

 

Hey Simon,

I cannot apply your patch against the tip of the product branch.  No
hurry but when you get a chance please revise your patch against the tip
of the product branch. I would like to test it to see how much of a
performance hit there is on windows.  I suspect it will be unacceptably
slow.  I seem to remember doing some testing with the multi-threaded
footprint loading disabled and the performance hit was just too severe
to be usable.

I just checked and unfortunately msys/mingw builds libcurl with openssl
instead of gnutls by default.  There is an option to build libcurl with
gnutls in the PKGBUILD file and there is gnutls package.  I did manage
to successfully build the curl packages (i686 and x86_64) against gnutls
without any issues (except skipping the gpg check of the curl archive).
However, KiCad does not build against curl built with gnutls so I'm not
sure what needs to be changes to make this build.  If someone else has
time to look at this, I would appreciate it.  As it stands right now, we
will have to build kicad with the openssl build of curl on windows.

Thanks,

Wayne

On 5/5/2016 5:56 PM, Simon Richter wrote:
> Hi Wayne,
> 
> On 05.05.2016 22:51, Wayne Stambaugh wrote:
> 
>> Is this patch for the stable branch, the product branch, or both?
> 
> Currently, for the product branch, and ideally it should be trivial to
> backport to the stable branch afterwards (it removes a lot of code that
> doesn't exist in the stable branch).
> 
> I'd probably wait pushing the avhttp -> curl changes to the stable
> branch until this has gotten sufficient eyeballs so we can introduce it
> without a regression on old Macs.
> 
>    Simon
> 
> 
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
> 



Follow ups

References