Commits on Oct 27, 2021

1. Align dependency handling with Bazel best practices …

   This commit removes the use of bind() since that function goes against
   Bazel best practices:
   https://docs.bazel.build/versions/main/external.html#repository-rules-1
   The bind() function basically maps a dependency into //external, but
   there is no good reason to do this. By mapping dependencies into
   //external and relying on this in our own BUILD files, we're forcing
   projects that depend on us to do the same. The one bind() call that I
   did leave in place was //:python_headers. This one seems to be doing
   something complicated I don't fully understand, and I don't want to risk
   breaking it.
   
   This change also moves our list of required Maven artifacts into a
   constant in protobuf_deps.bzl. This way, projects that depend on us can
   refer to this list when they invoke maven_install() and automatically
   pull in all the necesary dependencies.
   
   This fixes protocolbuffers#9132.

   

   acozzette committed Oct 27, 2021
   Copy the full SHA

   0753130 View commit details

   Browse the repository at this point in the history