Skip to content

Commit

Permalink
Propagate --working-dir argument
Browse files Browse the repository at this point in the history
  • Loading branch information
Michael B. Gale committed Jun 13, 2023
1 parent fce65da commit 6f07b50
Show file tree
Hide file tree
Showing 9 changed files with 25 additions and 22 deletions.
5 changes: 4 additions & 1 deletion lib/codeql.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion lib/codeql.js.map

Large diffs are not rendered by default.

6 changes: 1 addition & 5 deletions lib/resolve-environment-action.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion lib/resolve-environment-action.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 3 additions & 3 deletions lib/resolve-environment.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion lib/resolve-environment.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 8 additions & 1 deletion src/codeql.ts
Original file line number Diff line number Diff line change
Expand Up @@ -122,6 +122,7 @@ export interface CodeQL {
* Run 'codeql resolve build-environment'
*/
resolveBuildEnvironment(
workingDir: string | undefined,
language: Language
): Promise<ResolveBuildEnvironmentOutput>;

Expand Down Expand Up @@ -700,13 +701,19 @@ export async function getCodeQLForCmd(
throw new Error(`Unexpected output from codeql resolve queries: ${e}`);
}
},
async resolveBuildEnvironment(language: Language) {
async resolveBuildEnvironment(
workingDir: string | undefined,
language: Language
) {
const codeqlArgs = [
"resolve",
"build-environment",
`--language=${language}`,
...getExtraOptionsFromEnv(["resolve", "build-environment"]),
];
if (workingDir !== undefined) {
codeqlArgs.push("--working-dir", workingDir);
}
const output = await runTool(cmd, codeqlArgs);

try {
Expand Down
8 changes: 1 addition & 7 deletions src/resolve-environment-action.ts
Original file line number Diff line number Diff line change
Expand Up @@ -80,16 +80,10 @@ async function run() {
);

const workingDirectory = getOptionalInput("working-directory");
if (workingDirectory) {
logger.info(
`Changing autobuilder working directory to ${workingDirectory}`
);
process.chdir(workingDirectory);
}

const result = await runResolveBuildEnvironment(
initCodeQLResult.codeql.getPath(),
logger,
workingDirectory,
language
);
core.setOutput("environment", result);
Expand Down
7 changes: 5 additions & 2 deletions src/resolve-environment.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,11 +5,14 @@ import { Logger } from "./logging";
export async function runResolveBuildEnvironment(
cmd: string,
logger: Logger,
workingDir: string | undefined,
language: Language
) {
logger.startGroup(`Attempting to resolve build environment for ${language}`);
logger.startGroup(
`Attempting to resolve build environment for ${language} in ${workingDir}`
);
const codeQL = await getCodeQL(cmd);
const result = await codeQL.resolveBuildEnvironment(language);
const result = await codeQL.resolveBuildEnvironment(workingDir, language);
logger.endGroup();
return result;
}

0 comments on commit 6f07b50

Please sign in to comment.