Skip to content

Update scala to version 2.10.5#1980

Closed
ajozwik wants to merge 2 commits intosbt:0.13from
ajozwik:0.13
Closed

Update scala to version 2.10.5#1980
ajozwik wants to merge 2 commits intosbt:0.13from
ajozwik:0.13

Conversation

@ajozwik
Copy link
Contributor

@ajozwik ajozwik commented Apr 19, 2015

See #1666, scala/scala#4186 solved problem with xml groups.

@typesafe-tools
Copy link

Can one of the admins verify this patch?

@eed3si9n
Copy link
Member

Can you run scripted tests locally?

@eed3si9n
Copy link
Member

The last Scala version bump happened on #1212. This didn't require change to scripted as far as I can tell.

@ajozwik
Copy link
Contributor Author

ajozwik commented Apr 22, 2015

I can rollback two last commits and you will see the result.
On my local machine first I call sbt ";publishLocal;project bundledLauncherProj;publishLocal;project precompiled-2_9_2;publishLocal;project precompiled-2_8_2;publishLocal;project precompiled-2_9_3;publishLocal" and then sbt -Dsbt.version=0.13.9-SNAPSHOT <scripted name>

@ajozwik
Copy link
Contributor Author

ajozwik commented Apr 23, 2015

Failed test - problem with bintray ...

My local result:

About to run tests: 
 * compiler-project/*

Running compiler-project / semantic-errors
Running compiler-project / inc-pickled-existential
Running compiler-project / macro-config
Running compiler-project / separate-analysis-per-scala
Running compiler-project / src-dep-plugin
Running compiler-project / error-in-invalidated
Running compiler-project / run2.8
....

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this is required, we should fix the build script instead of putting it in .travis.yml so all required JARs are published locally before the scripted tests.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It will be required only - when 0.13.9 will be published. After it the .travis script can be restored.
Please checkout the code beaf39a and run the tests. You will receive: errors, see all the builds in: https://travis-ci.org/sbt/sbt/builds/54919087:

....
About to run tests:

* actions/*

java.lang.NoSuchMethodException: sbt.test.ScriptedTests$.run(java.io.File, boolean, [Ljava.lang.String;, java.io.File, [Ljava.lang.String;, scala.Function1)

at java.lang.Class.getMethod(Class.java:1667)

at Scripted$.doScripted(Scripted.scala:66)

at $0e28670edea040be68f9$$anonfun$scriptedTask$2$$anonfun$apply$2$$anonfun$apply$3.apply(build.sbt:474)

at $0e28670edea040be68f9$$anonfun$scriptedTask$2$$anonfun$apply$2$$anonfun$apply$3.apply(build.sbt:471)

at scala.Function1$$anonfun$compose$1.apply(Function1.scala:47)
...

I guess it is problem with classloader. Mentioned in stack method exist.
As the solution the new process should be run - by scala 2.10.5 - and then all is run by the same classloader.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, I ran into this. The way we use scripted is a bit gunky now. We actually no longer have it isolated, so we need to fix it.

@jsuereth
Copy link
Member

@eed3si9n We should figure out how to merge this before 0.13.9

@jsuereth
Copy link
Member

Ok, so we bumped to 2.11.6, what should we do with this PR?

@ajozwik
Copy link
Contributor Author

ajozwik commented May 20, 2015

You can jest close without merging. 2.11.6 contains the same correction

@eed3si9n
Copy link
Member

@jsuereth we bumped what to what?

eed3si9n added a commit that referenced this pull request Jun 19, 2015
To pass File => Unit callback across the classloader boundary
I am encoding it as a java.util.List[File] by overriding
 method.
This was needed since Java didn't allow me to cast
from one classloader to the other.
@eed3si9n eed3si9n self-assigned this Jun 19, 2015
@eed3si9n
Copy link
Member

Closing this in favor of #2068

@eed3si9n eed3si9n closed this Jun 20, 2015
eed3si9n added a commit that referenced this pull request Jun 20, 2015
Bumping up Scala version to 2.10.5/2.11.6. Fixes #1666, Ref #1980
adriaanm pushed a commit to adriaanm/sbt that referenced this pull request Dec 15, 2015
To pass File => Unit callback across the classloader boundary
I am encoding it as a java.util.List[File] by overriding
 method.
This was needed since Java didn't allow me to cast
from one classloader to the other.
adriaanm pushed a commit to adriaanm/sbt that referenced this pull request Dec 17, 2015
To pass File => Unit callback across the classloader boundary
I am encoding it as a java.util.List[File] by overriding
 method.
This was needed since Java didn't allow me to cast
from one classloader to the other.
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.

4 participants