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

setup_scala_testing_toolchain() failing when passed xxx_classpath #1706

Open
crt-31 opened this issue Feb 25, 2025 · 1 comment
Open

setup_scala_testing_toolchain() failing when passed xxx_classpath #1706

crt-31 opened this issue Feb 25, 2025 · 1 comment

Comments

@crt-31
Copy link
Contributor

crt-31 commented Feb 25, 2025

setup_scala_testing_toolchain() is failing when passing a custom set of labels as the scalatest_classpath argument.

If I pass in a classpath that has "@maven//:org_scalatest_scalatest_shouldmatchers_2_13", I am getting error:
target

'org_scalatest_scalatest_shouldmatchers_2_13_2_13_14' not declared in package.

It seems to be caused by the 'repositories()' function here:

repositories(scala_version, junit_classpath),

I believe it shouldn't be calling that repositories() mapping and just use the passed in labels as they are.

Code: master on (2/24/2025)

@mbland
Copy link
Contributor

mbland commented Feb 25, 2025

Thanks for catching this. I guess we don't have an existing test for this, and I made this change a few months ago. Given my experience at this point, I think you're right about those repositories calls in setup_scala_testing_toolchain() . They belong in scala_toolchains() somewhere, most likely, and will require a change to //scala:toolchains_repo.bzl as well.

I'm happy to try to cook up a test to reproduce this, then apply a fix.

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

2 participants