Skip to content

Commit

Permalink
bazel: Update rules_go
Browse files Browse the repository at this point in the history
Updates rules_go to the latest release so Pigweed is no longer relying
on a fork.

Fixes: b/345806988
Change-Id: Ifbd5c7278f1105152b3f9b7d2e86bb0da7153b76
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/232020
Reviewed-by: Ted Pudlik <[email protected]>
Pigweed-Auto-Submit: Armando Montanez <[email protected]>
Commit-Queue: Auto-Submit <[email protected]>
Lint: Lint 🤖 <[email protected]>
Presubmit-Verified: CQ Bot Account <[email protected]>
  • Loading branch information
armandomontanez authored and CQ Bot Account committed Aug 26, 2024
1 parent 2ceef95 commit 70e52ae
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 8 deletions.
9 changes: 1 addition & 8 deletions MODULE.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ bazel_dep(name = "platforms", version = "0.0.10")
bazel_dep(name = "protobuf", version = "24.4", repo_name = "com_google_protobuf")
bazel_dep(name = "rules_cc", version = "0.0.9")
bazel_dep(name = "rules_fuzzing", version = "0.5.2")
bazel_dep(name = "rules_go", version = "0.44.2", repo_name = "io_bazel_rules_go")
bazel_dep(name = "rules_go", version = "0.49.0", repo_name = "io_bazel_rules_go")
bazel_dep(name = "rules_jvm_external", version = "6.2")
bazel_dep(name = "rules_libusb", version = "0.1.0-rc1")
bazel_dep(name = "rules_platform", version = "0.1.0")
Expand Down Expand Up @@ -68,13 +68,6 @@ git_override(
remote = "https://github.com/nanopb/nanopb.git",
)

# TODO: b/345806988 - remove this fork and update to upstream HEAD.
git_override(
module_name = "rules_go",
commit = "d5ba42f3ca0b8510526ed5df2cf5807bdba43856",
remote = "https://github.com/cramertj/rules_go.git",
)

archive_override(
module_name = "rules_rust",
integrity = "sha256-+bWb47wg0VchIADaHt6L5Dma2Gn+Q589nz/MKcTi+lo=",
Expand Down
13 changes: 13 additions & 0 deletions pw_toolchain/host_clang/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -249,6 +249,17 @@ pw_cc_flag_set(
target_compatible_with = ["@platforms//os:linux"],
)

pw_cc_flag_set(
name = "silence_cgo_warnings",
actions = [
"@pw_toolchain//actions:all_c_compiler_actions",
"@pw_toolchain//actions:all_cpp_compiler_actions",
"@pw_toolchain//actions:all_link_actions",
],
flags = ["-Wno-unused-parameter"],
requires_any_of = [":rules_go_constraint"],
)

pw_cc_feature(
name = "supports_pic",
enabled = True,
Expand Down Expand Up @@ -322,6 +333,8 @@ pw_cc_toolchain(
"@pw_toolchain//flag_sets:wnon_virtual_dtor",
"//pw_toolchain/cc:common_warnings",
"//pw_toolchain/cc:color_diagnostics",
# Must go after the general warnings that are enabled.
":silence_cgo_warnings",
] + select({
"//pw_build:kythe": [":no_unknown_warning_option"],
"//conditions:default": [],
Expand Down

0 comments on commit 70e52ae

Please sign in to comment.