snleee commented on issue #3722: Update LICENSE and NOTICE files
URL: https://github.com/apache/incubator-pinot/pull/3722#issuecomment-456516171
 
 
   > My take is if it's MIT license it needs not be included:
   > http://www.apache.org/dev/licensing-howto.html
   > http://www.apache.org/dev/licensing-howto.html#permissive-deps
   > https://www.apache.org/legal/resolved.html#required-third-party-notices
   
   @felixcheung @kishoreg 
   ```
   Bundling a dependency which is issued under one of the following licenses is 
straightforward, assuming that said license applies uniformly to all files 
within the dependency:
   
   BSD (without advertising clause)
   MIT/X11
   In LICENSE, add a pointer to the dependency's license within the 
distribution and a short note summarizing its licensing:
   
   This product bundles SuperWidget 1.2.3, which is available under a
   "3-clause BSD" license.  For details, see deps/superwidget/.
   ```
   
   It seems that the pointer is needed for BSD, MIT licenses. The example above 
has the link pointer to the full license `deps/superwiget`. Am I reading this 
incorrectly? 
   
   As long as we don't violate the license issue, I can go ahead for removing 
copied license files from `/licenses` for BSD, MIT.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to