Updates for Windows 10 #2962

Merged
merged 1 commit into from Feb 9, 2016

Projects

None yet

3 participants

@arrmo
Contributor
arrmo commented Feb 9, 2016

Hi,

Not sure this is 100% ready yet, but want to flag this - by all means comment!

The changes are (reverse order, for a reason),
2) Recognize Windows 10. Builds are 10xxx (> 10000), so just look for the 10 at the start.

  1. I checked the response for sysObjectID, and it seems to be returning 'enterprises' rather than .1.3.6.1.4.1 ... is that just me? In any case, without this change - the Windows version checks were not being run.

Thanks!

@arrmo arrmo Updates for Windows 10
f2054d3
@laf laf merged commit da9f6e5 into librenms:master Feb 9, 2016

2 checks passed

Auto-Deploy Build finished. No test results found.
Details
Scrutinizer No new issues
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment