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

Update --remote_http_cache to --remote_cache #11771

Closed
wants to merge 1 commit into from

Conversation

angerson
Copy link
Contributor

bazel's help text refers only to --remote_cache, and this is the only instance of --remote_http_cache on this page. I tried a build earlier with --remote_http_cache, and nothing happened. Now I'm double-checking with --remote_cache but it certainly seems like this section is out-of-date.

`bazel`'s help text refers only to `--remote_cache`, and this is the only instance of `--remote_http_cache` on this page. I tried a build earlier with `--remote_http_cache`, and nothing happened. Now I'm double-checking with `--remote_cache` but it certainly seems like this section is out-of-date.
@meisterT
Copy link
Member

I am surprised that this is not working since it is still defined as old flag name (https://cs.opensource.google/bazel/bazel/+/master:src/main/java/com/google/devtools/build/lib/remote/options/RemoteOptions.java;l=80;drc=3e58cca5a72aa72ecfcaed6cee2393e59e284325) but it's a good change anyway.

@angerson
Copy link
Contributor Author

I think the error I was seeing is actually #11473 (I didn't notice because WARNING: Writing to Remote Cache: BulkTransferException doesn't break anything, it just doesn't upload).

@bazel-io bazel-io closed this in 981c0d4 Jul 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants