Skip to content

Commit 8845cfd

Browse files
committed
fix: use generator_output as output_dir
fix: nodejs/node-gyp#2305
1 parent 2f79158 commit 8845cfd

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

pylib/gyp/generator/compile_commands_json.py

+2-3
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,6 @@ def CalculateVariables(default_variables, params):
4242

4343

4444
def AddCommandsForTarget(cwd, target, params, per_config_commands):
45-
output_dir = params["generator_flags"].get("output_dir", "out")
4645
for configuration_name, configuration in target["configurations"].items():
4746
if IsMac(params):
4847
xcode_settings = gyp.xcode_emulation.XcodeSettings(target)
@@ -93,7 +92,7 @@ def resolve(filename):
9392
gyp.common.EncodePOSIXShellArgument(file),
9493
)
9594
)
96-
commands.append(dict(command=command, directory=output_dir, file=file))
95+
commands.append(dict(command=command, file=file))
9796

9897

9998
def GenerateOutput(target_list, target_dicts, data, params):
@@ -108,7 +107,7 @@ def GenerateOutput(target_list, target_dicts, data, params):
108107
cwd = os.path.dirname(build_file)
109108
AddCommandsForTarget(cwd, target, params, per_config_commands)
110109

111-
output_dir = params["generator_flags"].get("output_dir", "out")
110+
output_dir = params["options"].generator_output
112111
for configuration_name, commands in per_config_commands.items():
113112
filename = os.path.join(output_dir, configuration_name, "compile_commands.json")
114113
gyp.common.EnsureDirExists(filename)

0 commit comments

Comments
 (0)