Bug 231089 - net-mgmt/librenms typo's in default.inc.php
Summary: net-mgmt/librenms typo's in default.inc.php
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Dan Langille
Keywords: easy
Depends on:
Reported: 2018-09-02 09:19 UTC by Dries Michiels
Modified: 2018-09-02 14:17 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (bofh)


Note You need to log in before you can comment on or make changes to this bug.
Description Dries Michiels 2018-09-02 09:19:49 UTC
There are two typo's in the default PHP included configuration:

$config['nmap']           = '/usr/localbin/nmap';
Should be
$config['nmap']           = '/usr/local/bin/nmap';

$config['mtr']            = '/usr/local/bin/mtr';
Should be
$config['mtr']            = '/usr/local/sbin/mtr';
Comment 1 Dries Michiels 2018-09-02 10:44:43 UTC

Should the enry in .env

not be


as specified in config.php also?
Comment 2 Dan Langille freebsd_committer 2018-09-02 13:51:59 UTC
(In reply to Dries Michiels from comment #0)
Yes, those path corrections are appropriate.

Did you discover this via visual inspection? Was LibreNMS producing an error?
Comment 3 Dan Langille freebsd_committer 2018-09-02 13:53:15 UTC
(In reply to Dries Michiels from comment #1)
Yes, I think so.
Comment 4 Dries Michiels 2018-09-02 14:03:43 UTC
It was not producing an error altough I saw this through visual inspection of the recent commit to get it to 1.42 :)
Comment 5 commit-hook freebsd_committer 2018-09-02 14:07:00 UTC
A commit references this bug:

Author: dvl
Date: Sun Sep  2 14:06:55 UTC 2018
New revision: 478792
URL: https://svnweb.freebsd.org/changeset/ports/478792

  Correct paths for nmap and mtr in patch for includes/defaults.inc.php
  Correct user in patch for .env.example

  PR:		231089
  Submitted by:	Dries Michiels <driesm.michiels@gmail.com>

Comment 6 Dan Langille freebsd_committer 2018-09-02 14:09:09 UTC
Thank you.
Comment 7 Dries Michiels 2018-09-02 14:17:46 UTC
Thank you for updating it to newest version!