-
-
Notifications
You must be signed in to change notification settings - Fork 288
Introduce 'SCALA_VERSIONS' config setting #1552
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
Introduce 'SCALA_VERSIONS' config setting #1552
Conversation
f006730 to
12fe271
Compare
|
I think that build fails because of a problem with CI |
|
@simuons Can you take a look at this PR? As @mateuszkuta256 said, it focuses on adding a possibility to define all repositories and toolchains with variants for specified scala versions. This is a base for actually enabling crossbuild in next PRs |
|
@mateuszkuta256 please rebase your pr to see if it passes build. |
12fe271 to
9c7e170
Compare
|
Unfortunately CI still fails with jdk 21. I'm trying to solve that first and will get back to this PR |
|
Hi there! |
|
Closing as it was completed as part of #1290 |
Description
Extend
scala_configrepo with ascala_versionspropertyThis is the first step for cross-compilation feature. User should be able to define multiple scala versions supported by the repository:
the semantics will be:
scala_version- default version for repository. Will always fall back to this value when a rule does not specify the version explicitlyscala_versions- a list of all the versions supported by the repositoryThis change is backward-compatible.
scala_versionsis optional and default to[scala_version]This PR also registers toolchain-related repositories with a versions suffix. It is needed to avoid collisions, e.g. when
2.13.12and3.3.1are set, we will register twoio_bazel_rules_scala_scala_compiler:io_bazel_rules_scala_scala_compiler_2_13_12io_bazel_rules_scala_scala_compiler_3_3_1In the follow-up PR I'm gonna add a transition, that will select proper toolchains based on scala_version config
Motivation
Extracted from #1546