Skip to content

Commit 4066cbf

Browse files
Merge pull request #159 from erichdongubler-mozilla/relax-windows-detection
fix: stop requiring `win11_2009` to be true for Windows platform detection
2 parents 4f33b77 + 5034322 commit 4066cbf

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

moz-webgpu-cts/src/report.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,23 +27,23 @@ impl<'de> Deserialize<'de> for RunInfo {
2727
struct ActualRunInfo {
2828
os: String,
2929
processor: String,
30-
win11_2009: bool,
3130
debug: bool,
3231
}
3332

3433
let ActualRunInfo {
3534
os,
3635
processor,
37-
win11_2009,
3836
debug,
3937
} = ActualRunInfo::deserialize(deserializer)?;
4038

4139
let platform = match &*os {
4240
"win" => {
43-
if processor == "x86_64" && win11_2009 {
41+
if processor == "x86_64" {
4442
Platform::Windows
4543
} else {
46-
return Err(D::Error::custom("asdf"));
44+
return Err(D::Error::custom(
45+
"platform was `win`, but `processor` was not `x86_64`",
46+
));
4747
}
4848
}
4949
"mac" => Platform::MacOs,

0 commit comments

Comments
 (0)