Merge "Revert "skip soong tests by default for b command""
This commit is contained in:
29
envsetup.sh
29
envsetup.sh
@@ -1838,17 +1838,10 @@ function _trigger_build()
|
|||||||
# Convenience entry point (like m) to use Bazel in AOSP.
|
# Convenience entry point (like m) to use Bazel in AOSP.
|
||||||
function b()
|
function b()
|
||||||
(
|
(
|
||||||
# Look for the --run-soong-tests flag and skip passing --skip-soong-tests to Soong if present
|
|
||||||
local run_tests=$(echo "$@" | grep -ow -- "--run-soong-tests")
|
|
||||||
local bazel_args=(${@/--run-soong-tests})
|
|
||||||
local skip_tests="--skip-soong-tests"
|
|
||||||
if [[ -n $run_tests ]]; then
|
|
||||||
skip_tests=""
|
|
||||||
fi
|
|
||||||
# Generate BUILD, bzl files into the synthetic Bazel workspace (out/soong/workspace).
|
# Generate BUILD, bzl files into the synthetic Bazel workspace (out/soong/workspace).
|
||||||
_trigger_build "all-modules" bp2build USE_BAZEL_ANALYSIS= $skip_tests || return 1
|
_trigger_build "all-modules" bp2build USE_BAZEL_ANALYSIS= || return 1
|
||||||
# Then, run Bazel using the synthetic workspace as the --package_path.
|
# Then, run Bazel using the synthetic workspace as the --package_path.
|
||||||
if [[ -z "$bazel_args" ]]; then
|
if [[ -z "$@" ]]; then
|
||||||
# If there are no args, show help.
|
# If there are no args, show help.
|
||||||
bazel help
|
bazel help
|
||||||
else
|
else
|
||||||
@@ -1856,23 +1849,17 @@ function b()
|
|||||||
# Add the --config=bp2build after the first argument that doesn't start with a dash. That should be the bazel
|
# Add the --config=bp2build after the first argument that doesn't start with a dash. That should be the bazel
|
||||||
# command. (build, test, run, ect) If the --config was added at the end, it wouldn't work with commands like:
|
# command. (build, test, run, ect) If the --config was added at the end, it wouldn't work with commands like:
|
||||||
# b run //foo -- --args-for-foo
|
# b run //foo -- --args-for-foo
|
||||||
local pre_config_args=""
|
local previous_args=""
|
||||||
local post_config_args=""
|
for arg in $@;
|
||||||
local start_post_config_args=0
|
|
||||||
for arg in $bazel_args;
|
|
||||||
do
|
do
|
||||||
if [[ $start_post_config_args -eq 0 ]]; then
|
previous_args+="$arg "
|
||||||
pre_config_args+="$arg "
|
shift
|
||||||
else
|
|
||||||
post_config_args+="$arg "
|
|
||||||
fi
|
|
||||||
|
|
||||||
if [[ $arg != -* ]]; # if $arg doesn't start with a dash
|
if [[ $arg != -* ]]; # if $arg doesn't start with a dash
|
||||||
then
|
then
|
||||||
start_post_config_args=1
|
break
|
||||||
fi
|
fi
|
||||||
done
|
done
|
||||||
eval "bazel $pre_config_args --config=bp2build $post_config_args"
|
bazel $previous_args --config=bp2build $@
|
||||||
fi
|
fi
|
||||||
)
|
)
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user