Drop requirement for a CLA #369

Merged
danh-arm merged 2 commits from dh/drop-cla into development 2020-03-03 05:28:14 -06:00
danh-arm commented 2020-02-19 05:51:18 -06:00 (Migrated from github.com)

The Mbed Crypto project no longer requires a CLA. Contributions from now on
must be made under both Apache-2.0 AND GPL-2.0-or-later licenses, to enable
LTS (Long Term Support) branches of the software to continue to be provided
under either Apache-2.0 OR GPL-2.0-or-later. Contributors must accept the terms of the Developer Certificate of Origin (DCO) by adding a Signed-off-by: line to each commit
message.

The software on the development branch continues to be provided under
Apache-2.0.

Update README.md and CONTRIBUTING.md to explain the new licensing model.
Add a copy of the DCO to the project.

Expand the full Apache-2.0 license text in the LICENSE file and remove the
redundant apache-2.0.txt.

Fix inconsistent list formatting in CONTRIBUTING.md.

The Mbed Crypto project no longer requires a CLA. Contributions from now on must be made under both Apache-2.0 AND GPL-2.0-or-later licenses, to enable LTS (Long Term Support) branches of the software to continue to be provided under either Apache-2.0 OR GPL-2.0-or-later. Contributors must accept the terms of the Developer Certificate of Origin (DCO) by adding a Signed-off-by: line to each commit message. The software on the development branch continues to be provided under Apache-2.0. Update README.md and CONTRIBUTING.md to explain the new licensing model. Add a copy of the DCO to the project. Expand the full Apache-2.0 license text in the LICENSE file and remove the redundant apache-2.0.txt. Fix inconsistent list formatting in CONTRIBUTING.md.
Patater (Migrated from github.com) reviewed 2020-02-19 05:51:18 -06:00
yanesca (Migrated from github.com) reviewed 2020-02-19 05:51:18 -06:00
simonbutcher (Migrated from github.com) reviewed 2020-02-19 05:51:18 -06:00
yanesca (Migrated from github.com) reviewed 2020-02-19 07:58:34 -06:00
@ -24,7 +19,8 @@ Making a Contribution
1. Write a test which shows that the bug was fixed or that the feature works as expected.
yanesca (Migrated from github.com) commented 2020-02-19 07:58:26 -06:00

Additionally to the dco, we could consider mentioning the inbound licences here too. (The dco makes a vague reference to an open source licence, it is not impossible to figure out what it is, but I think we should make the job easier for our contributors.)

Additionally to the dco, we could consider mentioning the inbound licences here too. (The dco makes a vague reference to an open source licence, it is not impossible to figure out what it is, but I think we should make the job easier for our contributors.)
@ -2,8 +2,6 @@
yanesca (Migrated from github.com) commented 2020-02-19 07:54:57 -06:00

As discussed earlier, we could try clarifying the inbound licenses a little bit. Maybe writing something like this in addition: "The inbound licences are both Apache-2.0 AND GPL-2.0-or-later simultaneously, that is, contributions are made under both."

As discussed earlier, we could try clarifying the inbound licenses a little bit. Maybe writing something like this in addition: "The inbound licences are both Apache-2.0 AND GPL-2.0-or-later simultaneously, that is, contributions are made under both."
danh-arm (Migrated from github.com) reviewed 2020-02-19 10:06:27 -06:00
@ -24,7 +19,8 @@ Making a Contribution
1. Write a test which shows that the bug was fixed or that the feature works as expected.
danh-arm (Migrated from github.com) commented 2020-02-19 10:06:27 -06:00

OK, I'll link the license section of the readme here.

OK, I'll link the license section of the readme here.
danh-arm (Migrated from github.com) reviewed 2020-02-19 10:06:33 -06:00
@ -2,8 +2,6 @@
danh-arm (Migrated from github.com) commented 2020-02-19 10:06:33 -06:00

I think I already clarified that point. Your suggestion just says the same thing twice IMO.

I think I already clarified that point. Your suggestion just says the same thing twice IMO.
gilles-peskine-arm (Migrated from github.com) approved these changes 2020-02-19 12:40:41 -06:00
mpg (Migrated from github.com) approved these changes 2020-02-20 02:32:17 -06:00
mpg (Migrated from github.com) left a comment

LGTM

LGTM
mpg commented 2020-02-20 02:37:45 -06:00 (Migrated from github.com)

Note: the CI is only failing the Mbed OS tests, which are a known issue clearly independent from this PR.

Note: the CI is only failing the Mbed OS tests, which are a known issue clearly independent from this PR.
yanesca commented 2020-02-25 04:50:08 -06:00 (Migrated from github.com)

The CI fails on Mbed OS, but not for the known issue we had. This PR deletes the file apache-2.0.txt, which is targeted explicitly by the importer script in Mbed OS.

@Patater The Mbed OS importer script will need updating once this PR is merged.

The CI fails on Mbed OS, but not for the known issue we had. This PR deletes the file `apache-2.0.txt`, which is targeted explicitly by the importer script in Mbed OS. @Patater The Mbed OS importer script will need updating once this PR is merged.
Patater commented 2020-02-25 05:04:43 -06:00 (Migrated from github.com)

The CI fails on Mbed OS, but not for the known issue we had. This PR deletes the file apache-2.0.txt, which is targeted explicitly by the importer script in Mbed OS.

@Patater The Mbed OS importer script will need updating once this PR is merged.

Thanks for the heads up. We'll fix this when we update the version of Mbed TLS in Mbed OS.

> The CI fails on Mbed OS, but not for the known issue we had. This PR deletes the file `apache-2.0.txt`, which is targeted explicitly by the importer script in Mbed OS. > > @Patater The Mbed OS importer script will need updating once this PR is merged. Thanks for the heads up. We'll fix this when we update the version of Mbed TLS in Mbed OS.
Sign in to join this conversation.
No milestone
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
gh-ARMmbed/mbed-crypto!369
No description provided.