Skip to content

Commit

Permalink
build: Do not allow PATH leakage into Bazel build
Browse files Browse the repository at this point in the history
Bug: b/294284927
Change-Id: Ifaaab94a5485fced1aa88d72115a96703a97d7eb
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/162610
Reviewed-by: Erik Gilling <[email protected]>
Presubmit-Verified: CQ Bot Account <[email protected]>
Commit-Queue: Auto-Submit <[email protected]>
Pigweed-Auto-Submit: Armando Montanez <[email protected]>
  • Loading branch information
armandomontanez authored and CQ Bot Account committed Aug 3, 2023
1 parent f6437da commit 8363537
Showing 1 changed file with 0 additions and 9 deletions.
9 changes: 0 additions & 9 deletions .bazelrc
Original file line number Diff line number Diff line change
Expand Up @@ -47,15 +47,6 @@ build --cxxopt="-Wnon-virtual-dtor"
# Allow uses of the register keyword, which may appear in C headers.
build --cxxopt="-Wno-register"

# This leaks the PATH variable into the Bazel build environment, which
# enables the Python pw_protobuf plugins to find the Python version
# via CIPD and pw_env_setup. This is a partial fix for pwbug/437,
# however this does not provide a fix for downstream projects that
# use system Python < 3.6. This approach is problematic because of the
# Protobuf rebuild sensitivity to environment variable changes.
# TODO(pwbug/437): Remove this once pwbug/437 is completely resolved.
build --action_env=PATH

build --@mbedtls//:mbedtls_config=//third_party/mbedtls:default_config

# Define the --config=asan-libfuzzer configuration.
Expand Down

0 comments on commit 8363537

Please sign in to comment.