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

Fix semanticdb-scalac checksums for Scala 2.12.18 and 2.13.11 #1518

Merged
merged 2 commits into from
Oct 3, 2023

Conversation

scoquelin
Copy link
Contributor

@scoquelin scoquelin commented Oct 2, 2023

Description

This PR addresses current build issues regarding semanticdb-scalac checksums and Scala 2.13.x version used in examples/semanticdb that was not up-to-date

Motivation

Builds are currently failing because of :

(15:35:14) WARNING: Download from https://repo.maven.apache.org/maven2/org/scalameta/semanticdb-scalac_2.12.18/4.8.4/semanticdb-scalac_2.12.18-4.8.4.jar failed: class com.google.devtools.build.lib.bazel.repository.downloader.UnrecoverableHttpException Checksum was 11d28a73d182453454451aef4768462730f8c3e369df47b224a4ff2e943f1db7 but wanted f31614cd13b6dc5c97804aa814b6f7ad4d67124290c08d0c9296b53e46d744e0

and :

Error in fail: Scala config (2.13.6) version does not match repository version (2.13.11)

cc @crt-31

@@ -21,7 +21,7 @@ local_repository(

load("@io_bazel_rules_scala//:scala_config.bzl", "scala_config")

scala_config(scala_version = "2.13.6")
scala_config(scala_version = "2.13.11")
Copy link
Contributor Author

@scoquelin scoquelin Oct 2, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also updating examples/semanticdb to use latest Scala 2.13.x version or build fails with version mismatch

@scoquelin
Copy link
Contributor Author

scoquelin commented Oct 3, 2023

@liucijus please consider merging this PR asap as it fixes current master build

@liucijus liucijus merged commit acebcfe into bazelbuild:master Oct 3, 2023
@liucijus
Copy link
Collaborator

liucijus commented Oct 3, 2023

Thanks, @scoquelin!

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

Successfully merging this pull request may close these issues.

3 participants