From 224907d153a04bf89781c1058d14c19958415dd1 Mon Sep 17 00:00:00 2001 From: Henner Zeller Date: Mon, 25 May 2026 19:21:05 +0200 Subject: [PATCH] Explicitly mark dependencies as keep for bracket-headers used. This project uses bracket includes for dependencies that are actually vendored via the MODULE.bazel. So they should use `"zlib.h"`, `"tcl.h"` etc, but here they use ``, `` which makes them look like system headers. They are meant to be using the vendored headers, to explicitly tell build_cleaner (`bant`) that the dependencies should be kept. (ideally, we fix the includes, but since this is a fork, this might be harder to maintain). Also keeps `@openmp` for now, but see #364 if this is actually needed. Signed-off-by: Henner Zeller --- BUILD | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/BUILD b/BUILD index 1c493f06..5d867b50 100644 --- a/BUILD +++ b/BUILD @@ -396,11 +396,11 @@ cc_library( visibility = ["//:__subpackages__"], deps = [ "@cudd", - "@eigen", - "@openmp", + "@eigen", # keep. Is used, but with <>-includes + "@openmp", # keep. Is needed ? Nobody includes omp.h "@rules_flex//flex:current_flex_toolchain", - "@tcl_lang//:tcl", - "@zlib", + "@tcl_lang//:tcl", # keep. Is used, but with <>-includes + "@zlib", # keep. Is used, but with <>-includes ], )