This is an archive of the discontinued LLVM Phabricator instance.

[PATCH] Add CERT license clarification
ClosedPublic

Authored by aaron.ballman on Nov 30 2015, 12:32 PM.

Details

Summary

During the review of D13446, Daniel pointed out that the CERT website's Terms of Use appear to prohibit using the rule title and minimal descriptions in our documentation. CERT's legal team has confirmed that our uses are acceptable, and drafted a license agreement that states such. I have verified that none of our documentation replicates the descriptions used in CERT's guidelines, and this patch is updating the license file with CERT's confirmation.

Diff Detail

Event Timeline

aaron.ballman retitled this revision from to [PATCH] Add CERT license clarification.
aaron.ballman updated this object.
aaron.ballman added a subscriber: cfe-commits.
alexfh edited edge metadata.Dec 8 2015, 8:52 AM

Daniel, is the proposed solution fine?

dberlin edited edge metadata.Dec 8 2015, 9:06 AM

The license looks fine to me

chandlerc accepted this revision.Dec 10 2015, 1:40 AM
chandlerc edited edge metadata.

Danny's OK is enough for me. LGTM

This revision is now accepted and ready to land.Dec 10 2015, 1:40 AM
alexfh accepted this revision.Dec 10 2015, 2:03 AM
alexfh edited edge metadata.

Just to be in a good company ;)
LGTM

aaron.ballman closed this revision.Dec 10 2015, 5:57 AM

Commit in r255248, thank you for the review!

~Aaron