[Asterisk-code-review] cel/mongodb: Added mongodb backend for cel (asterisk[master])

[Matt Jordan]

Matt Jordan has posted comments on this change.

Change subject: cel/mongodb: Added mongodb backend for cel
......................................................................


Patch Set 2: Code-Review-2

Hi Catalin -

First, thanks for writing this patch. I think it would have been great for Asterisk to have a CDR/CEL backend for MongoDB. Unfortunately, after investigating the implementation here, I think we've hit a permanent snag in including this in the Asterisk code base.

The library you're using here, mongo-c-driver (https://github.com/mongodb/mongo-c-driver/blob/master/src/mongoc/mongoc.h), is licensed under the Apache License Version 2.0. This license is, unfortunately, not compatible with the GPLv2. For explanations, see:
 * http://www.apache.org/licenses/GPL-compatibility.html
 * http://www.gnu.org/licenses/license-list.en.html#apache2

It sucks that we have to reject this patch because of license incompatibility, but unfortunately, since Asterisk is licensed under the GPLv2, that's what we're going to have to do.

Again - sorry for having to reject this patch.

-- 
To view, visit https://gerrit.asterisk.org/2361
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I8bf4f02fbc6a3a5edd7f10256ea0e14f49193c4f
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Catalin Stanciu <catacsdev at gmail.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Catalin Stanciu <catacsdev at gmail.com>
Gerrit-Reviewer: George Joseph <george.joseph at fairview5.com>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-HasComments: No