← Back to team overview

kicad-developers team mailing list archive

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

 

On Thu, May 05, 2016 at 10:15:02AM -0400, Wayne Stambaugh wrote:
> [snip]
> 
> 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.

Hm, did he write any of the code in these patches?

> 
> 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
> > 
> 
> _______________________________________________
> 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