← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH 0/4] Backport libcurl code from development branch

 

Thanks Simon!  Nice work.  I just tested this on windows and it appears
to work fine.  I'm assuming you tested this on Linux.  Would one of our
osx devs please apply these patches to the 4 stable branch and make sure
that it builds and works correctly on osx?  Once I finish vetting Chris`
file versioning patch, I'll apply that to the stable release and push a
4.0.3 stable release so the github plugin can be enabled.

I didn't see any credit to Dick for the work he did on libcurl
transition so that will need to be added to commit message.

On 5/5/2016 4:39 AM, Simon Richter wrote:
> Hi,
> 
> this backports Mark's avhttp -> libcurl transition to the stable series.
> With these, I've been able to build a stable version of KiCad with the
> GitHub plugin enabled that does not link against OpenSSL, which should
> allow everyone to ship binaries with the Github plugin enabled.
> 
>    Simon
> 
> Mark Roszko (2):
>   Replace the avhttp library used by the github plugin with libcurl.
>   Replace avhttp with libcurl: Some fixes:     1. Fixed an assumption
>     somebody originally made in the plugin that std::string had
>     contiguous storage. This is not specced behavior pre C++11 so we
>     gamble by calling .reserve() which should give a far better
>     guarantee.     2. Added copy to clipboard information for curl    
>     3. Removed some openssl references in compiling.md     4. Renamed
>     struct vars to be uppercase to match "public var" code style policy
> 
> Simon Richter (2):
>   Add missing dependency github_plugin -> pcbcommon
>   Remove unused FindOpenSSL.cmake
> 
> 
>  CMakeLists.txt                         |   5 +
>  CMakeModules/FindOpenSSL.cmake         | 342 ---------------------------------
>  Documentation/development/compiling.md |  12 +-
>  common/CMakeLists.txt                  |   6 +-
>  common/basicframe.cpp                  |   3 +
>  common/kicad_curl/kicad_curl.cpp       |  80 ++++++++
>  common/kicad_curl/kicad_curl_easy.cpp  | 163 ++++++++++++++++
>  common/pgm_base.cpp                    |   9 +
>  include/kicad_curl/kicad_curl.h        |  89 +++++++++
>  include/kicad_curl/kicad_curl_easy.h   | 166 ++++++++++++++++
>  pcbnew/CMakeLists.txt                  |   2 +
>  pcbnew/github/CMakeLists.txt           |  41 +---
>  pcbnew/github/github_getliblist.cpp    |  69 ++++---
>  pcbnew/github/github_getliblist.h      |   4 +-
>  pcbnew/github/github_plugin.cpp        | 152 +++------------
>  pcbnew/github/github_plugin.h          |   6 +-
>  16 files changed, 601 insertions(+), 548 deletions(-)
>  delete mode 100644 CMakeModules/FindOpenSSL.cmake
>  create mode 100644 common/kicad_curl/kicad_curl.cpp
>  create mode 100644 common/kicad_curl/kicad_curl_easy.cpp
>  create mode 100644 include/kicad_curl/kicad_curl.h
>  create mode 100644 include/kicad_curl/kicad_curl_easy.h
> 
> 
> 
> _______________________________________________
> 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