by

Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • Aug 10 13:58
    pshirshov closed #206
  • Aug 10 13:58
    pshirshov commented #206
  • Aug 08 16:30

    cquiroz on libupdates

    Lib updates Signed-off-by: Car… (compare)

  • Aug 08 16:28

    cquiroz on testfx

    (compare)

  • Aug 08 16:28

    cquiroz on master

    Fix running test on the latest … Update scala versions Signed-o… Merge pull request #208 from cq… (compare)

  • Aug 08 16:28
    cquiroz closed #208
  • Aug 08 15:59
    cquiroz synchronize #208
  • Aug 08 15:59

    cquiroz on testfx

    Update scala versions Signed-o… (compare)

  • Aug 08 15:48
    cquiroz opened #208
  • Aug 08 15:19

    cquiroz on testfx

    Fix running test on the latest … (compare)

  • Aug 08 15:15

    cquiroz on testfx

    Fix running test on the latest … (compare)

  • Aug 08 15:07
    cquiroz commented #206
  • Aug 08 14:49
    pshirshov commented #206
  • Aug 07 04:32
    scala-steward opened #207
  • Aug 06 19:38
    cquiroz commented #206
  • Aug 06 13:44
    pshirshov opened #206
  • Aug 01 18:30
    scala-steward opened #205
  • Jul 24 23:50
    cquiroz closed #204
  • Jul 20 18:34
    scala-steward opened #204
  • Jul 10 21:03

    cquiroz on master

    Update scalafmt-core to 2.6.3 Merge pull request #203 from sc… (compare)

Carlos Quiroz
@cquiroz
Ah thanks for the heads up, I haven’t noted that
Matthew de Detrich
@mdedetrich
@cquiroz I have done some updating of dependencies (which actually didn't include scala-java-time) and now I am getting a
[info] > Exception: java.time.zone.ZoneRulesException: Unknown time-zone ID: Europe/Berlin
Any idea what could be causing this/
Hmm, maybe its related to cquiroz/scala-java-time#86 ?
Matthew de Detrich
@mdedetrich

I tried adding 

"io.github.cquiroz"          %%% "scala-java-locales" % "0.3.12-cldr34"`

as a dependency and it didn't work

Carlos Quiroz
@cquiroz
you’re maybe missing sbt-tzdb?
Matthew de Detrich
@mdedetrich
Thanks, that seems to have solved this problem, now I have another one which appears to be unrelated
@cquiroz Actually I found another problem, it appears that the sbt-plugin doesn't generate tzdb_java.scala for all versions of Scala
Carlos Quiroz
@cquiroz
hmm
Matthew de Detrich
@mdedetrich
i.e. if you are using crossScalaVersions I get the following error if I do +compile 
[error] source file '/Users/mdedetrich/zalando-github/logistics-common/logistics-common/js/target/scala-2.11/src_managed/main/tzdb/tzdb_java.scala' could not be found
[error] one error found
[error] (logisticsCommonJS / Compile / compileIncremental) Compilation failed
[error] Total time: 5 s, completed Oct 22, 2018 7:18:34 PM
Carlos Quiroz
@cquiroz
I haven’t tried that
so it seems to tell that it was generated
but it wasn’t?
Matthew de Detrich
@mdedetrich
It generated it for Scala 2.12.x, but not for Scala 2.11.x
Carlos Quiroz
@cquiroz
what if you compile each version separated?
Matthew de Detrich
@mdedetrich
Tried that by using set scalaVersion in ThisBuild := "2.11.12"
Carlos Quiroz
@cquiroz
or ++2.11.12
Matthew de Detrich
@mdedetrich
nope
never new that command existed though, thanks!
So I figured out why its happening
If you go into an SBT project, do ++2.11.12 and then do compile:sourceManaged it still points to 2.11
same deal with compile:resourceManaged
Which is what you are using here https://github.com/cquiroz/sbt-tzdb/blob/master/src/main/scala/io/github/sbt/tzdb/SbtTzdb.scala#L53-L54
Carlos Quiroz
@cquiroz
the generated file gos to resourceManaged
Matthew de Detrich
@mdedetrich
resourceManaged is the same
Should I ask in sbt?
Carlos Quiroz
@cquiroz
i think so, My expertise on sbt is very limited :S
Matthew de Detrich
@mdedetrich
I think your detection may be the issue
https://github.com/cquiroz/sbt-tzdb/blob/master/src/main/scala/io/github/sbt/tzdb/SbtTzdb.scala#L83
Should probably be 
IOTasks.providerFile(sourceManaged in compile, "TzdbZoneRulesProvider.scala", "java.time.zone")
Lets see if that fixes it
Carlos Quiroz
@cquiroz
i see
Matthew de Detrich
@mdedetrich
I am trying it locally to see if it fixes it, gimme 20-30 minutes (sbt is kinda slow)
Hmm now its being kinda weird, doesn't detect it in any case
Carlos Quiroz
@cquiroz
sbt is infuriating
Matthew de Detrich
@mdedetrich
Can't agree with you more
Is there a task to manually generate the resource?
Carlos Quiroz
@cquiroz
there is a single task that will generate the tzdb
Matthew de Detrich
@mdedetrich
sbt:logistics-common> show logisticsCommonTestJS:tzdbCodeGen
[info] * /Users/mdedetrich/zalando-github/logistics-common/logistics-common/js/target/scala-2.11/src_managed/main/java/time/zone/TzdbZoneRulesProvider.scala
[info] * /Users/mdedetrich/zalando-github/logistics-common/logistics-common/js/target/scala-2.11/src_managed/main/tzdb/tzdb_java.scala
[success] Total time: 0 s, completed Oct 22, 2018 7:47:45 PM
sbt:logistics-common> logisticsCommonTestJS/test
[info] Compiling 2 Scala sources to /Users/mdedetrich/zalando-github/logistics-common/logistics-common/js/target/scala-2.11/classes ...
[error] source file '/Users/mdedetrich/zalando-github/logistics-common/logistics-common/js/target/scala-2.11/src_managed/main/tzdb/tzdb_java.scala' could not be found
[error] one error found
[error] (logisticsCommonJS / Compile / compileIncremental) Compilation failed
[error] Total time: 2 s, completed Oct 22, 2018 7:47:52 PM
sbt:logistics-common>
The hell
Okay so I think I may have found an issue
This file definitely doesn't exist
That is /Users/mdedetrich/zalando-github/logistics-common/logistics-common/js/target/scala-2.11/src_managed/main/tzdb/tzdb_java.scala
Carlos Quiroz
@cquiroz
you can see that I check if the file exists and it won’t be generated in that case
But it maybe checking in the incorrect place
in this case
Matthew de Detrich
@mdedetrich
Yeah the thing is this file doesn't even exist
So its not even downloading it