Bug 225161 - [PATCH] net-mgmt/librenms: Update to 1.35, many improvements
Summary: [PATCH] net-mgmt/librenms: Update to 1.35, many improvements
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Mark Felder
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2018-01-14 17:40 UTC by Mark Felder
Modified: 2018-01-18 20:29 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Felder freebsd_committer 2018-01-14 17:40:28 UTC
Please see the patch at the following review:

https://reviews.freebsd.org/D13907


Thanks
Comment 1 commit-hook freebsd_committer 2018-01-18 20:22:00 UTC
A commit references this bug:

Author: feld
Date: Thu Jan 18 20:21:43 UTC 2018
New revision: 459372
URL: https://svnweb.freebsd.org/changeset/ports/459372

Log:
  net-mgmt/librenms: Update to 1.35, many improvements

  Improvements:

  -    All files should be owned root:wheel except logs and rrd which need to be writable by the app
  -    Add missing php posix extension
  -    Do not install config.php by default. This breaks the install process which won't run if this file exists
  -    Clean up automatic PLIST creation: don't install .orig or .bak files, don't add @dir as they aren't needed
  -    Patch LibreNMS to make /validate/ page not produce warnings about files not being writable (for git updates)
  -    Remove the Updates validation check altogether as we won't be using git to update
  -    Patch the User validation check to only check the logs and rrd dir and ensure the correct user owns them
  -    Change the default user in the generated config to "www"
  -    Patch the File Lock code to put the lock file in /tmp and not in the WWWDIR which should not be writable
  -    Update message in installer to use WWWDIR as suggested path for config.php
  -    Use shebangfix instead of patch where applicable
  -    Fix APACHEMOD port option and declaration of the USES=php

  PR:		225161
  MFH:		2018Q1
  Differential Revision:	https://reviews.freebsd.org/D13907

Changes:
  head/net-mgmt/librenms/Makefile
  head/net-mgmt/librenms/distinfo
  head/net-mgmt/librenms/files/patch-LibreNMS_Util_FileLock.php
  head/net-mgmt/librenms/files/patch-LibreNMS_Validations_User.php
  head/net-mgmt/librenms/files/patch-config.php.default
  head/net-mgmt/librenms/files/patch-html_install.php
  head/net-mgmt/librenms/files/patch-includes_defaults.inc.php
  head/net-mgmt/librenms/files/patch-poller-wrapper.py
  head/net-mgmt/librenms/files/patch-poller.php
Comment 2 commit-hook freebsd_committer 2018-01-18 20:24:03 UTC
A commit references this bug:

Author: feld
Date: Thu Jan 18 20:23:32 UTC 2018
New revision: 459374
URL: https://svnweb.freebsd.org/changeset/ports/459374

Log:
  MFH: r459372

  net-mgmt/librenms: Update to 1.35, many improvements

  Improvements:

  -    All files should be owned root:wheel except logs and rrd which need to be writable by the app
  -    Add missing php posix extension
  -    Do not install config.php by default. This breaks the install process which won't run if this file exists
  -    Clean up automatic PLIST creation: don't install .orig or .bak files, don't add @dir as they aren't needed
  -    Patch LibreNMS to make /validate/ page not produce warnings about files not being writable (for git updates)
  -    Remove the Updates validation check altogether as we won't be using git to update
  -    Patch the User validation check to only check the logs and rrd dir and ensure the correct user owns them
  -    Change the default user in the generated config to "www"
  -    Patch the File Lock code to put the lock file in /tmp and not in the WWWDIR which should not be writable
  -    Update message in installer to use WWWDIR as suggested path for config.php
  -    Use shebangfix instead of patch where applicable
  -    Fix APACHEMOD port option and declaration of the USES=php

  PR:		225161
  Differential Revision:	https://reviews.freebsd.org/D13907

Changes:
_U  branches/2018Q1/
  branches/2018Q1/net-mgmt/librenms/Makefile
  branches/2018Q1/net-mgmt/librenms/distinfo
  branches/2018Q1/net-mgmt/librenms/files/patch-LibreNMS_Util_FileLock.php
  branches/2018Q1/net-mgmt/librenms/files/patch-LibreNMS_Validations_User.php
  branches/2018Q1/net-mgmt/librenms/files/patch-config.php.default
  branches/2018Q1/net-mgmt/librenms/files/patch-html_install.php
  branches/2018Q1/net-mgmt/librenms/files/patch-includes_defaults.inc.php
  branches/2018Q1/net-mgmt/librenms/files/patch-poller-wrapper.py
  branches/2018Q1/net-mgmt/librenms/files/patch-poller.php
Comment 3 Mark Felder freebsd_committer 2018-01-18 20:29:28 UTC
Resolving. Thanks Muhammad!!!