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

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