From a4bc3fdf327c8cdf629d5a79579e7591f5b99545 Mon Sep 17 00:00:00 2001
From: Tim Wojtulewicz <tim@corelight.com>
Date: Fri, 30 Jun 2023 14:04:12 -0700
Subject: [PATCH] binpac: Add /J flag on Windows to force unsigned char

---
 tools/binpac/CMakeLists.txt | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/tools/binpac/CMakeLists.txt b/tools/binpac/CMakeLists.txt
index 30a4ad41c1..b5ca54a665 100644
--- a/tools/binpac/CMakeLists.txt
+++ b/tools/binpac/CMakeLists.txt
@@ -31,12 +31,16 @@ FindRequiredPackage(FLEX)
 FindRequiredPackage(BISON)
 
 if (MISSING_PREREQS)
-    foreach (prereq ${MISSING_PREREQ_DESCS})
-        message(SEND_ERROR ${prereq})
-    endforeach ()
-    message(FATAL_ERROR "Configuration aborted due to missing prerequisites")
+  foreach (prereq ${MISSING_PREREQ_DESCS})
+      message(SEND_ERROR ${prereq})
+  endforeach ()
+  message(FATAL_ERROR "Configuration aborted due to missing prerequisites")
 endif ()
 
+if (MSVC)
+  add_compile_options(/J) # Similar to -funsigned-char on other platforms
+endif()
+
 ########################################################################
 ## System Introspection