Skip to content

Commit

Permalink
Fix --make-repro-path option (#90989)
Browse files Browse the repository at this point in the history
Co-authored-by: Ivan Povazan <ivan.povazan@gmail.com>
Co-authored-by: Jan Kotas <jkotas@microsoft.com>
  • Loading branch information
3 people authored Aug 23, 2023
1 parent ac16ae1 commit 303f1d3
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions src/coreclr/tools/Common/CommandLineHelpers.cs
Original file line number Diff line number Diff line change
Expand Up @@ -210,7 +210,7 @@ public static void MakeReproPackage(string makeReproPath, string outputFilePath,
foreach (CliOption option in res.CommandResult.Command.Options)
{
OptionResult optionResult = res.GetResult(option);
if (optionResult is null || option.Name == "make-repro-path")
if (optionResult is null || option.Name == "--make-repro-path")
{
continue;
}
Expand All @@ -233,15 +233,15 @@ public static void MakeReproPackage(string makeReproPath, string outputFilePath,
}
foreach (string inputFile in dictionary.Values)
{
rspFile.Add($"--{option.Name}:{ConvertFromOriginalPathToReproPackagePath(input: true, inputFile)}");
rspFile.Add($"{option.Name}:{ConvertFromOriginalPathToReproPackagePath(input: true, inputFile)}");
}
}
else
{
foreach (string optInList in values)
{
if (!string.IsNullOrEmpty(optInList))
rspFile.Add($"--{option.Name}:{optInList}");
rspFile.Add($"{option.Name}:{optInList}");
}
}
}
Expand All @@ -254,11 +254,11 @@ public static void MakeReproPackage(string makeReproPath, string outputFilePath,
// if output option is used, overwrite the path to the repro package
stringVal = ConvertFromOriginalPathToReproPackagePath(input: false, stringVal);
}
rspFile.Add($"--{option.Name}:{stringVal}");
rspFile.Add($"{option.Name}:{stringVal}");
}
else
{
rspFile.Add($"--{option.Name}:{val}");
rspFile.Add($"{option.Name}:{val}");
}
}
}
Expand Down

0 comments on commit 303f1d3

Please sign in to comment.