Added dnos to this, does not seem worth duplicating the code for another os #1782

Merged
merged 2 commits into from Aug 28, 2015

Projects

None yet

3 participants

@laf
Member
laf commented Aug 27, 2015

Fix #1779

@laf laf Added dnos to this, does not seem worth duplicating the code for anot…
…her os
e7df128
@laf laf added the Discovery label Aug 27, 2015
@f0o
Member
f0o commented Aug 28, 2015

Could we rename the file to indicate that it's no longer ftos-centric?

@laf
Member
laf commented Aug 28, 2015

It's not dnos specific either, it's dnos + ftos-s-series, any suggestions?

@f0o
Member
f0o commented Aug 28, 2015

how about dnos_ftos-s-series.inc.php ?

@laf laf Renamed file
813b518
@laf
Member
laf commented Aug 28, 2015

Pushed update.

@f0o f0o merged commit 8e19a57 into librenms:master Aug 28, 2015

1 of 2 checks passed

Scrutinizer Created
Details
Auto-Deploy Build finished.
Details
@laf laf deleted the laf:issue-1779 branch Aug 28, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment