Re: [PATCH] D15087: [PATCH] Add CERT license clarification

2015-12-10 Thread Chandler Carruth via cfe-commits
chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.

Danny's OK is enough for me. LGTM


http://reviews.llvm.org/D15087



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


Re: [PATCH] D15087: [PATCH] Add CERT license clarification

2015-12-10 Thread Aaron Ballman via cfe-commits
aaron.ballman closed this revision.
aaron.ballman added a comment.

Commit in r255248, thank you for the review!

~Aaron


http://reviews.llvm.org/D15087



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


Re: [PATCH] D15087: [PATCH] Add CERT license clarification

2015-12-08 Thread Aaron Ballman via cfe-commits
aaron.ballman added a comment.

Ping


http://reviews.llvm.org/D15087



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


Re: [PATCH] D15087: [PATCH] Add CERT license clarification

2015-12-08 Thread Daniel Berlin via cfe-commits
dberlin added a comment.

The license  looks fine to me


http://reviews.llvm.org/D15087



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


Re: [PATCH] D15087: [PATCH] Add CERT license clarification

2015-12-08 Thread Alexander Kornienko via cfe-commits
alexfh added a comment.

Daniel, is the proposed solution fine?


http://reviews.llvm.org/D15087



___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


[PATCH] D15087: [PATCH] Add CERT license clarification

2015-11-30 Thread Aaron Ballman via cfe-commits
aaron.ballman created this revision.
aaron.ballman added reviewers: dberlin, chandlerc, alexfh.
aaron.ballman added a subscriber: cfe-commits.

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.

http://reviews.llvm.org/D15087

Files:
  LICENSE.TXT
  clang-tidy/cert/LICENSE.TXT

Index: LICENSE.TXT
===
--- LICENSE.TXT
+++ LICENSE.TXT
@@ -59,5 +59,4 @@
 
 Program Directory
 --- -
-
-
+clang-tidy  clang-tidy/cert
Index: clang-tidy/cert/LICENSE.TXT
===
--- clang-tidy/cert/LICENSE.TXT
+++ clang-tidy/cert/LICENSE.TXT
@@ -0,0 +1,22 @@
+--
+clang-tidy CERT Files
+--
+All clang-tidy files are licensed under the LLVM license with the following
+additions:
+
+Any file referencing a CERT Secure Coding guideline:
+Please allow this letter to serve as confirmation that open source projects on
+http://llvm.org are permitted to link via hypertext to the CERT(R) secure 
coding
+guidelines available at https://www.securecoding.cert.org.   
+
+The foregoing is permitted by the Terms of Use as follows:
+"Linking to the Service
+Because we update many of our Web documents regularly, we would prefer that you
+link to our Web pages whenever possible rather than reproduce them. It is not
+necessary to request permission to make referential hypertext links to The
+Service."
+http://www.sei.cmu.edu/legal/ip/index.cfm.
+
+Please allow this letter to also confirm that no formal permission is required
+to reproduce the title of the content being linked to, nor to reproduce any
+de Minimis description of such content.


Index: LICENSE.TXT
===
--- LICENSE.TXT
+++ LICENSE.TXT
@@ -59,5 +59,4 @@
 
 Program Directory
 --- -
-
-
+clang-tidy  clang-tidy/cert
Index: clang-tidy/cert/LICENSE.TXT
===
--- clang-tidy/cert/LICENSE.TXT
+++ clang-tidy/cert/LICENSE.TXT
@@ -0,0 +1,22 @@
+--
+clang-tidy CERT Files
+--
+All clang-tidy files are licensed under the LLVM license with the following
+additions:
+
+Any file referencing a CERT Secure Coding guideline:
+Please allow this letter to serve as confirmation that open source projects on
+http://llvm.org are permitted to link via hypertext to the CERT(R) secure coding
+guidelines available at https://www.securecoding.cert.org.   
+
+The foregoing is permitted by the Terms of Use as follows:
+"Linking to the Service
+Because we update many of our Web documents regularly, we would prefer that you
+link to our Web pages whenever possible rather than reproduce them. It is not
+necessary to request permission to make referential hypertext links to The
+Service."
+http://www.sei.cmu.edu/legal/ip/index.cfm.
+
+Please allow this letter to also confirm that no formal permission is required
+to reproduce the title of the content being linked to, nor to reproduce any
+de Minimis description of such content.
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits