support for scala-native #45
Conversation
MasseGuillaume
force-pushed
the
scala-native
branch
3 times, most recently
from
c10e1e9 to
3ed38d5
Compare
currently fail to link: > scala-collection-compatNative/test [warn] MiMa will NOT run because no mimaPreviousVersion is provided. [info] Linking (1002 ms) [error] cannot link: @java.lang.Class::getField_java.lang.String_java.lang.reflect.Field [error] cannot link: @java.lang.ClassLoader::loadClass_java.lang.String_java.lang.Class [error] unable to link [error] (scala-collection-compatNative/nativetest:nativeLink) unable to link [error] Total time: 2 s, completed Jun 19, 2018 3:21:42 PM We cannot use openjdk6 since maven central wont serve TLS1.0 Server access Error: Received fatal alert: protocol_version url=https://repo1.maven.org/maven2/org/scala-sbt/compiler-interface/0.13.17/compiler-interface-0.13.17.jar see https://stackoverflow.com/a/50824799/449071
MasseGuillaume
force-pushed
the
scala-native
branch
from
3ed38d5 to
e5a117f
Compare
|
ping @densh This library is a shim to cross-build between 2.11, 2.12 and 2.13. It fails to link with the usual suspects: I assume this is due to |
| @@ -1,60 +1,60 @@ | |||
| package test.scala.collection | |||
|
|
|||
| import org.junit.{Assert, Test} | |||
| import utest._ | |||
There was a problem hiding this comment.
there is no junit for Scala-Native.
|
Introducing a dependency on uTest is very dangerous. First, it prevents uTest itself from ever using scala-collection-compat. But more critically, it will introduce further delay on the republishing of the ecosystem for new versions of Scala. Now virtually everyone is going to transitively wait on uTest. It's cool that the migration of the tests was done to demonstrate that scala-collection-compat actually works with Scala Native. But given the downsides of that migration, I would advocate that, in the main build, the Scala Native version be only built and published, but not tested. We can keep a separate branch with the migrated tests, to be rebased every now and then to ensure Scala Native keeps working. |
|
@sjrd makes sense, we should compile + link on native, right now ArraySeq won't link. Eventually, a |
|
related: scala-native/scala-native#1256 |
Run the CI only on the right module
No description provided.