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.

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

Scala 3 / Dotty Support #55

Closed
9 tasks done
ekrich opened this issue Nov 5, 2019 · 6 comments
Closed
9 tasks done

Scala 3 / Dotty Support #55

ekrich opened this issue Nov 5, 2019 · 6 comments

Comments

@ekrich
Copy link
Owner

ekrich commented Nov 5, 2019

Blockers

This issue is a overall issue to keep track of Dotty support. Different PRs are being worked on and merged to master as long as the changes are backward compatible.

Fix was merged in 0.20.0-RC1 so we have serialization but the output is not the same.

  • Investigate serialization difference. These seem to be caused by the inability to assign an Enum to a value running into the following Bad type operand on stack errors or the like.

Blocked on the following issues.

Update: 9-2-2020 - Dotty 0.27.0-RC1 - One test fails - org.ekrich.config.impl.ValidationTest.validationFailedSerializable
Error: java.io.NotSerializableException: org.ekrich.config.impl.SimpleConfigOrigin
Ignored for now - needs investigating.

This is currently not passing the tests due to a Callable[Config] that produces a value correctly but does not get assigned correctly. This call returns a config - https://github.com/ekrich/sconfig/blob/topic/dotty2/sconfig/shared/src/main/scala/org/ekrich/config/impl/ConfigImpl.scala#L54 when debugged from here - https://github.com/ekrich/sconfig/blob/topic/dotty2/sconfig/shared/src/main/scala/org/ekrich/config/impl/ConfigImpl.scala#L353-L365 - but then always falls through to this code - https://github.com/ekrich/sconfig/blob/topic/dotty2/sconfig/shared/src/main/scala/org/ekrich/config/impl/ConfigImpl.scala#L65-L66

The above has been reported and considered.

This code now has been modified to avoid this problem so the code will work and test. See 2ae7d88

Java Blockers

These are lower priority as Java can use the Java API but these keep all the functionality from working.

@ekrich
Copy link
Owner Author

ekrich commented Nov 5, 2019

First PR. #48

@ekrich
Copy link
Owner Author

ekrich commented Nov 5, 2019

Second PR. #49

@ekrich
Copy link
Owner Author

ekrich commented Dec 24, 2019

Third and fourth: #62 and #65
Modified the build so a subset of the tests will compile. Include fixes for non-local returns which removes the warnings.

@ekrich
Copy link
Owner Author

ekrich commented Jul 14, 2020

Vararg will be fixed on the next release of Dotty. See #85

@ekrich
Copy link
Owner Author

ekrich commented Aug 10, 2020

This issue scala/scala3#7312 is now fixed so the Java files need to be reverted. Fix check mark above when completed.
The 20200810 nightly or later can be tested.

@ekrich
Copy link
Owner Author

ekrich commented Sep 16, 2020

There is one serialization test failing now based on changes in this PR - #92

Once this has been fixed and the pull merged, this issue can be closed.

@ekrich ekrich changed the title Dotty Support Scala 3 / Dotty Support Oct 30, 2020
@ekrich ekrich closed this as completed in 4219fe0 Nov 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant