Use a relative path to ignore the runner temporary directory.

github · Sep 29, 2020 · dbecf76 · dbecf76
1 parent 206e34c
commit dbecf76
Showing 6 changed files with 6 additions and 6 deletions.
diff --git a/lib/analysis-paths.js b/lib/analysis-paths.js
diff --git a/lib/analysis-paths.js.map b/lib/analysis-paths.js.map
diff --git a/lib/analysis-paths.test.js b/lib/analysis-paths.test.js
diff --git a/lib/analysis-paths.test.js.map b/lib/analysis-paths.test.js.map
diff --git a/src/analysis-paths.test.ts b/src/analysis-paths.test.ts
@@ -64,7 +64,7 @@ test("exclude temp dir", async (t) => {
     };
     analysisPaths.includeAndExcludeAnalysisPaths(config, getRunnerLogger(true));
     t.is(process.env["LGTM_INDEX_INCLUDE"], undefined);
-    t.is(process.env["LGTM_INDEX_EXCLUDE"], tempDir);
+    t.is(process.env["LGTM_INDEX_EXCLUDE"], "codeql-runner-temp");
     t.is(process.env["LGTM_INDEX_FILTERS"], undefined);
   });
 });
diff --git a/src/analysis-paths.ts b/src/analysis-paths.ts
@@ -59,7 +59,7 @@ export function includeAndExcludeAnalysisPaths(
     logger.warning(
       "Storing the CodeQL Runner in the directory being analyzed is not recommended."
     );
-    pathsIgnore = pathsIgnore.concat(config.tempDir);
+    pathsIgnore = pathsIgnore.concat(tempRelativeToWorking);
   }
   if (pathsIgnore.length !== 0) {
     process.env["LGTM_INDEX_EXCLUDE"] = buildIncludeExcludeEnvVar(pathsIgnore);