Address comments

Author: minggangw

test/test-rosidl-message-generator.js

@@ -63,13 +63,46 @@ function sourceSetupScript(setupPath) {
 function buildTestMessage() {
   // Build the custom_msg_test package synchronously before running the test
   const customMsgTestPath = path.join(__dirname, 'custom_msg_test');
+  const fs = require('fs');
 
-  const buildResult = require('child_process').spawnSync('colcon', ['build'], {
+  // Try to create log directory with proper permissions, or use temp directory
+  const logPath = path.join(customMsgTestPath, 'log');
+  let logBase = null;
+
+  try {
+    // Try to create the log directory
+    if (!fs.existsSync(logPath)) {
+      fs.mkdirSync(logPath, { recursive: true, mode: 0o755 });
+    }
+  } catch (err) {
+    // If we can't create log directory, use a temp directory instead
+    logBase = path.join(os.tmpdir(), `colcon-log-${process.pid}`);
+    if (!fs.existsSync(logBase)) {
+      fs.mkdirSync(logBase, { recursive: true, mode: 0o755 });
+    }
+  }
+
+  // Build colcon command arguments
+  const colconArgs = ['build'];
+  if (logBase) {
+    colconArgs.push('--log-base', logBase);
+  }
+
+  const buildResult = require('child_process').spawnSync('colcon', colconArgs, {
     cwd: customMsgTestPath,
     stdio: 'inherit',
     timeout: 60000, // 60 second timeout
   });
 
+  // Cleanup temp log directory if we created one
+  if (logBase && fs.existsSync(logBase)) {
+    try {
+      fs.rmSync(logBase, { recursive: true, force: true });
+    } catch (err) {
+      // Ignore cleanup errors
+    }
+  }
+
   if (buildResult.error) {
     throw new Error(
       `Failed to build custom_msg_test package: ${buildResult.error.message}`