Skip to content

3.1.3-RC2 is not able to compile scala3-compiler from current main #14938

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.

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

Closed
Kordyjan opened this issue Apr 14, 2022 · 4 comments · Fixed by #14940
Closed

3.1.3-RC2 is not able to compile scala3-compiler from current main #14938

Kordyjan opened this issue Apr 14, 2022 · 4 comments · Fixed by #14940

Comments

@Kordyjan
Copy link
Contributor

Compiler version

3.1.3-RC2

Output

[error] -- [E007] Type Mismatch Error: /Users/pmarks/dotty/compiler/src/dotty/tools/dotc/core/tasty/TreeUnpickler.scala:126:18
[error] 126 |              |  $ex
[error]     |                  ^^
[error]     |                  Found:    (ex : Throwable)
[error]     |                  Required: dotty.tools.dotc.printing.Formatting.Shown
[error]     |
[error]     | longer explanation available when compiling with `-explain`
[error] one error found
[error] (scala3-compiler / Compile / compileIncremental) Compilation failed

Expectation

Stuff compiles!

@Kordyjan Kordyjan added itype:bug prio:blocker stat:needs minimization Needs a self contained minimization stat:needs triage Every issue needs to have an "area" and "itype" label labels Apr 14, 2022
@nicolasstucki
Copy link
Contributor

Probably introduced in #14455

@nicolasstucki nicolasstucki removed the stat:needs triage Every issue needs to have an "area" and "itype" label label Apr 14, 2022
@nicolasstucki
Copy link
Contributor

We seem to be missing a given Show[Throwable] but we have one for Show[Exception].

https://github.com/lampepfl/dotty/pull/14455/files#diff-1aeccc1856f9265f1358bb106fe44428b9aa9b4d353bd1a85b5799a00e810b9cR69

nicolasstucki added a commit to dotty-staging/dotty that referenced this issue Apr 14, 2022
@nicolasstucki nicolasstucki removed the stat:needs minimization Needs a self contained minimization label Apr 14, 2022
@dwijnand
Copy link
Member

How did this show up? The other community build?

@dwijnand
Copy link
Member

Ah, #14934 I assume.

michelou pushed a commit to michelou/scala3 that referenced this issue Apr 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants