Skip to content

community build status #108

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

Closed
lrytz opened this issue Mar 24, 2016 · 7 comments
Closed

community build status #108

lrytz opened this issue Mar 24, 2016 · 7 comments
Assignees

Comments

@lrytz
Copy link
Member

lrytz commented Mar 24, 2016

I ran a community build against 2.12.x after merging the flat-classpath-by-default PR (scala/scala@952da60)

Logs

Failures: acyclic and shapeless play-twirl

acyclic

There seems to be a source-incompatibility with utest. We are tracking the master branches for both projects (see https://github.com/scala/community-builds/blob/2.12.x/common.conf#L92-L93). Before diving into details, maybe @lihaoyi will just say "oh, right"?

[acyclic] [error] /home/jenkins/workspace/scala-2.12.x-integrate-community-build/target-0.9.5/project-builds/acyclic-1d868ca75d00f4a1d93435c88e0dae97cd16c902/src/test/scala/acyclic/TestUtils.scala:62: not found: value intercept
[acyclic] [error]     val ex = intercept[CompilationException]{ make(path) }
[acyclic] [error]              ^
[acyclic] [warn] one warning found
[acyclic] [error] one error found
[acyclic] [error] (test:compileIncremental) Compilation failed

play-twirl

10 failed tests, for example

[play-twirl] [info] New twirl parser should
[play-twirl] [info]   succeed for
[play-twirl] [error]     x static.scala.html
[play-twirl] [error]      'Success(Template(,None,(),ArrayBuffer(),ArrayBuffer(),ArrayBuffer(),ArrayBuffer(),ArrayBuffer(Plain(<h1>It works</h1>))),Input())' is incorrect:  (ParserSpec.scala:30)

The test succeeds for case Success(_, rest) if rest.atEnd. It looks like rest is Input() in this case.

@lrytz
Copy link
Member Author

lrytz commented Mar 24, 2016

cc @pvlugter - any idea about the play-twirl test failures?

@lihaoyi
Copy link

lihaoyi commented Mar 24, 2016

@lrytz oh right, yeah it's a source incompatibility. I don't have time to look now but maybe over the weekend

@pvlugter
Copy link

@lrytz, sorry, most of this was just copied out of play into a separate repo. Looks like @adriaanm has fixed already by using the same parser instance.

@SethTisue SethTisue self-assigned this Mar 30, 2016
@SethTisue
Copy link
Member

I'm on this now, it's reached the front of my queue.

@SethTisue
Copy link
Member

Adriaan's test run with acyclic and play-twirl pinned: https://scala-ci.typesafe.com/job/scala-2.12.x-integrate-community-build/354/

@SethTisue
Copy link
Member

Adriaan's notes on his wip changes to the 2.12 community build: scala/scala#4971 (comment)

@SethTisue
Copy link
Member

superseded by #203

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

4 participants