[GitHub] zeppelin issue #1276: Add Highcharts support

[bzz <gi...@git.apache.org>]

Github user bzz commented on the issue:

    https://github.com/apache/zeppelin/pull/1276
  
    @rockiey thank you for contributing!
    
    Highcharts is a great library, but I belive it will be hard to include it as a dependency to ASF project.
    
    If highchart gets included to project codebase, it will end up in convenience release binary the project that is distributed by ASF, so we must make sure that it's license complies with legal requirements from Apache Software Foundation.
    
    Here is a process that needs to be done, in order to archive the compliance:
     - verify that it's license belongs to ASF [category-a](http://www.apache.org/legal/resolved.html#category-a) license, if that is the case - it can be included as dependency. Then
     - full text of the license must be included in the `zeppelin-distribution/src/bin_license/licenses`,  then
     - record must be added to `zeppelin-distribution/src/bin_license/LICENSE`
     - copyrights notice, in case it exists needs to be recorded in `zeppelin-distribution/src/bin_license/NOTICE` 
    
    
    Looking at https://github.com/knockdata/zeppelin-highcharts - it looks great and it does acutally make a lot of sence to have it as separate binary, so that user can add it at his own will (and be responsible for adhering it's licence terms).
    



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---