Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

assertion failed: duplicate type F#7718; previous was type F#7369 (using Scala 2 published cats-effect) #11984

Closed
scf37 opened this issue Apr 4, 2021 · 2 comments · Fixed by #12120
Closed

assertion failed: duplicate type F#7718; previous was type F#7369 (using Scala 2 published cats-effect) #11984

scf37 opened this issue Apr 4, 2021 · 2 comments · Fixed by #12120
Labels
area:pickling compat:scala2 itype:bug stat:duplicate

Comments

@scf37
Copy link

scf37 commented Apr 4, 2021 •
edited
Loading

Compiler version

3.0.0-RC1 and 3.0.0-RC2

Minimized code & Output

https://scastie.scala-lang.org/EzwUElRQS2uavuyN6TU4ow

Another one:
https://scastie.scala-lang.org/Kx6GbNFJRPenigB5LiOdxg

Expectation

Successful compilation
@scf37 scf37 added the itype:bug label Apr 4, 2021
@griggt
Copy link
Collaborator

griggt commented Apr 5, 2021

Both examples seem to work fine when depending on the Scala 3 published cats-effect artifact instead of using .withDottyCompat to get the Scala 2 artifact.

Aside: is there some reason that Scastie uses Scala 2 libraries and .withDottyCompat in Scala 3 build mode, even when Scala 3 published versions are available? This doesn't always work and can be quite confusing, as seen here and in this recent issue on gitter re: ScalaTest: https://gitter.im/lampepfl/dotty?at=6067745692a3431fd66fa63b

@griggt griggt changed the title assertion failed: duplicate type F#7718; previous was type F#7369 assertion failed: duplicate type F#7718; previous was type F#7369 (using Scala 2 published cats-effect) Apr 5, 2021
@griggt
Copy link
Collaborator

griggt commented Apr 5, 2021

Duplicate of #11173

@griggt griggt marked this as a duplicate of #11173 Apr 5, 2021
@odersky odersky mentioned this issue Apr 5, 2021
Closed
smarter pushed a commit to dotty-staging/dotty that referenced this issue Apr 16, 2021
@odersky @smarter
Scala2Unpickler: don't unpickle the same type parameter twice
1d4d279 
Fixes scala#11173.
Fixes scala#11984.
@smarter smarter mentioned this issue Apr 16, 2021
Merged
smarter added a commit to dotty-staging/dotty that referenced this issue Apr 16, 2021
@smarter
Scala2Unpickler: don't unpickle the same type parameter twice
0859084 
Fixes scala#11173.
Fixes scala#11984.
smarter added a commit to dotty-staging/dotty that referenced this issue Apr 17, 2021
@smarter
Scala2Unpickler: don't unpickle the same type parameter twice
f40070a 
Fixes scala#11173.
Fixes scala#11984.
michelou pushed a commit to michelou/dotty that referenced this issue Apr 17, 2021
@smarter @michelou
Scala2Unpickler: don't unpickle the same type parameter twice
f41859b 
Fixes scala#11173.
Fixes scala#11984.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area:pickling compat:scala2 itype:bug stat:duplicate
Projects
None yet
Milestone
No milestone
2 participants
@scf37 @griggt