Project

General

Profile

Bug #11648

Review use of BSD License in view of inclusion of GPLv2 polarssl code

Added by Ben Morgan almost 4 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
Start date:
02/05/2016
Due date:
% Done:

100%

Estimated time:
0.00 h
Spent time:
Duration:

Description

Cetlib's current license file is BSD, but the library includes code from the polarssl project, the files of which are GPLv2. It's likely (but I am not a lawyer) that this falls under the "combining two modules" FAQ on the GPLv2: http://www.gnu.org/licenses/old-licenses/gpl-2.0-faq.en.html#MereAggregation . That may require that cetlib is also GPLv2 licensed, and possibly that clients of cetlib are also GPLv2 licensed (http://www.gnu.org/licenses/old-licenses/gpl-2.0-faq.en.html#IfLibraryIsGPL)

FWIW I have no opinion on the best open/free license, but simply want to avoid problems down the road!

Associated revisions

Revision 00e7d8ee (diff)
Added by Kyle Knoepfel over 3 years ago

Resolve issue #11648: Remove polarssl dependency.

Revision 2f25d8f5 (diff)
Added by Kyle Knoepfel over 3 years ago

Resolve issue #11648: Remove polarssl dependency.

History

#1 Updated by Kyle Knoepfel almost 4 years ago

  • Status changed from New to Accepted

Thank you, we will review.

#2 Updated by Kyle Knoepfel over 3 years ago

  • Status changed from Accepted to Resolved
  • Assignee set to Kyle Knoepfel
  • % Done changed from 0 to 100
  • Estimated time set to 0.00 h

The md5 and sha1 cetlib classes now depend on natively-installed openssl instead of polarssl, allowing us to remove the polarssl dependency entirely. Note, however, that once we support El Capitan distributions of art, we may need to choose a different solution, since openssl is no longer included with El Capitan. One option may be using the Apple-provided CommonCrypto library.

Implemented with cetlib:00e7d8ee.

#3 Updated by Kyle Knoepfel over 3 years ago

We have now moved from using openssl on OSX builds to the Apple-provided CommonCrypto library.

Implemented with cetlib:adc2aa9.

#4 Updated by Kyle Knoepfel over 3 years ago

  • Status changed from Resolved to Closed
  • Target version set to 2.02.00


Also available in: Atom PDF