Skip to content

Conversation

@svenwiegand
Copy link

Ports jackson-module-scala to scala 2.10.0-M7. This removes the dependency to scalabeans and uses scala 2.10.0-M7 reflections instead.

Please see information about current compiler issues in the README.md of my fork.

@christophercurrie
Copy link
Member

The good news is that I've isolated the single character in the code that was causing the compiler to choke (SI-6530 if you're interested in the gory detail).

The bad news is that it's going to take some time to absorb this change, firstly because one of the tests is failing, but mostly that I think that I'm going to need to support 2.9 and 2.10 for a while, and I need to figure out a strategy for doing so.

That said, I hope to have it resolved by the time 2.10 officially releases.

@svenwiegand
Copy link
Author

Thanks for the info about the current status and your further plans.

@rocketraman
Copy link

👍

@christophercurrie
Copy link
Member

Status update for those interested: I've just released 2.1.3 with multi-version support, using a private port of scalabeans, instead of 2.10 reflection, which has some bugs that will not be resolved until 2.10.1 at the earliest.

The release should be making its way to the maven repository now and available shortly.

@francisdb
Copy link

for completeness, the correct sbt dep is now:
"com.fasterxml.jackson.module" %% "jackson-module-scala" % "2.1.3"

@slorber
Copy link

slorber commented Feb 17, 2013

thanks this made my day

@christophercurrie
Copy link
Member

Final status update: 2.10 reflection is DOA. I won't be able to use Scala reflection until we're off 2.10, which won't be for a long while. Also, the code base has evolved quite a bit since this PR, and while I've kept it open as a reminder, at this point it's not going to merge as-is. @svenwiegand I appreciate the work you put into it! If only the compiler had been ready.

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.

5 participants