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

Play 2.5 #48

Closed
morellik opened this issue Mar 7, 2016 · 19 comments
Closed

Play 2.5 #48

morellik opened this issue Mar 7, 2016 · 19 comments

Comments

@morellik
Copy link
Contributor

morellik commented Mar 7, 2016

Did you try play-jongo on Play 2.5? Is there a way to use under Play 2.5?

@magro
Copy link
Collaborator

magro commented Mar 7, 2016

It should be possible to use it with play 2.4/2.5, this was done with #44. If it doesn't work this issue is valid :-)

@morellik
Copy link
Contributor Author

morellik commented Mar 8, 2016

Using the following libraries in build.sbt
"org.mongodb" % "mongo-java-driver" % "3.2.2",
"org.jongo" % "jongo" % "1.2",
"com.fasterxml.jackson.module" %% "jackson-module-scala" % "2.4.1" % "optional",
"uk.co.panaxiom" %% "play-jongo" % "1.0.1-jongo1.2"

I receive the following warning:

[warn] There may be incompatibilities among your library dependencies.
[warn] Here are some of the libraries that were evicted:
[warn] * com.typesafe.play:play-netty-server_2.11:2.4.2 -> 2.5.0 (caller: docarchive:docarchive_2.11:1.0-SNAPSHOT, uk.co.panaxiom:play-jongo_2.11:1.0.1-jongo1.2)
[warn] * com.typesafe.play:play-java_2.11:2.4.2 -> 2.5.0 (caller: com.typesafe.play:filters-helpers_2.11:2.5.0, docarchive:docarchive_2.11:1.0-SNAPSHOT, com.typesafe.play:play-java-jdbc_2.11:2.5.0, uk.co.panaxiom:play-jongo_2.11:1.0.1-jongo1.2, com.typesafe.play:play-java-ws_2.11:2.5.0)
[warn] * com.typesafe.play:play-server_2.11:2.4.2 -> 2.5.0 (caller: com.typesafe.play:play-netty-server_2.11:2.5.0, docarchive:docarchive_2.11:1.0-SNAPSHOT, uk.co.panaxiom:play-jongo_2.11:1.0.1-jongo1.2)

@mancvso
Copy link

mancvso commented Mar 9, 2016

jackson-module-scala says on its Github page

Please use versions 2.6.5, or 2.7.2; earlier released versions are known to contain significant bugs.

@magro
Copy link
Collaborator

magro commented Mar 9, 2016

I'd say that you can ignore the warnings, or exclude transitive dependencies where needed.

@morellik
Copy link
Contributor Author

If I've to write a production application I prefer to have compatibility libraries. In this case the Play! evicts the incompatibility libraries so I don't know how to use them.
Another issue regards the application.conf that was changed and I don't know where put the database declaration.

@morellik
Copy link
Contributor Author

One of the Play! contributors told me: "The migration for Play modules is not as much as it was in 2.4, so it should not be too difficult for library authors to update."

So, if you can, we are happy!

Thanks

@magro
Copy link
Collaborator

magro commented Mar 10, 2016

I'm going to vacation tomorrow, not sure about my workload afterwards. If you submit a pull request I can make a release then.

@morellik
Copy link
Contributor Author

I tried, but github ask me to compare branches and I didn't know what I've to do. I'll wait until you come back from you vacation.

Good vacation ;-))

@magro
Copy link
Collaborator

magro commented Mar 11, 2016

Github has rather good docs... :-) Thanks!

@morellik
Copy link
Contributor Author

I checked the Play! module documentation and the jongo sources and I don't understand where is the incompatibility, You follow exactly the documentation guidelines. I wrote on the Play! group to ask about that.

@morellik
Copy link
Contributor Author

The group answer is: Modules don't exist anymore in 2.5. A "module" in 2.5 is just a bunch of classes that are bound in DI.

@magro
Copy link
Collaborator

magro commented Mar 11, 2016

Right, so what's the problem? Just register the JongoModule.

@morellik
Copy link
Contributor Author

Another Play Group answer: Looks like it depends on Play 2.4, which isn't compatible with 2.5 on a binary level.

@gmethvin
Copy link

@magro If it works for 2.5, can you create a release that is compiled against 2.5, to ensure binary compatibility?

@morellik
Copy link
Contributor Author

Recompiled using Play 2.5 libraries seems to works fine.

@morellik
Copy link
Contributor Author

morellik commented Jun 8, 2016

Is it possible release a version compiled against 2.5?

@ShahBinoy
Copy link
Contributor

Submitted a PR for it #53

@morellik
Copy link
Contributor Author

morellik commented Jun 9, 2016

Thanks

@magro
Copy link
Collaborator

magro commented Jun 10, 2016

@ShahBinoy Thanks a lot for the PR! I just released the change (with some other lib updates) as 2.0.0-jongo1.3, here's the list of changes.

Please note that I didn't have time to battle test this in integration, if there's anything not working please submit an issue or update this one.

The README/Usage documentation also needs adjustments, it would be great if anybody could submit a PR - many thanks in advance!

@magro magro closed this as completed Jun 10, 2016
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

No branches or pull requests

5 participants