Linux build working: JSCOnly port + system malloc + Gigacage stub

Use -DPORT=JSCOnly which only needs ICU (no GTK/Cairo/GStreamer deps).
Use -DUSE_SYSTEM_MALLOC=ON to avoid bmalloc dependency.
Stub Gigacage::ensureGigacage() and YARR Unicode property functions.

Flatten JSC headers into include bridge so <JavaScriptCore/Yarr.h>
resolves correctly from Source/JavaScriptCore/yarr/Yarr.h.

Verified in Docker: ubuntu:24.04 with cmake, ninja, g++, libicu-dev.

Author: chrmod

.github/workflows/validate-dnr-rules.yml

@@ -30,7 +30,7 @@ jobs:
         run: |
           sudo apt-get update
           sudo apt-get install -y --no-install-recommends \
-            cmake ninja-build pkg-config ruby unifdef libicu-dev
+            cmake ninja-build pkg-config ruby unifdef libicu-dev g++ perl python3
 
       - name: Cache CMake build
         uses: actions/cache@v4
@@ -45,6 +45,7 @@ jobs:
           cmake -B build -G Ninja \
             -DCMAKE_BUILD_TYPE=Release \
             -DPORT=JSCOnly \
+            -DUSE_SYSTEM_MALLOC=ON \
             .
 
       - name: Build
@@ -60,16 +61,16 @@ jobs:
             {"id":4,"priority":1,"action":{"type":"block"},"condition":{"regexFilter":"pixel\\b"}}
           ]
           RULES
-          ./build/validate-dnr-rules /tmp/test-rules.json
+          ./build/bin/validate-dnr-rules /tmp/test-rules.json
 
       - name: Binary info
         run: |
-          file build/validate-dnr-rules
-          du -h build/validate-dnr-rules
-          ldd build/validate-dnr-rules || true
+          file build/bin/validate-dnr-rules
+          du -h build/bin/validate-dnr-rules
+          ldd build/bin/validate-dnr-rules || true
 
       - name: Upload artifact
         uses: actions/upload-artifact@v4
         with:
           name: validate-dnr-rules-${{ matrix.name }}
-          path: build/validate-dnr-rules
+          path: build/bin/validate-dnr-rules

ghostery/validate-dnr-rules/CMakeLists.txt

@@ -25,7 +25,6 @@ set(TOOL_SOURCES
 add_executable(validate-dnr-rules ${TOOL_SOURCES} ${CE_SOURCES})
 
 target_compile_definitions(validate-dnr-rules PRIVATE
-    ENABLE_CONTENT_EXTENSIONS=1
     WEBCORE_EXPORT=
     WEBCORE_TESTSUPPORT_EXPORT=
     STATICALLY_LINKED_WITH_WTF=1
@@ -40,14 +39,35 @@ target_include_directories(validate-dnr-rules PRIVATE
     ${WTF_DERIVED_SOURCES_DIR}
 )
 
-# Create include bridge so <WebCore/...> and <JavaScriptCore/...> resolve
+# Create include bridge so <WebCore/...> and <JavaScriptCore/...> resolve.
+# WebKit headers use flat <JavaScriptCore/Yarr.h> but files live in subdirs.
 set(INCLUDE_BRIDGE_DIR "${CMAKE_BINARY_DIR}/validate-dnr-includes")
-file(MAKE_DIRECTORY "${INCLUDE_BRIDGE_DIR}")
-file(CREATE_LINK "${CE_DIR}" "${INCLUDE_BRIDGE_DIR}/WebCore" SYMBOLIC)
-file(CREATE_LINK "${JSC_DIR}/yarr" "${INCLUDE_BRIDGE_DIR}/JavaScriptCore" SYMBOLIC)
+file(MAKE_DIRECTORY "${INCLUDE_BRIDGE_DIR}/WebCore")
+file(MAKE_DIRECTORY "${INCLUDE_BRIDGE_DIR}/JavaScriptCore")
+
+file(GLOB CE_HEADERS "${CE_DIR}/*.h")
+foreach(H ${CE_HEADERS})
+    get_filename_component(HNAME ${H} NAME)
+    file(CREATE_LINK "${H}" "${INCLUDE_BRIDGE_DIR}/WebCore/${HNAME}" SYMBOLIC)
+endforeach()
+
+file(GLOB YARR_HEADERS "${JSC_DIR}/yarr/*.h")
+foreach(H ${YARR_HEADERS})
+    get_filename_component(HNAME ${H} NAME)
+    file(CREATE_LINK "${H}" "${INCLUDE_BRIDGE_DIR}/JavaScriptCore/${HNAME}" SYMBOLIC)
+endforeach()
+
+# Also link JSC headers from subdirectories that YARR needs
+file(GLOB_RECURSE ALL_JSC_HEADERS "${JSC_DIR}/*.h")
+foreach(H ${ALL_JSC_HEADERS})
+    get_filename_component(HNAME ${H} NAME)
+    if(NOT EXISTS "${INCLUDE_BRIDGE_DIR}/JavaScriptCore/${HNAME}")
+        file(CREATE_LINK "${H}" "${INCLUDE_BRIDGE_DIR}/JavaScriptCore/${HNAME}" SYMBOLIC)
+    endif()
+endforeach()
+
 target_include_directories(validate-dnr-rules SYSTEM PRIVATE
     ${INCLUDE_BRIDGE_DIR}
-    ${JSC_DIR}
 )
 
 target_link_libraries(validate-dnr-rules PRIVATE WTF)

ghostery/validate-dnr-rules/src/stubs.cpp

@@ -1,42 +1,14 @@
 #include "config.h"
-#include <cstdio>
-#include <cstdlib>
 #include <JavaScriptCore/Yarr.h>
 #include <JavaScriptCore/YarrUnicodeProperties.h>
 #include <wtf/text/WTFString.h>
 
-// WTF assertion stubs — these are normally provided by libWTF
-extern "C" {
-
-void WTFCrash()
-{
-    fprintf(stderr, "WTFCrash\n");
-    abort();
-}
-
-void WTFCrashWithSecurityImplication()
-{
-    fprintf(stderr, "WTFCrashWithSecurityImplication\n");
-    abort();
-}
-
-void WTFReportAssertionFailure(const char* file, int line, const char* function, const char* assertion)
-{
-    fprintf(stderr, "ASSERTION FAILED: %s (%s:%d %s)\n", assertion, file, line, function);
+// Gigacage stub — not needed when using system malloc
+namespace Gigacage {
+void ensureGigacage() { }
 }
 
-void WTFReportAssertionFailureWithMessage(const char* file, int line, const char* function, const char* assertion, const char* format, ...)
-{
-    fprintf(stderr, "ASSERTION FAILED: %s (%s:%d %s)\n", assertion, file, line, function);
-}
-
-void WTFReportBacktrace()
-{
-}
-
-} // extern "C"
-
-// YARR Unicode property stubs — these are only needed for \p{} property
+// YARR Unicode property stubs — only needed for \p{} property
 // escapes which DNR regexFilter patterns never use.
 namespace JSC::Yarr {