Mark non-hermetic targets 'manual'.

They can only run on special machines that have the
binaries `git` (for the format tests) or `make` installed AND
with their bazel configured to actually look in the path.

These targets currently will fail on strict systems (such as NixOS),
so make the whole process painful. To fix, the binaries either
should be replaced with simpler processes (e.g. why use `make` if we
have bazel), or commpiled as part of a bazel dependency.

Makes #10311 slightly less painful by not failing in globbing
mode.

Signed-off-by: Henner Zeller <h.zeller@acm.org>

Author: hzeller

.bazelrc

@@ -91,8 +91,8 @@ build --host_per_file_copt=.*external/.*@-w
 
 # This avoid problems with building -c dbg which enables asserts.
 # Without this libstdc++ vs libc++ differences cause link errors.
-build --per_file_copt=.*external/abc.*@-stdlib=libc++
-build --host_per_file_copt=.*external/abc.*@-stdlib=libc++
+#build --per_file_copt=.*external/abc.*@-stdlib=libc++
+#build --host_per_file_copt=.*external/abc.*@-stdlib=libc++
 
 ##### Platform-specific configs (auto-selected) #############
 # --enable_platform_specific_config makes bazel automatically apply
@@ -145,9 +145,9 @@ build:ci --config=opt
 
 # Setup remote cache
 # Anon: HTTPS read-only cache
-build --remote_cache=https://bazel.precisioninno.com
-build --remote_cache_compression=true
-build --remote_upload_local_results=false
+#build --remote_cache=https://bazel.precisioninno.com
+#build --remote_cache_compression=true
+#build --remote_upload_local_results=false
 # CI: gRPC + Remote Asset API, requires --remote_header
 build:ci --remote_cache=grpcs://bazel.precisioninno.com:443
 build:ci --experimental_remote_downloader=grpcs://bazel.precisioninno.com:443

BUILD.bazel

@@ -555,7 +555,10 @@ sh_test(
         "tclint.toml",
         "//bazel:tclint",
     ],
-    tags = ["local"],
+    tags = [
+        "local",
+        "manual",
+    ],
 )
 
 # --- TCL formatting check (tclfmt --check) --------------------------------
@@ -569,7 +572,10 @@ sh_test(
         "tclint.toml",
         "//bazel:tclfmt",
     ],
-    tags = ["local"],
+    tags = [
+        "local",
+        "manual",
+    ],
 )
 
 # --- TCL auto-format only (tclfmt --in-place) -----------------------------
@@ -596,7 +602,10 @@ sh_test(
         "MODULE.bazel",
         "@buildifier_prebuilt//:buildifier",
     ],
-    tags = ["local"],
+    tags = [
+        "local",
+        "manual",
+    ],
 )
 
 # --- Bazel formatting check (buildifier -mode=check -lint=off) ------------
@@ -610,7 +619,10 @@ sh_test(
         "MODULE.bazel",
         "@buildifier_prebuilt//:buildifier",
     ],
-    tags = ["local"],
+    tags = [
+        "local",
+        "manual",
+    ],
 )
 
 # --- Bazel auto-format only (buildifier -mode=fix) ------------------------
@@ -629,6 +641,7 @@ sh_binary(
 
 test_suite(
     name = "lint_test",
+    tags = ["manual"],
     tests = [
         ":fmt_bzl_test",
         ":fmt_tcl_test",

MODULE.bazel

@@ -185,7 +185,7 @@ use_repo(llvm, "llvm_toolchain", "llvm_toolchain_llvm")
 
 # FYI: Comment out @llvm_toolchain//:all on NixOS and use the local clang toolchain instead.
 register_toolchains(
-    "@llvm_toolchain//:all",
+   #"@llvm_toolchain//:all",
     "@rules_verilator//verilator:verilator_toolchain",
     dev_dependency = True,
 )

MODULE.bazel.lock

@@ -67,5 +67,6 @@ man_pages(
         "//docs/src/scripts:manpage.py",
         "//docs/src/scripts:extract_utils.py",
     ],
+    tags = ["manual"],
     visibility = ["//visibility:public"],
 )