net-mgmt/librenms: Upgrade to 1.45
ClosedPublic
Actions

Authored by dvl on Oct 24 2018, 7:24 PM.

Details

Reviewers

bofh
dvl

Summary

Patch files to make validate.php make sense for a package:

LibreNMS/Validations/Dependencies.php
LibreNMS/Validations/Programs.php
LibreNMS/Validations/User.php

Patch check_git_exists() in files/patch-includes_common.php to return false.

Mention:

sql-mode="" is no longer required
build-base.php is what you run after an upgrade

Test Plan

1.44: https://services.unixathome.org/poudriere/build.html?mastername=112amd64-default-master-list&build=2018-10-24_19h05m24s
1.45: https://services.unixathome.org/poudriere/build.html?mastername=112amd64-working&build=2018-10-31_14h18m14s

Diff Detail

Repository

rP FreeBSD ports repository

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 20571
Build 19992: arc lint + arc unit

Event Timeline

dvl created this revision.Oct 24 2018, 7:24 PM

Herald added a subscriber: mat. · View Herald TranscriptOct 24 2018, 7:24 PM

Harbormaster completed remote builds in B20405: Diff 49578.Oct 24 2018, 7:24 PM

dvl retitled this revision from Upgrade to 1.44 Patch files to make validate.php make sense for a package: * LibreNMS/Validations/Dependencies.php * LibreNMS/Validations/Programs.php * LibreNMS/Validations/User.php Patch check_git_exists() in files/patch-includes_common.php to... to Upgrade to 1.44.Oct 24 2018, 7:27 PM

dvl edited the summary of this revision. (Show Details)

dvl edited the summary of this revision. (Show Details)Oct 24 2018, 7:29 PM

linimon retitled this revision from Upgrade to 1.44 to net-mgmt/librenms: Upgrade to 1.44.Oct 27 2018, 5:59 AM

LibreNMS 1.45 has since been released. Update patch for that.

Harbormaster completed remote builds in B20571: Diff 49879.Nov 1 2018, 12:59 PM

dvl retitled this revision from net-mgmt/librenms: Upgrade to 1.44 to net-mgmt/librenms: Upgrade to 1.45.Nov 1 2018, 1:00 PM

dvl edited the test plan for this revision. (Show Details)

Commited: https://svnweb.freebsd.org/changeset/ports/484452

I should have referenced this review in the commit.

This revision is now accepted and ready to land.Nov 8 2018, 1:38 PM

dvl closed this revision.Nov 8 2018, 1:38 PM