Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove bazel startup options in favor of .bazelrc #163

Merged
merged 1 commit into from
Jan 15, 2025

Conversation

shYkiSto
Copy link
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
4 checks passed
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