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

Bumping Scala version to 2.11.8 to fix SI-9375 #356

Merged
merged 5 commits into from
Apr 25, 2016

Conversation

Craigacp
Copy link
Contributor

This bumps the Scala version to 2.11.8, and removes the workaround we patched in to the java.io.Serialization of Attr objects.

It's not actually listed as being a bug fixed in Scala 2.11.8 due to a JIRA oddity, but the backport commit for it is scala/scala#4757 and I've verified the fix is in the binary from scala-lang.org.

@strubell
Copy link
Member

Could you update this PR for latest version of Factorie and update Build.scala (to update the version for sbt)? If all goes well I will happily merge

@Craigacp
Copy link
Contributor Author

Sure. The Build.scala in master currently says 1.2, should it say 1.3-SNAPSHOT?

@strubell
Copy link
Member

Oops, yes! Thanks for catching that.

On Mon, Apr 25, 2016 at 12:44 PM Adam Pocock notifications@github.com
wrote:

Sure. The Build.scala in master currently says 1.2, should it say
1.3-SNAPSHOT?


You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#356 (comment)

@Craigacp
Copy link
Contributor Author

I pushed the merge commit, so it should be good.

@strubell
Copy link
Member

Great, thanks!

@strubell strubell merged commit c9b9f81 into factorie:master Apr 25, 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

Successfully merging this pull request may close these issues.

2 participants