Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check Extension Licenses #25

Closed
mvcbot opened this issue Dec 7, 2018 · 12 comments
Closed

Check Extension Licenses #25

mvcbot opened this issue Dec 7, 2018 · 12 comments
Labels
help wanted Extra attention is needed

Comments

@mvcbot
Copy link

mvcbot commented Dec 7, 2018

Issue by ivargrimstad
Thursday Jun 21, 2018 at 06:16 GMT
Originally opened as mvc-spec/ozark#179


Go through the contributed extensions and verify that they have a license that is compatible with ASLv2.

@mvcbot
Copy link
Author

mvcbot commented Dec 7, 2018

Comment by chkal
Thursday Jun 21, 2018 at 07:05 GMT


You are referring to the dependency on the 3rd party template engines, correct? So actually we have to check the license of all template engines for which MVC view engines exist, correct?

@mvcbot mvcbot added the help wanted Extra attention is needed label Dec 7, 2018
@mvcbot
Copy link
Author

mvcbot commented Dec 7, 2018

Comment by ivargrimstad
Thursday Jun 21, 2018 at 07:18 GMT


Yes, if we package them together in the Ozark deliverable. This is definitely something that will be checked by the IP-team if/when we transfer to Eclipse.

A solution for potential non-compliant view engines is to extract them to a separate extensions project with appropriate licensing

@mvcbot
Copy link
Author

mvcbot commented Dec 7, 2018

Comment by chkal
Thursday Jun 21, 2018 at 07:31 GMT


What about provided scoped dependencies? During one of my pull request reviews I asked myself, whether having provided instead of compile scoped dependencies for the template engines would make more sense. Users would have to add the dependency explicitly, but they would also have more control over the exact version and updating would be easier.

So would that have any impact on the license requirements? Any idea?

@mvcbot
Copy link
Author

mvcbot commented Dec 7, 2018

Comment by ivargrimstad
Thursday Jun 21, 2018 at 07:41 GMT


I have to check up whether that makes a difference. Anyway, I think that if we provide the view engine, we should also provide the implementation with a specific version that we have tested it with. Letting the users do this themselves just adds complexity and lead to errors.

@mvcbot
Copy link
Author

mvcbot commented Dec 7, 2018

Comment by chkal
Thursday Jun 21, 2018 at 10:47 GMT


Ok! I'm not strong on the idea of using provided scope. It just came to my mind as an alternative. I'm fine with leaving it as it is. But maybe it could be a valid workaround if any template engine has a problematic license.

@mvcbot
Copy link
Author

mvcbot commented Dec 7, 2018

Comment by ivargrimstad
Tuesday Jun 26, 2018 at 09:30 GMT


Extensions:

  • - asciidoc - Apache-2.0
  • - freemarker - Apache-2.0
  • - handlebars - Apache-2.0
  • - jade - MIT
  • - jetbrick - Apache-2.0, BSD (antlr 4)
  • - jsr223 - BCL
  • - mustache - Apache-2.0
  • - pebble - proprietary license?
  • - stringtemplate - BSD (antlr 4)
  • - thymeleaf - Apache-2.0
  • - velocity - Apache-2.0

@mvcbot
Copy link
Author

mvcbot commented Dec 7, 2018

Comment by Daniel-Dos
Tuesday Jun 26, 2018 at 14:16 GMT


Hi @ivargrimstad ,

In site - >pebble/license , really seems to be a proprietary license .

In link -> jetbrick-template informs that it is about the apache license.

@mvcbot
Copy link
Author

mvcbot commented Dec 7, 2018

Comment by chkal
Tuesday Jun 26, 2018 at 15:13 GMT


@ivargrimstad So Jetbrick may be problematic because of the transitive antlr dependency?

@mvcbot
Copy link
Author

mvcbot commented Dec 7, 2018

Comment by chkal
Saturday Sep 08, 2018 at 06:12 GMT


Just a quick note on this. If we find incompatible license in our dependencies because of 3rd party view engines, we should most likely drop them before moving to EE4J. We can still keep them alive in a separate GitHub project if we want.

@mvcbot
Copy link
Author

mvcbot commented Dec 7, 2018

Comment by ivargrimstad
Saturday Sep 08, 2018 at 06:32 GMT


Let's wait and see. The CQ team at EF will do a pretty thorough job going through it when we transfer the code and they're much better at doing that than we are I guess...
Then we can omit whatever doesn't meet their requirements.

@mvcbot
Copy link
Author

mvcbot commented Dec 7, 2018

Comment by chkal
Saturday Sep 08, 2018 at 08:01 GMT


Ok, sounds great!

@chkal
Copy link
Contributor

chkal commented Jan 19, 2019

I guess we should close this in favor of #30.

@chkal chkal closed this as completed Jan 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants