update-go-pbs.sh: fixing script for mac users & updating bazel dependency (#11937)

* fixing script for mac users

* updating checksum for bazel tool

---------

Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
This commit is contained in:
james-prysm 2023-01-30 19:17:12 -06:00 committed by GitHub
parent 950d0c0282
commit 53a135adbd
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 9 additions and 2 deletions

View File

@ -69,7 +69,7 @@ http_archive(
http_archive( http_archive(
name = "com_github_atlassian_bazel_tools", name = "com_github_atlassian_bazel_tools",
sha256 = "60821f298a7399450b51b9020394904bbad477c18718d2ad6c789f231e5b8b45", sha256 = "e783be1fca064c2eaa8eb3770ab4f5901abdfa85a0268dbfb2cfe6dc8b171dab",
strip_prefix = "bazel-tools-a2138311856f55add11cd7009a5abc8d4fd6f163", strip_prefix = "bazel-tools-a2138311856f55add11cd7009a5abc8d4fd6f163",
urls = ["https://github.com/atlassian/bazel-tools/archive/a2138311856f55add11cd7009a5abc8d4fd6f163.tar.gz"], urls = ["https://github.com/atlassian/bazel-tools/archive/a2138311856f55add11cd7009a5abc8d4fd6f163.tar.gz"],
) )

View File

@ -1,3 +1,10 @@
# Bash Scripts # Bash Scripts
This subproject contains useful bash scripts for working with our repository. We have a simple tool that outputs coverage, a simple tool to check for gazelle requirements, update Go protobuf generated files, visibility rules tools for Bazel packages, and more. This subproject contains useful bash scripts for working with our repository. We have a simple tool that outputs coverage, a simple tool to check for gazelle requirements, update Go protobuf generated files, visibility rules tools for Bazel packages, and more.
## update-go-pbs.sh
This script generates the *.pb.go and *.pb.gw.go files from the *.proto files.
After running `update-go-pbs.sh` keep only the *.pb.go and *pb.gw.go for the protos that have changed before checking in.
*Note*: the generated files may not have imports correctly linted and will need to be fixed to remote associated errors.

View File

@ -23,7 +23,7 @@ readonly system
findutil="find" findutil="find"
# On OSX `find` is not GNU find compatible, so require "findutils" package. # On OSX `find` is not GNU find compatible, so require "findutils" package.
if [ "$system" == "darwin" ]; then if [ "$system" == "darwin" ]; then
if [[ ! -x "/usr/local/bin/gfind" ]]; then if [[ ! -x "/usr/local/bin/gfind" && ! -x "/opt/homebrew/bin/gfind" ]]; then
color 31 "Make sure that GNU 'findutils' package is installed: brew install findutils" color 31 "Make sure that GNU 'findutils' package is installed: brew install findutils"
exit 1 exit 1
else else