Add transformation of "defines" to gyp-to-cmake (#97)

* Add test for command expansion

* Emit "defines" via target_compile_definitions

Author: kraenhansen

packages/gyp-to-cmake/src/gyp.ts

@@ -7,6 +7,7 @@ export type GypTarget = {
   target_name: string;
   sources: string[];
   include_dirs?: string[];
+  defines?: string[];
 };
 
 export type GypBinding = {

packages/gyp-to-cmake/src/transformer.test.ts

@@ -64,4 +64,63 @@ describe("bindingGypToCmakeLists", () => {
 
     assert(output.includes("add_library(foo SHARED file\\ with\\ spaces.cc"));
   });
+
+  describe("command expansions", () => {
+    it("should expand", () => {
+      const output = bindingGypToCmakeLists({
+        projectName: "some-project",
+        gyp: {
+          targets: [
+            {
+              target_name: "foo",
+              sources: ["<!echo bar baz"],
+            },
+          ],
+        },
+      });
+
+      // Adding \ between bar and baz, as we expect the "bar baz" to be handled like a path with spaces
+      assert(output.includes("add_library(foo SHARED bar\\ baz"));
+    });
+
+    it("should expand into lists when prefixed with '@'", () => {
+      const output = bindingGypToCmakeLists({
+        projectName: "some-project",
+        gyp: {
+          targets: [
+            {
+              target_name: "foo",
+              sources: ["<!@echo bar baz"],
+            },
+          ],
+        },
+      });
+
+      assert(output.includes("add_library(foo SHARED bar baz"));
+    });
+  });
+
+  describe("defines", () => {
+    it("should add defines as target-specific compile definitions", () => {
+      const output = bindingGypToCmakeLists({
+        projectName: "some-project",
+        gyp: {
+          targets: [
+            {
+              target_name: "foo",
+              sources: ["foo.cc"],
+              defines: ["FOO", "BAR=value"],
+            },
+          ],
+        },
+      });
+
+      assert(
+        output.includes(
+          "target_compile_definitions(foo PRIVATE FOO BAR=value)"
+        ),
+        `Expected output to include target_compile_definitions:\n${output}`
+      );
+    });
+  });
 });

packages/gyp-to-cmake/src/transformer.ts

@@ -19,7 +19,7 @@ function isCmdExpansion(value: string) {
   return trimmedValue.startsWith("<!");
 }
 
-function escapePath(source: string) {
+function escapeSpaces(source: string) {
   return source.replace(/ /g, "\\ ");
 }
 
@@ -65,27 +65,33 @@ export function bindingGypToCmakeLists({
     //"cmake_policy(SET CMP0042 NEW)",
     `project(${projectName})`,
     "",
-    `add_compile_definitions(-DNAPI_VERSION=${napiVersion})`,
+    // Declaring a project-wide NAPI_VERSION as a fallback for targets that don't explicitly set it
+    `add_compile_definitions(NAPI_VERSION=${napiVersion})`,
   ];
 
   for (const target of gyp.targets) {
     const { target_name: targetName } = target;
 
     // TODO: Handle "conditions"
-    // TODO: Handle "defines"
     // TODO: Handle "cflags"
     // TODO: Handle "ldflags"
 
     const escapedJoinedSources = target.sources
       .flatMap(mapExpansion)
       .map(transformPath)
-      .map(escapePath)
+      .map(escapeSpaces)
       .join(" ");
 
     const escapedJoinedIncludes = (target.include_dirs || [])
       .flatMap(mapExpansion)
       .map(transformPath)
-      .map(escapePath)
+      .map(escapeSpaces)
+      .join(" ");
+
+    const escapedJoinedDefines = (target.defines || [])
+      .flatMap(mapExpansion)
+      .map(transformPath)
+      .map(escapeSpaces)
       .join(" ");
 
     lines.push(
@@ -94,7 +100,12 @@ export function bindingGypToCmakeLists({
       `set_target_properties(${targetName} PROPERTIES PREFIX "" SUFFIX ".node")`,
       `target_include_directories(${targetName} PRIVATE ${escapedJoinedIncludes} \${CMAKE_JS_INC})`,
       `target_link_libraries(${targetName} PRIVATE \${CMAKE_JS_LIB})`,
-      `target_compile_features(${targetName} PRIVATE cxx_std_17)`
+      `target_compile_features(${targetName} PRIVATE cxx_std_17)`,
+      ...(escapedJoinedDefines
+        ? [
+            `target_compile_definitions(${targetName} PRIVATE ${escapedJoinedDefines})`,
+          ]
+        : [])
       // or
       // `set_target_properties(${targetName} PROPERTIES CXX_STANDARD 11 CXX_STANDARD_REQUIRED YES CXX_EXTENSIONS NO)`,
     );