← Back to team overview

kicad-developers team mailing list archive

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

 

Please excuse my fat fingers.  That should be 6440.

On 5/5/2016 11:04 AM, Chris Pavlina wrote:
> I do hope you didn't intent to credit Dick for 6640, that one's mine! ;)
> 
> On Thu, May 05, 2016 at 10:57:34AM -0400, Wayne Stambaugh wrote:
>> On 5/5/2016 10:47 AM, Chris Pavlina wrote:
>>> 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?
>>
>> If any code from commits 6640 (which I forgot to give Dick credit for),
>> 6538, or 6467 was backported, then yes Dick needs to be in the credits.
>>
>>>
>>>>
>>>> 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


References