Skip to content

Remove bazel startup options in favor of .bazelrc#163

Merged
shYkiSto merged 1 commit intouber-web:mainfrom
shYkiSto:fix/bazel-startup
Jan 15, 2025
Merged

Remove bazel startup options in favor of .bazelrc#163
shYkiSto merged 1 commit intouber-web:mainfrom
shYkiSto:fix/bazel-startup

Conversation

@shYkiSto
Copy link
Copy Markdown
Collaborator

@shYkiSto shYkiSto commented Dec 20, 2024

Remove default startup options from bazel invocations in favor of the user/project configuration specified in the .bazelrc. This avoids introducing changes that would cause the server to restart between bazel invocations, reducing the likelihood of cache invalidation. E.g.,

# `jz changes` command invokes bazel w/ different startup options (main invocation vs. subsequent `bazel query`)
% echo "WORKSPACE" | jz changes
...
# Running build after `jz changes` will require restarting the server, as startup options are different from the `bazel query`, which invalidates all analysis cache
% jz noop
WARNING: Running Bazel server needs to be killed, because the startup options are different.
Starting local Bazel server and connecting to it...

Example .bazelrc:

startup --host_jvm_args="-Xmx15g"

@shYkiSto shYkiSto changed the title Remove implicit startup options in favor of .bazelrc Remove custom startup options in favor of .bazelrc Dec 20, 2024
@shYkiSto shYkiSto changed the title Remove custom startup options in favor of .bazelrc Remove bazel startup options in favor of .bazelrc Dec 20, 2024
@shYkiSto shYkiSto merged commit 95deb90 into uber-web:main Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants