Re: Remove file with invalid license

2017-12-19 Thread Chris Lemmons
Ah, you are correct, then. I'm not a fan, but I do see the point in having it brief. On Tue, Dec 19, 2017 at 10:14 AM, Dan Kirkwood wrote: > ```It is important to keep NOTICE as brief and simple as possible, as > each addition places a burden on downstream consumers. > > Do not add anything to NO

Re: Remove file with invalid license

2017-12-19 Thread Dan Kirkwood
```It is important to keep NOTICE as brief and simple as possible, as each addition places a burden on downstream consumers. Do not add anything to NOTICE which is not legally required. ``` https://www.apache.org/dev/licensing-howto.html#mod-notice apache.org Assembling LICENSE and NOTICE. Home pa

Re: Remove file with invalid license

2017-12-19 Thread Robert Butts
I don't agree with https://github.com/apache/incubator-trafficcontrol/commit/d7422b3f05f2628de07614efa20799b01cfc1e41 "remove from NOTICE to keep it short " While the MIT doesn't require Attribution, Daniel and the SecLists project originally did, it was very specifically licensed "CC Attribution"

Re: Remove file with invalid license

2017-12-19 Thread Dave Neuman
I merged it, you need to do a backport to 2.1 as well. On Tue, Dec 19, 2017 at 9:16 AM, Robert Butts wrote: > PR updating the license: > https://github.com/apache/incubator-trafficcontrol/pull/1681 > > On Tue, Dec 19, 2017 at 9:13 AM, Chris Lemmons wrote: > > > https://github.com/danielmiessler

Re: Remove file with invalid license

2017-12-19 Thread Robert Butts
PR updating the license: https://github.com/apache/incubator-trafficcontrol/pull/1681 On Tue, Dec 19, 2017 at 9:13 AM, Chris Lemmons wrote: > https://github.com/danielmiessler/SecLists is now licensed MIT. > Thanks, Eric, for talking to Daniel Miessler for us and getting this > taken care of! >

Re: Remove file with invalid license

2017-12-19 Thread Chris Lemmons
https://github.com/danielmiessler/SecLists is now licensed MIT. Thanks, Eric, for talking to Daniel Miessler for us and getting this taken care of! On Mon, Dec 18, 2017 at 1:56 PM, Chris Lemmons wrote: > Excellent, Eric. That neatly cleans up the problem. I do think we > should merge my PR (1677)