Added detction for OSX El Capitan #3690

Merged
merged 2 commits into from Jun 21, 2016

Projects

None yet

3 participants

@laf
Member
laf commented Jun 20, 2016

Fix #3017

@laf laf Added detction for OSX El Capitan
3ee0ff6
@laf laf added the New-Device label Jun 20, 2016
@Rosiak
Contributor
Rosiak commented Jun 21, 2016

Pretty sure 1.3.6.1.4.1.8072.3.2.255 is a generic one, at least it was when I was looking last time.
Think we should use the sysDescr instead.

@laf laf Updated to use sysDescr
ee2ec6c
@laf
Member
laf commented Jun 21, 2016

Updated

@Rosiak
Contributor
Rosiak commented Jun 21, 2016

Sorry to bother again, but why not strstr?

@laf
Member
laf commented Jun 21, 2016

Taken from another file, feel free to submit your own pr if you want strstr.

@Rosiak
Contributor
Rosiak commented Jun 21, 2016

meh.

@Rosiak Rosiak merged commit 9d62339 into librenms:master Jun 21, 2016

2 checks passed

Auto-Deploy Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@laf laf deleted the laf:issue-3017 branch Jan 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment