Skip to content
This repository was archived by the owner on May 3, 2018. It is now read-only.

Build for Scala 2.11.7 and 2.12.0-M3. #13

Merged
merged 1 commit into from
Dec 8, 2015

Conversation

szeiger
Copy link
Contributor

@szeiger szeiger commented Dec 4, 2015

To be published as release 0.7.0.

To be published as release 0.7.0.
@szeiger
Copy link
Contributor Author

szeiger commented Dec 5, 2015

Review by @adriaanm

@adriaanm
Copy link
Contributor

adriaanm commented Dec 7, 2015

LGTM -- cc @SethTisue for a quick check on whether this is dbuild compatible? Something about scalaVersion := crossScalaVersions.value.head triggers my memory, but I'm not sure why.

@SethTisue
Copy link
Member

this appears to be based on what I did in this commit over in the scala-xml repo back in July.

you're right to remember that it caused community build trouble. the issue was scala/community-build#142, fixed in scala/community-build#153

but scala-xml was special because it's a circular dependency of scala; for that reason, they are built together by dbuild's "assemble" build system, and 142 was an issue specifically with that build system.

with scala-partest and scala-partest-interface, there is no circular dependency, so dbuild builds them in the usual way (with the "sbt" build system), and so I don't think 142 will be a problem here.

SethTisue added a commit that referenced this pull request Dec 8, 2015
Build for Scala 2.11.7 and 2.12.0-M3.
@SethTisue SethTisue merged commit 7c96df8 into scala:master Dec 8, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants