Force dynamic linking

Author: kraenhansen

packages/ferric/src/napi-rs.ts

@@ -44,15 +44,27 @@ export async function generateTypeScriptDeclarations({
     );
     const tempOutputPath = path.join(tempPath, outputFilename);
     // Call into napi.rs to generate TypeScript declarations
-    // RUSTFLAGS: "-C link-arg=-undefined -C link-arg=dynamic_lookup",
-    const { task } = await napiCli.build({
-      verbose: false,
-      dts: outputFilename,
-      outputDir: tempPath,
-      cwd: createPath,
-      cargoOptions: ["--quiet"],
-    });
-    await task;
+    try {
+      assert.equal(
+        typeof process.env.RUSTFLAGS,
+        "undefined",
+        "Expected RUSTFLAGS to be undefined"
+      );
+      // Workaround to instruct the rust compiler to not worry about undefined symbols
+      // TODO: Figure out why this is actually necessary - it should link against sys::nodeapi
+      process.env.RUSTFLAGS =
+        "-C link-arg=-undefined -C link-arg=dynamic_lookup";
+      const { task } = await napiCli.build({
+        verbose: false,
+        dts: outputFilename,
+        outputDir: tempPath,
+        cwd: createPath,
+        cargoOptions: ["--quiet"],
+      });
+      await task;
+    } finally {
+      delete process.env.RUSTFLAGS;
+    }
     // Override the banner
     assert(
       fs.existsSync(tempOutputPath),