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

508 jackson 2.1.x #536

Merged
merged 3 commits into from
Apr 22, 2013
Merged

508 jackson 2.1.x #536

merged 3 commits into from
Apr 22, 2013

Conversation

zznate
Copy link

@zznate zznate commented Mar 15, 2013

Updates to support Jackson 2.1.x as outlined in #508

@purplefox
Copy link
Contributor

Is this the ASL version?

I believe Jackson has a weird licensing policy ASL and LGPL.

We wouldn't be able to use the LGPL version in Vert.x

Just want to confirm first that this is the ASL 2.0 version we're using....

@zznate
Copy link
Author

zznate commented Apr 19, 2013

I think so, but it's time for the @cowtowncoder bat signal just to be sure.

[Edit] http://wiki.fasterxml.com/JacksonLicensing so I guess you can just say "ASF.v2.0 for Jackson" in a licenses dir and be done with it (i've previously structured stuff similar to https://github.com/apache/cassandra/tree/trunk/lib/licenses in these situations)

@cowtowncoder
Copy link

There is just one jar for 2.x, and the human-readable answer is: if you want to use Apache, you will use Apache. You can clarify this externally if you like ("we use Jackson under Apache license" or whatever).
We just allow licensing under more than one license, you get to choose.

Dual-licensing really only affects contributions (i.e. they need to be done to allow licensing under multiple). And well Jackson team that gets to answer questions :).
But should not confuse lives of downstream teams.

We hope to clarify this further in near future (2.2 if possible, 2.3 if it takes longer), so that rule would be "unless otherwise stated, it's Apache all the way".

I hope this helps.

Oh, and obviously all help would be welcome, this is being discussed on Jackson dev list.

@zznate
Copy link
Author

zznate commented Apr 19, 2013

Rad! Thanks @cowtowncoder for dropping in to clarify - much appreciated.

@purplefox
Copy link
Contributor

Ok thanks guys

@purplefox
Copy link
Contributor

Great. Will merge this.

Nate - can you sign the SpringSource CLA https://support.springsource.com/spring_committer_signup if you haven't done so already. I know it seems pretty anal for a small contrib like this but we have to be pretty strict as we move to Eclipse.

Thanks!

@zznate
Copy link
Author

zznate commented Apr 22, 2013

@purplefox Np. Done w/ conf number 47820130422093329. Thx.

purplefox added a commit that referenced this pull request Apr 22, 2013
@purplefox purplefox merged commit e8df4b0 into eclipse-vertx:master Apr 22, 2013
purplefox added a commit that referenced this pull request May 10, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants