From 8ae4454bbaf1a2b76536d54d8222bf00ac153241 Mon Sep 17 00:00:00 2001 From: Oleksandr Hnatiuk Date: Wed, 28 Aug 2024 21:20:04 -0700 Subject: [PATCH] vapi: remove directory name from include guards Script vapi_c_gen.py uses full paths to source files to generate include guards in files with `.vapi.h` extension. This makes the content of header files depend on build directory which causes reproducibility issues (cannot get identical build output when building in a different directory). Commit 4c64b6edc86f7 fixes this for .hpp files, but seems to forget about .h files. Use its fix for these as well. Type: fix Change-Id: Ida638c7b4630fac1f6d968a9203a409648bef55c Signed-off-by: Oleksandr Hnatiuk --- src/vpp-api/vapi/vapi_c_gen.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/vpp-api/vapi/vapi_c_gen.py b/src/vpp-api/vapi/vapi_c_gen.py index 9d1efb5e438..709063f164d 100755 --- a/src/vpp-api/vapi/vapi_c_gen.py +++ b/src/vpp-api/vapi/vapi_c_gen.py @@ -891,7 +891,7 @@ def gen_json_unified_header(parser, logger, j, io, name): orig_stdout = sys.stdout sys.stdout = io include_guard = "__included_%s" % ( - j.replace(".", "_") + f.replace(".", "_") .replace("/", "_") .replace("-", "_") .replace("+", "_")