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

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

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