Skip to content

Commit

Permalink
Add arrow patch, to fix inclusion of emscripten.h within extern C in …
Browse files Browse the repository at this point in the history
…xxhash.
  • Loading branch information
carlopi committed Jul 31, 2024
1 parent cd42c15 commit a8c094c
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 0 deletions.
1 change: 1 addition & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -435,6 +435,7 @@ build/docker_ci_image:

patch_duckdb:
find patches/duckdb/* -type f -name '*.patch' -print0 | xargs -0 cat | patch -p1 --forward -d submodules/duckdb
find patches/arrow/* -type f -name '*.patch' -print0 | xargs -0 cat | patch -p1 --forward -d submodules/arrow

submodules:
git submodule update --init --recursive
Expand Down
15 changes: 15 additions & 0 deletions patches/arrow/hashing_compile_in_emscripten.patch
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
diff --git a/cpp/src/arrow/util/hashing.h b/cpp/src/arrow/util/hashing.h
index 2de9f41532..1a7803d2d8 100644
--- a/cpp/src/arrow/util/hashing.h
+++ b/cpp/src/arrow/util/hashing.h
@@ -31,6 +31,10 @@
#include <utility>
#include <vector>

+#ifdef EMSCRIPTEN
+#include <emscripten.h>
+#endif
+
#include "arrow/array/builder_binary.h"
#include "arrow/buffer_builder.h"
#include "arrow/result.h"

0 comments on commit a8c094c

Please sign in to comment.