Skip to content

Commit 4a46fec

Browse files
committed
Switch to non-deprecated proto rules and update necessary dependencies.
1 parent 1698f4f commit 4a46fec

File tree

9 files changed

+35
-25
lines changed

9 files changed

+35
-25
lines changed

.bazelci/presubmit.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,10 @@
22
matrix:
33
unittest_platform:
44
- ubuntu2004
5+
- ubuntu1804
56
integration_platform:
67
- ubuntu2004
8+
- ubuntu1804
79
# - macos
810
# - windows re-enable when rules_bazel_integration_test can support custom test runner on windows.
911
test_flags:

MODULE.bazel

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,6 @@ maven.install(
5252
artifacts = [
5353
"com.google.code.findbugs:jsr305:3.0.2",
5454
"junit:junit:4.13-beta-3",
55-
"com.google.protobuf:protobuf-java:3.6.0",
56-
"com.google.protobuf:protobuf-java-util:3.6.0",
5755
"com.google.guava:guava:27.1-jre",
5856
"com.google.truth:truth:0.45",
5957
"com.google.auto.service:auto-service:1.1.1",
@@ -99,7 +97,7 @@ maven.override(
9997
use_repo(maven, "kotlin_rules_maven", "unpinned_kotlin_rules_maven")
10098

10199
bazel_dep(name = "rules_pkg", version = "1.0.1")
102-
bazel_dep(name = "rules_proto", version = "7.0.2")
100+
bazel_dep(name = "protobuf", version = "29.0", repo_name = "com_google_protobuf")
103101
bazel_dep(name = "stardoc", version = "0.7.2", repo_name = "io_bazel_stardoc")
104102

105103
bazel_dep(name = "rules_testing", version = "0.5.0", dev_dependency = True)

kotlin/compiler/BUILD.release.bazel

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,5 @@ bzl_library(
2828
name = "compiler",
2929
srcs = glob(["*.bzl"]),
3030
deps = [
31-
"@rules_proto//proto:repositories",
3231
],
3332
)

src/main/kotlin/io/bazel/kotlin/builder/toolchain/BUILD.bazel

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,8 @@ kt_bootstrap_library(
3131
"//src/main/kotlin/io/bazel/kotlin/plugin:skip-code-gen-lib",
3232
"//src/main/kotlin/io/bazel/kotlin/plugin/jdeps:jdeps-gen-lib",
3333
"//src/main/protobuf:kotlin_model_java_proto",
34-
"@kotlin_rules_maven//:com_google_protobuf_protobuf_java",
35-
"@kotlin_rules_maven//:com_google_protobuf_protobuf_java_util",
34+
"@com_google_protobuf//:protobuf_java",
35+
"@com_google_protobuf//:protobuf_java_util",
3636
"@kotlin_rules_maven//:javax_inject_javax_inject",
3737
],
3838
)

src/main/protobuf/BUILD

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
load("@rules_java//java:defs.bzl", "java_library", "java_proto_library")
2-
load("@rules_proto//proto:defs.bzl", "proto_library")
1+
load("@com_google_protobuf//bazel:java_proto_library.bzl", "java_proto_library")
2+
load("@com_google_protobuf//bazel:proto_library.bzl", "proto_library")
33

44
package(default_visibility = ["//visibility:public"])
55

src/main/starlark/core/repositories/BUILD.release.bazel

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,6 @@ bzl_library(
1919
srcs = glob(["*.bzl"]),
2020
visibility = ["//visibility:public"],
2121
deps = [
22-
"@rules_proto//proto:repositories",
22+
"@com_google_protobuf//proto:repositories",
2323
],
2424
)

src/main/starlark/core/repositories/download.bzl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ def kt_download_local_dev_dependencies():
2323
Must be called before setup_dependencies in the WORKSPACE.
2424
"""
2525
versions.use_repository(
26-
name = "rules_proto",
26+
name = "com_google_protobuf",
2727
version = versions.RULES_PROTO,
2828
rule = rules_stardoc_repository,
2929
starlark_packages = [

src/main/starlark/core/repositories/setup.bzl

Lines changed: 14 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,21 +14,32 @@
1414

1515
load("@bazel_skylib//:workspace.bzl", "bazel_skylib_workspace")
1616
load("@cgrindel_bazel_starlib//:deps.bzl", "bazel_starlib_dependencies")
17+
load("@com_google_protobuf//:protobuf_deps.bzl", "PROTOBUF_MAVEN_ARTIFACTS", "protobuf_deps")
18+
19+
#load("@com_google_protobuf//:protobuf_extra_deps.bzl", "protobuf_extra_deps")
1720
load("@io_bazel_stardoc//:setup.bzl", "stardoc_repositories")
1821
load("@rules_bazel_integration_test//bazel_integration_test:deps.bzl", "bazel_integration_test_rules_dependencies")
1922
load("@rules_cc//cc:repositories.bzl", "rules_cc_dependencies", "rules_cc_toolchains")
2023
load("@rules_java//java:repositories.bzl", "rules_java_dependencies", "rules_java_toolchains")
2124
load("@rules_jvm_external//:defs.bzl", "maven_install")
2225
load("@rules_pkg//:deps.bzl", "rules_pkg_dependencies")
23-
load("@rules_proto//proto:repositories.bzl", "rules_proto_dependencies", "rules_proto_toolchains")
2426

2527
def kt_configure():
2628
"""Setup dependencies. Must be called AFTER kt_download_local_dev_dependencies() """
2729
rules_cc_dependencies()
2830
rules_cc_toolchains()
2931

30-
rules_proto_dependencies()
31-
rules_proto_toolchains()
32+
protobuf_deps()
33+
maven_install(
34+
name = "protobuf_maven",
35+
fetch_sources = True,
36+
artifacts = PROTOBUF_MAVEN_ARTIFACTS,
37+
repositories = [
38+
"https://maven-central.storage.googleapis.com/repos/central/data/",
39+
"https://maven.google.com",
40+
"https://repo1.maven.org/maven2",
41+
],
42+
)
3243

3344
rules_java_dependencies()
3445
rules_java_toolchains()

src/main/starlark/core/repositories/versions.bzl

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -29,16 +29,16 @@ versions = struct(
2929
# 1. Download archive
3030
# 2. Download dependencies and Configure rules
3131
# --> 3. Configure dependencies <--
32-
SKYLIB_VERSION = "1.4.2",
33-
SKYLIB_SHA = "66ffd9315665bfaafc96b52278f57c7e2dd09f5ede279ea6d39b2be471e7e3aa",
32+
SKYLIB_VERSION = "1.7.0",
33+
SKYLIB_SHA = "d00f1389ee20b60018e92644e0948e16e350a7707219e7a390fb0a99b6ec9262",
3434
RULES_JVM_EXTERNAL_TAG = "5.3",
3535
RULES_JVM_EXTERNAL_SHA = "d31e369b854322ca5098ea12c69d7175ded971435e55c18dd9dd5f29cc5249ac",
3636
RULES_PROTO = version(
37-
version = "5.3.0-21.7",
38-
sha256 = "dc3fb206a2cb3441b485eb1e423165b231235a1ea9b031b4433cf7bc1fa460dd",
39-
strip_prefix_template = "rules_proto-{version}",
37+
version = "29.3",
38+
sha256 = "",
39+
strip_prefix_template = "protobuf-{version}",
4040
url_templates = [
41-
"https://github.com/bazelbuild/rules_proto/archive/refs/tags/{version}.tar.gz",
41+
"https://github.com/protocolbuffers/protobuf/archive/v{version}.tar.gz",
4242
],
4343
),
4444
IO_BAZEL_STARDOC = version(
@@ -101,19 +101,19 @@ versions = struct(
101101
),
102102
# needed for rules_pkg and java
103103
RULES_JAVA = version(
104-
version = "7.2.0",
104+
version = "8.3.2",
105105
url_templates = [
106106
"https://github.com/bazelbuild/rules_java/releases/download/{version}/rules_java-{version}.tar.gz",
107107
],
108-
sha256 = "eb7db63ed826567b2ceb1ec53d6b729e01636f72c9f5dfb6d2dfe55ad69d1d2a",
108+
sha256 = "9b9614f8a7f7b7ed93cb7975d227ece30fe7daed2c0a76f03a5ee37f69e437de",
109109
),
110110
RULES_LICENSE = version(
111-
version = "0.0.3",
111+
version = "1.0.0",
112112
url_templates = [
113113
"https://mirror.bazel.build/github.com/bazelbuild/rules_license/releases/download/{version}/rules_license-{version}.tar.gz",
114114
"https://github.com/bazelbuild/rules_license/releases/download/{version}/rules_license-{version}.tar.gz",
115115
],
116-
sha256 = None,
116+
sha256 = "26d4021f6898e23b82ef953078389dd49ac2b5618ac564ade4ef87cced147b38",
117117
),
118118
RULES_TESTING = version(
119119
version = "0.6.0",
@@ -158,9 +158,9 @@ versions = struct(
158158
],
159159
),
160160
RULES_CC = version(
161-
version = "0.0.9",
161+
version = "0.0.15",
162162
url_templates = ["https://github.com/bazelbuild/rules_cc/releases/download/{version}/rules_cc-{version}.tar.gz"],
163-
sha256 = "2037875b9a4456dce4a79d112a8ae885bbc4aad968e6587dca6e64f3a0900cdf",
163+
sha256 = "f4aadd8387f381033a9ad0500443a52a0cea5f8ad1ede4369d3c614eb7b2682e",
164164
strip_prefix_template = "rules_cc-{version}",
165165
),
166166
use_repository = _use_repository,

0 commit comments

Comments
 (0)