Skip to content

Commit

Permalink
Don't include parsed_args.framework in the artifact storage path, we …
Browse files Browse the repository at this point in the history
…shouldn't need it as the problem was due to dotnet_mono conflicts.
  • Loading branch information
LoopedBard3 committed Feb 14, 2024
1 parent 515e58d commit e3676e2
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion scripts/benchmarks_local.py
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ def generate_layout(parsed_args: Namespace, repo_path: str, additional_args: Lis
RunCommand(generate_layout_command, verbose=True).run(os.path.join(repo_path, "src", "tests"))

def get_run_artifact_path(parsed_args: Namespace, run_type: RunType, commit: str) -> str:
return os.path.join(parsed_args.artifact_storage_path, f"{run_type.name}-{commit}-{parsed_args.os}-{parsed_args.architecture}-{parsed_args.framework}")
return os.path.join(parsed_args.artifact_storage_path, f"{run_type.name}-{commit}-{parsed_args.os}-{parsed_args.architecture}")

def get_mono_corerun(parsed_args: Namespace, run_type: RunType, commit: str) -> str:
corerun_capture = glob.glob(os.path.join(get_run_artifact_path(parsed_args, run_type, commit), "dotnet_mono", "shared", "Microsoft.NETCore.App", f"*", f'corerun{".exe" if is_windows(parsed_args) else ""}'))
Expand Down

0 comments on commit e3676e2

Please sign in to comment.