From c4ce9919aa0193fb233d4908a9539120c4401a65 Mon Sep 17 00:00:00 2001
From: Kyle Barron <kylebarron2@gmail.com>
Date: Thu, 18 Apr 2024 19:57:01 -0400
Subject: [PATCH] Ensure apache-arrow is external in rollup config (#118)

---
 CHANGELOG.md     | 1 +
 rollup.config.js | 8 ++++++++
 2 files changed, 9 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 82dd6ab..0d1dae6 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -3,6 +3,7 @@
 ## [0.4.2] - 2024-04-18
 
 - Allow `Uint32Array` type in `parseTable`.
+- Ensure `apache-arrow` is marked as an external library in Rollup
 
 ## [0.4.1] - 2024-01-31
 
diff --git a/rollup.config.js b/rollup.config.js
index 3dcdea2..2d13b57 100644
--- a/rollup.config.js
+++ b/rollup.config.js
@@ -4,6 +4,7 @@ import dts from "rollup-plugin-dts";
 
 const input = "./src/index.ts";
 const sourcemap = true;
+const external = ["apache-arrow"];
 
 export default [
   {
@@ -14,6 +15,7 @@ export default [
       sourcemap,
     },
     plugins: [typescript()],
+    external,
   },
   {
     input,
@@ -22,6 +24,7 @@ export default [
       format: "es",
     },
     plugins: [dts()],
+    external,
   },
   {
     input,
@@ -31,6 +34,7 @@ export default [
       sourcemap,
     },
     plugins: [typescript()],
+    external,
   },
   {
     input,
@@ -39,7 +43,11 @@ export default [
       format: "umd",
       name: "arrowJsFFI",
       sourcemap,
+      globals: {
+        "apache-arrow": "Arrow",
+      },
     },
     plugins: [typescript(), terser()],
+    external,
   },
 ];