Skip to content

Commit f00f7c7

Browse files
authored
JitDiagnosers should print an error when run on non-Windows OS (#1861)
1 parent 0c03e86 commit f00f7c7

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

src/BenchmarkDotNet.Diagnostics.Windows/JitDiagnoser.cs

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
using BenchmarkDotNet.Diagnosers;
44
using BenchmarkDotNet.Engines;
55
using BenchmarkDotNet.Loggers;
6+
using BenchmarkDotNet.Portability;
67
using BenchmarkDotNet.Reports;
78
using BenchmarkDotNet.Running;
89
using BenchmarkDotNet.Validators;
@@ -30,7 +31,13 @@ public void Handle(HostSignal signal, DiagnoserActionParameters parameters)
3031

3132
public virtual IEnumerable<Metric> ProcessResults(DiagnoserResults results) => Array.Empty<Metric>();
3233

33-
public IEnumerable<ValidationError> Validate(ValidationParameters validationParameters) => Array.Empty<ValidationError>();
34+
public IEnumerable<ValidationError> Validate(ValidationParameters validationParameters)
35+
{
36+
if (!RuntimeInformation.IsWindows())
37+
{
38+
yield return new ValidationError(true, $"{GetType().Name} is supported only on Windows");
39+
}
40+
}
3441

3542
public void DisplayResults(ILogger outputLogger)
3643
{

0 commit comments

Comments
 (0)