Correctly identify Server 2008, Server 2008 R2, Server 2012, and Windows 8
Review Request #311 - Created May 17, 2011 and submitted
Tankmaster Finesmith | Reviewers | ||
viewer | |||
STORM-1258 | |||
None | viewer-development |
The stated OS in help -> about for server 2008 and server 2008 R2 is inaccurate. It shows up as "Microsoft Windows Vista Server" or "Microsoft Windows 7 Server" instead of "Microsoft Server 2008" or "Microsoft Server 2008 R2" respectively. This patch corrects the text display error and also adds support for "Microsoft Server 2012" and "Microsoft Windows 8" which I have confirmed works on those OSs. I came to this work by mimicking the code logic for how server 2003 was handled. This patch does not correct the error of XP always showing up as running in compatibility mode. This also doesn’t affect the redundant display of x64 for Server 2008 R2 and Server 2012, which are only available in 64bit versions. I have applied my patch to my clone of viewer-dev at https://bitbucket.org/Tank_Master/storm-1258/changeset/f4b293bbdedf and my initial firestorm commit can be found at http://hg.phoenixviewer.com/phoenix-firestorm-lgpl/rev/6bf2de50ea83
I made this patch for Firestorm, where it was applied and tested initially and no errors have been found.
Other reviews