Skip to content
This repository has been archived by the owner on Jun 6, 2018. It is now read-only.

WIP: Add sbt 1.1.x support #35

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

periklis
Copy link

@periklis periklis commented Jan 29, 2018

@teozkr thx for the guidance and since i am on that path i decided to push this forward later this week.
Current state of affairs #34 :

[info] Compiling 7 Scala sources to /home/username/projects/github/Sbtix/plugin/target/scala-2.12/sbt-1.0/classes ...
[error] /home/username/projects/github/Sbtix/plugin/src/main/scala/se/nullable/sbtix/CoursierArtifactFetcher.scala:22:41: type mismatch;
[error]  found   : coursier.Dependency
[error]     (which expands to)  coursier.core.Dependency
[error]  required: (coursier.Dependency, Map[String,String])
[error]     (which expands to)  (coursier.core.Dependency, Map[String,String])
[error]   private val moduleId = ToSbt.moduleId(dep)
[error]                                         ^
[error] /home/username/projects/github/Sbtix/plugin/src/main/scala/se/nullable/sbtix/NixResolver.scala:19:17: not found: type JavaNet1Repository
[error]       case jn1: JavaNet1Repository => ???
[error]                 ^
[error] /home/username/projects/github/Sbtix/plugin/src/main/scala/se/nullable/sbtix/NixPlugin.scala:21:116: too many arguments (4) for method dependencies: (module: sbt.ModuleID, scalaVersion: Strin
g, scalaBinaryVersion: String)Seq[(String, coursier.Dependency)]                                                                                                                                          
[error]       val depends = modules.flatMap(coursier.FromSbt.dependencies(_, scalaVersion.value, scalaBinaryVersion.value, "jar")).map(_._2)
[error]                                                                                                                    ^
[error] three errors found
[error] (Compile / compileIncremental) Compilation failed

to be continued

@periklis
Copy link
Author

periklis commented Feb 1, 2018

@teozkr I had some time today to fix the build. The current version just compiles and has zero deprecation warnings, nothing more. I am actually stuck with a local error with the ensime global plugin, hence i could not proceed with sbt genNix "reload plugins" genNix.

@nightkr
Copy link
Owner

nightkr commented Feb 1, 2018

Yeah, you should probably disable Ensime while running genNix by hand. Normally sbtix-gen-* is set up to ignore global plugins, but that becomes a bit circular when there isn't a packaged Sbtix for that version yet.

@periklis
Copy link
Author

periklis commented Feb 2, 2018

Actually coursier changed (See CachedPath) a little bit and it caches paths and locks in files with a dot prepended, which breaks with nix, because its an names with dots are illegal.

[info] Set current project to sbtix (in build file:/home/username/projects/github/Sbtix/plugin/)
file:/home/username/.coursier/cache/v1/https/repo1.maven.org/maven2/org/scala-lang/scala-compiler/2.12.4/scala-compiler-2.12.4.pom
-Meta- Artifact https://repo1.maven.org/maven2/org/scala-lang/scala-compiler/2.12.4/scala-compiler-2.12.4.pom => ‘/nix/store/yvpw000523
i665ls7l9369qxv7brl8iw-scala-compiler-2.12.4.pom’                                                                                     
file:/home/username/.coursier/cache/v1/https/repo1.maven.org/maven2/org/scala-lang/scala-compiler/2.12.4/.directory
error: illegal name: ‘.directory’
[error] java.lang.RuntimeException: Nonzero exit value: 1
[error]         at scala.sys.package$.error(package.scala:27)
[error]         at scala.sys.process.ProcessBuilderImpl$AbstractBuilder.slurp(ProcessBuilderImpl.scala:134)
[error]         at scala.sys.process.ProcessBuilderImpl$AbstractBuilder.$bang$bang(ProcessBuilderImpl.scala:105)
[error]         at se.nullable.sbtix.FindArtifactsOfRepo$.fetchChecksum(FindArtifacts.scala:29)
[error]         at se.nullable.sbtix.CoursierArtifactFetcher.$anonfun$CacheFetch_WithCollector$3(CoursierArtifactFetcher.scala:189)

@nightkr
Copy link
Owner

nightkr commented Jun 6, 2018

The project has moved to GitLab: https://gitlab.com/teozkr/Sbtix/merge_requests/35

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.

2 participants