commit 78142a6bf3dbf802e3140c1098cf3fda8d4be883
author ccalvarin <ccalvarin@google.com> Wed Aug 01 19:25:22 2018 -0700
committer Copybara-Service <copybara-piper@google.com> Wed Aug 01 19:26:50 2018 -0700
tree 0cabf6381320a3c39fecbc14926ee1196e1182bf
parent c82074a3848e4b7340a85ce855f3d5b9fb157fd7

Add a normal startup-option for setting the digest function.

We continue to support the jvm property -Dbazel.DigestFunction, for backwards compatibility, but this will go away. The startup-option is marked experimental for now as we iron out issues. (note: leaving this out of release notes until the experimental tag is removed)

As part of this refactor, the default constructor calls for FileSystem and derived classes will now use this default. This should remove the need for constructors that accept custom hash functions.

RELNOTES: None.
PiperOrigin-RevId: 207035217

src/test/cpp/bazel_startup_options_test.cc

diff --git a/src/test/cpp/bazel_startup_options_test.cc b/src/test/cpp/bazel_startup_options_test.cc
index ff37ed4..4f0e460 100644
--- a/src/test/cpp/bazel_startup_options_test.cc
+++ b/src/test/cpp/bazel_startup_options_test.cc
@@ -93,6 +93,7 @@
   ExpectIsUnaryOption(options, "bazelrc");
   ExpectIsUnaryOption(options, "command_port");
   ExpectIsUnaryOption(options, "connect_timeout_secs");
+  ExpectIsUnaryOption(options, "digest_function");
   ExpectIsUnaryOption(options, "experimental_oom_more_eagerly_threshold");
   ExpectIsUnaryOption(options, "host_javabase");
   ExpectIsUnaryOption(options, "server_javabase");