Review request for 6843003: Windows Server 2008 R2 system recognition (original) (raw)

Alan Bateman Alan.Bateman at Sun.COM
Mon Jun 29 20:39:58 UTC 2009

Xueming Shen wrote:

:

The fix itself looks OK. 2 random thoughts (1)I see lots of sprops.osname = "Windows NT (unknown)"; It might be better to simply initialize the sprops.osname to "Windows NT (unknown)" at the very beginning of the nt block and then depends on the version number to see if we have a better guess. (2)We have been doing the same exercise every time MS released a new os, it might be the time to extract this out to a "configurable" file and simple do a lookup based on the platform, verMajor, verMinor. It is true that we need to touch this code whenever a new edition of Windows comes along. As it runs at startup I would be hesitant to introduce a lookup file. I agree that the paths in the decision tree leading to unknown aren't prettry and this could be better. Unfortunately I don't have cycles for it just now (changing this code is trivial, but verifying it on each edition is tedious). So are you okay if this is pushed as is?

-Alan.

More information about the core-libs-dev mailing list