Merge pull request #78 from lucidsoftware/upgrade-to-bazel-8

Upgrade to Bazel 8

Author: jadenPete

.bazelrc_shared

@@ -1,44 +1,63 @@
-common --announce_rc
-common --color=yes
-# TODO: remove this when we move to Bazel 8. This is the default in Bazel 8.
-common --experimental_remote_cache_eviction_retries=5
-
-# TODO: enable path mapping when we move to Bazel 8. There is a bug in 7. See:
-# https://github.com/bazelbuild/bazel/issues/23990
-# In the meantime, if you need to test path mapping you should disable multiplex
-# workers with the following build option:
-#build --noworker_multiplex
-#common --experimental_output_paths=strip
-#common --modify_execution_info=CppCompile=+supports-path-mapping
-
-build --experimental_strict_action_env
-build --javacopt="-source 21 -target 21"
+# Java options
 build --java_language_version="21"
 build --java_runtime_version="remotejdk_21"
+build --javacopt="-source 21 -target 21"
 build --tool_java_language_version="21"
 build --tool_java_runtime_version="remotejdk_21"
 
+# Other options
+build --experimental_use_hermetic_linux_sandbox
+build --experimental_worker_sandbox_hardening
+build --experimental_worker_cancellation
 build --strategy=worker,sandboxed,local
-
+build --verbose_failures
 build --worker_max_instances=4
 build --worker_sandboxing
+
 # Disable multiplex sandboxing because there is a bug that causes files which
 # should be in the sandbox to not be. Not sure if this is a Bazel bug or a rule
 # set bug. Something we need to deal with either way.
 # Once we enable path mapping, we should also be able to get rid of the outgoing
 # transition that we're currently relying on to prevent an explosion in the number
 # of builds.
 #build --experimental_worker_multiplex_sandboxing
-build --experimental_worker_cancellation
-build --verbose_failures
 
-# This is disabled as it was causing some issues with some javac targets when used
-# with path mapping. If that is resolved, then we can turn it back on.
-#build --experimental_worker_sandbox_hardening
-build --experimental_use_hermetic_linux_sandbox
+# TODO: enable path mapping when we move to Bazel 8. There is a bug in 7. See:
+# https://github.com/bazelbuild/bazel/issues/23990
+# In the meantime, if you need to test path mapping you should disable multiplex
+# workers with the following build option:
+#build --noworker_multiplex
+#common --experimental_output_paths=strip
+#common --modify_execution_info=CppCompile=+supports-path-mapping
 
-test --test_output=all
 
+common --announce_rc
+common --color=yes
 common:rules --disk_cache=.bazel_cache
 common:tests --disk_cache=../.bazel_cache
 common:tests --@rules_scala_annex//rules/scala:scala-toolchain=test_zinc_2_13
+
+# These are backwards incompatible options; we should check to see if their values have been flipped
+# when upgrading to new major Bazel version.
+common --incompatible_auto_exec_groups
+common --incompatible_autoload_externally=sh_binary # sh_binary is used by rules_jvm_external
+common --incompatible_config_setting_private_default_visibility
+common --incompatible_disable_native_repo_rules
+common --incompatible_disable_starlark_host_transitions
+common --incompatible_disable_target_provider_fields
+common --incompatible_strict_action_env
+
+# Unfortunately, this can't be enabled just yet because with it enabled, we get the following error:
+# ERROR: Traceback (most recent call last):
+#     File "/home/jpeterson/.cache/bazel/*/external/rules_java+/java/common/rules/java_runtime.bzl", line 197, column 48, in <toplevel>
+#         "hermetic_static_libs": attr.label_list(
+# Error in label_list: Illegal argument: element in 'providers' is of unexpected type. Either all elements should be providers, or all elements should be lists of providers, but got an element of type com.google.devtools.build.lib.starlarkbuildapi.core.ContextAndFlagGuardedValue$1.
+#
+#common --incompatible_stop_exporting_language_modules
+
+# This is disabled because rules_python (and possibly other rulesets we depend on) rely on
+# deprecated `Label` APIs:
+# https://github.com/bazelbuild/rules_python/blob/main/python/private/pythons_hub.bzl#L82
+#common --noincompatible_enable_deprecated_label_apis
+
+test --test_output=all

.bazelversion

@@ -1 +1 @@
-7.4.1
+8.1.0