Skip to content

Commit

Permalink
Merge #58891 #58893
Browse files Browse the repository at this point in the history
58891: *: upgrade vendored protobuf to v3.9.2 r=rickystewart a=rickystewart

The old version of `protobuf` that we had doesn't contain some utilities
that we need for Bazel (namely, the file `protobuf_deps.bzl`), so move
to a more recent version. Also update `WORKSPACE` so that our Bazel
build uses the same version of `protobuf`, and point to our fork rather
than upstream.

Fixes #58227.

Release note (general change): Upgrade to v3.9.2 of `protobuf` to consume new changes for Bazel.

58893: build: assert minimum bazel version in use r=rickystewart a=rickystewart

Use `bazel_skylib`'s built-in functionality to assert the minimum
supported Bazel version. We could also use `.bazelversion` to specify an
*exact* version that must be used, but that seems unnecessarily
restrictive.

v3.5.0 was arbitrarily chosen as being probably "recent enough", but I
did verify that a build does work with that version of Bazel.

Fixes #56059.

Release note: None

Co-authored-by: Ricky Stewart <[email protected]>
  • Loading branch information
craig[bot] and rickystewart committed Jan 14, 2021
3 parents 80e6094 + f6b6675 + dadd5f1 commit 690c122
Show file tree
Hide file tree
Showing 94 changed files with 1,146 additions and 1,141 deletions.
9 changes: 7 additions & 2 deletions WORKSPACE
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,11 @@ go_rules_dependencies()

go_register_toolchains(go_version = "1.15.6")

# NB: @bazel_skylib comes from go_rules_dependencies().
load("@bazel_skylib//lib:versions.bzl", "versions")

versions.check(minimum_bazel_version = "3.5.0")

# Load gazelle dependencies.
load("@bazel_gazelle//:deps.bzl", "gazelle_dependencies")

Expand All @@ -93,8 +98,8 @@ gazelle_dependencies()
# https://github.com/bazelbuild/bazel-gazelle/issues/591
git_repository(
name = "com_google_protobuf",
commit = "09745575a923640154bcf307fba8aedff47f240a",
remote = "https://github.com/protocolbuffers/protobuf",
commit = "9b23a34c7275aa0ceb2fc69ed1ae6737b34656a3",
remote = "https://github.com/cockroachdb/protobuf",
)

load("@com_google_protobuf//:protobuf_deps.bzl", "protobuf_deps")
Expand Down
2 changes: 1 addition & 1 deletion c-deps/protobuf
Submodule protobuf updated 1435 files
12 changes: 6 additions & 6 deletions pkg/acceptance/cluster/testconfig.pb.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

28 changes: 14 additions & 14 deletions pkg/blobs/blobspb/blobs.pb.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 3 additions & 3 deletions pkg/build/info.pb.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

26 changes: 13 additions & 13 deletions pkg/ccl/backupccl/backup.pb.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

10 changes: 5 additions & 5 deletions pkg/ccl/baseccl/encryption_options.pb.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 690c122

Please sign in to comment.