facebook / buck2

Build system, successor to Buck
https://buck2.build/
Apache License 2.0
3.33k stars 194 forks source link

Add windows-specific call for OS version #630

Closed ejnguyen closed 2 weeks ago

ejnguyen commented 2 weeks ago

Summary: Right now, we rely on the sys_info library to get the OS version.

However, that functionality is broken for Windows: https://github.com/FillZpp/sys-info-rs/issues/86 Specifically, it produces 6.2.9200 for all versions across Windows: https://fburl.com/scuba/buck2_builds/iq6hsudp

This diff swaps the check to use the winver library, which has much better support for fetching the Windows version.

Reviewed By: JakobDegen

Differential Revision: D56546196