build: Switch to Bazel 8.0.1 for development and default testing

We drop the dev dependency to stardoc, which we do not actually need.

We stop using `--nolegacy_external_runfiles` as it is the default with Bazel 8.
Strictly speaking this reduces the performance in some aspect integration tests.
But given the difference is minimal, we prefer an overall clean and consistent
setup not mentioning this flag.

Author: martis42

.bazelrc

@@ -1,9 +1,5 @@
 # Always tell why tests fail
 test --test_output=errors
 
-# When working with hermetic Python toolchains, supporting the legacy runfiles layout is needlessly wasting resources.
-# See https://github.com/bazelbuild/rules_python/issues/1653
-common --nolegacy_external_runfiles
-
 # Allow users to provide their own workspace settings
 try-import %workspace%/user.bazelrc

.bazelversion

@@ -1 +1 @@
-7.4.1
+8.0.1

MODULE.bazel

@@ -17,7 +17,9 @@ use_repo(non_module_dependencies, "dwyu_pcpp")
 ###
 
 bazel_dep(name = "aspect_bazel_lib", version = "2.10.0", dev_dependency = True)
-bazel_dep(name = "stardoc", version = "0.7.2", dev_dependency = True)
+
+# Hidden dependencies of aspect_bazel_lib macro stardoc_with_diff_test
+bazel_dep(name = "platforms", version = "0.0.11", dev_dependency = True)
 
 python = use_extension(
     "@rules_python//python/extensions:python.bzl",

examples/.bazelrc

@@ -1,9 +1,5 @@
 # Do not waste memory by keeping idle Bazel servers around
 startup --max_idle_secs=10
 
-# When working with hermetic Python toolchains, supporting the legacy runfiles layout is needlessly wasting resources.
-# See https://github.com/bazelbuild/rules_python/issues/1653
-common --nolegacy_external_runfiles
-
 # The symlinks are cluttering the file tree without much value
 common --experimental_convenience_symlinks=ignore

examples/.bazelversion

@@ -1 +1 @@
-7.4.1
+8.0.1

examples/test.py

@@ -108,7 +108,7 @@ class Result:
 def make_cmd(example: Example, bazel_bin: str, legacy_workspace: bool) -> list[str]:
     cmd = [bazel_bin, "build"]
     if legacy_workspace:
-        cmd.append("--noenable_bzlmod")
+        cmd.extend(["--noenable_bzlmod", "--enable_workspace"])
     cmd.extend(shlex.split(example.build_cmd))
     return cmd
 

test/apply_fixes/execution_logic.py

@@ -37,18 +37,14 @@
 """
 
 BAZEL_RC_FILE = """
-# Decrease Python toolchain overhead
-# See https://github.com/bazelbuild/rules_python/issues/1653
-common --nolegacy_external_runfiles
-
 # Useless as the workspace are thrown away after each creation
 common --lockfile_mode=off
 
 # Some users require this setting to mitigate issues due to a large PYTHONPATH created by rules_python
 build --noexperimental_python_import_all_repositories
 """
 
-BAZEL_VERSION = "7.4.1"
+BAZEL_VERSION = "8.0.1"
 
 
 def dwyu_path_as_string(dwyu_path: Path) -> str:

test/aspect/.bazelrc

@@ -4,9 +4,5 @@ startup --max_idle_secs=10
 # Always tell why tests fail
 test --test_output=errors
 
-# When working with hermetic Python toolchains, supporting the legacy runfiles layout is needlessly wasting resources.
-# See https://github.com/bazelbuild/rules_python/issues/1653
-common --nolegacy_external_runfiles
-
 # The symlinks are cluttering the file tree without much value
 common --experimental_convenience_symlinks=ignore

test/aspect/.bazelversion

@@ -1 +1 @@
-7.4.1
+8.0.1

test/aspect/test_case.py

@@ -109,7 +109,6 @@ def _run_bazel_build(
             "build",
             "--experimental_convenience_symlinks=ignore",
             "--noshow_progress",
-            "--nolegacy_external_runfiles",
             *self._extra_args,
             *extra_args,
             "--",

test/aspect/test_case_test.py

@@ -91,7 +91,6 @@ def test_dwyu_command_without_any_extra_args(self, run_mock: MagicMock) -> None:
                 "build",
                 "--experimental_convenience_symlinks=ignore",
                 "--noshow_progress",
-                "--nolegacy_external_runfiles",
                 "--@rules_python//python/config_settings:python_version=13.37",
                 "--aspects=//some:aspect",
                 "--output_groups=dwyu",
@@ -122,7 +121,6 @@ def test_dwyu_command_with_global_and_dwyu_extra_args(self, run_mock: MagicMock)
                 "build",
                 "--experimental_convenience_symlinks=ignore",
                 "--noshow_progress",
-                "--nolegacy_external_runfiles",
                 "--global_arg=23",
                 "--another_global_arg",
                 "--@rules_python//python/config_settings:python_version=13.37",
@@ -157,7 +155,6 @@ def test_dwyu_command_with_multiple_targets(self, run_mock: MagicMock) -> None:
                 "build",
                 "--experimental_convenience_symlinks=ignore",
                 "--noshow_progress",
-                "--nolegacy_external_runfiles",
                 "--@rules_python//python/config_settings:python_version=13.37",
                 "--aspects=//some:aspect",
                 "--output_groups=dwyu",