kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #24529
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