Skip to content

Commit

Permalink
Bazel: Clean up configuration options
Browse files Browse the repository at this point in the history
De-duplicate options for local and remote build configurations.
Note, though, that we don't re-use build_shared configuration for
default build configuration, e.g.: build --config=build_shared to
avoid these confusing warnings during the build, when building
non default configurations, e.g.:

$ bazel build --config=java11 java/com/google/gerrit/common:server
[...] WARNING:
option '--java_language_version' was expanded to from both option
'--config=build_shared' (source /home/davido/projects/gerrit2/.bazelrc)
and option '--config=java11' (source command line options)

Also remove unused remote-cache configuration options.

Release-Notes: skip
Change-Id: Ia557c949227b3038cbd62e4a0a3bad0f17d42218
  • Loading branch information
davido authored and avilaplana committed Feb 7, 2024
1 parent 7384258 commit efa90ce
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 27 deletions.
44 changes: 24 additions & 20 deletions .bazelrc
Original file line number Diff line number Diff line change
Expand Up @@ -3,31 +3,35 @@ build --repository_cache=~/.gerritcodereview/bazel-cache/repository
build --action_env=PATH
build --disk_cache=~/.gerritcodereview/bazel-cache/cas

# Builds using remotejdk_11, executes using remotejdk_11 or local_jdk
build:java11 --java_language_version=11
build:java11 --java_runtime_version=remotejdk_11
build:java11 --tool_java_language_version=11
build:java11 --tool_java_runtime_version=remotejdk_11
# Define configuration using remotejdk_11, executes using remotejdk_11 or local_jdk
build:build_shared --java_language_version=11
build:build_shared --java_runtime_version=remotejdk_11
build:build_shared --tool_java_language_version=11
build:build_shared --tool_java_runtime_version=remotejdk_11

# Builds using remotejdk_17, executes using remotejdk_17 or local_jdk
build --java_language_version=17
build --java_runtime_version=remotejdk_17
build --tool_java_language_version=17
build --tool_java_runtime_version=remotejdk_17
# Builds using remotejdk_11, executes using remotejdk_11 or local_jdk
# Avoid warnings for non default configurations:
# build --config=build_shared
build --java_language_version=11
build --java_runtime_version=remotejdk_11
build --tool_java_language_version=11
build --tool_java_runtime_version=remotejdk_11

# Builds and executes on RBE using remotejdk_11
build:remote11 --java_language_version=11
build:remote11 --java_runtime_version=remotejdk_11
build:remote11 --tool_java_language_version=11
build:remote11 --tool_java_runtime_version=remotejdk_11
build:remote11 --config=remote_shared
build:remote --config=remote_shared
build:remote --config=build_shared

# Define configuration using remotejdk_17, executes using remotejdk_17 or local_jdk
build:build_java17_shared --java_language_version=17
build:build_java17_shared --java_runtime_version=remotejdk_17
build:build_java17_shared --tool_java_language_version=17
build:build_java17_shared --tool_java_runtime_version=remotejdk_17

build:java17 --config=build_java17_shared

# Builds and executes on RBE using remotejdk_17
build:remote --java_language_version=17
build:remote --java_runtime_version=remotejdk_17
build:remote --tool_java_language_version=17
build:remote --tool_java_runtime_version=remotejdk_17
build:remote --config=remote_shared
build:remote17 --config=remote_shared
build:remote17 --config=build_java11_shared

# Enable strict_action_env flag to. For more information on this feature see
# https://groups.google.com/forum/#!topic/bazel-discuss/_VmRfMyyHBk.
Expand Down
7 changes: 0 additions & 7 deletions tools/remote-bazelrc
Original file line number Diff line number Diff line change
Expand Up @@ -56,10 +56,3 @@ build:remote_shared --remote_timeout=3600
# default. You can use --auth_credentials=some_file.json to use a service
# account credential instead.
build:remote_shared --google_default_credentials

# The following flags enable the remote cache so action results can be shared
# across machines, developers, and workspaces.
build:remote-cache --remote_cache=remotebuildexecution.googleapis.com
build:remote-cache --tls_enabled=true
build:remote-cache --remote_timeout=3600
build:remote-cache --auth_enabled=true

0 comments on commit efa90ce

Please sign in to comment.