Bug 200955
| Summary: | [patch] Update net-mgmt/victorops-nagios to 1.3.16 | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Brad Davis <brd> | ||||||||||
| Component: | Individual Port(s) | Assignee: | freebsd-ports-bugs (Nobody) <ports-bugs> | ||||||||||
| Status: | Closed Overcome By Events | ||||||||||||
| Severity: | Affects Only Me | CC: | ports-bugs | ||||||||||
| Priority: | --- | Keywords: | patch | ||||||||||
| Version: | Latest | ||||||||||||
| Hardware: | Any | ||||||||||||
| OS: | Any | ||||||||||||
| Attachments: |
|
||||||||||||
Not sure why this was assigned to portmgr.. Created attachment 157864 [details]
poudriere build log
Note that I do not have a victorops account, so I just forward ported the existing patches and simplified them a little. Created attachment 158023 [details]
Updated patch to bring victorops-nagios to version 1.3.16.
I took Brad Davis' patch and worked on it a bit.
This patch leaves some empty patch-* in the files directory that should be removed.
Needed to add coreutils dependency to get gtimeout added, and added the missing ca_root_nss dependency.
What about the @sample parts of the plist? Created attachment 158024 [details]
Fixed patch with @sample in pkg-plist
Oops, forgot the @sample decorations in the pkg-plist.
Thanks for the reminder Brad.
Brad, thanks for the work on this. I made some changes to your patch: * Added coreutils to RUN_DEPENDS so that I could get gtimeout. And tweaked the patch-check_victorops_cmds patch to use it. * I kept the -a option to psin patch-check_victorops_forwarder to make sure we looked at all processes * I updated the qpage-patch to VictroOps new style. Sorry that vacation intruded on getting on to this quickly. The attached patch orphans some empty patch files that should be removed: * patch-do_victorops_heartbeat * patch-enqueue_alert * patch-forward_alerts * patch-support_collect.sh * patch-victorops.cron Looks good, with your approval I will test and commit. Sorry for the delay Brad. (Email response seems delayed somewhere.) Go right ahead. Thanks. Superseded by Bug 206350. Not sure why this was Assigned to a non-committer (though it turned out swimmingly!) |
Created attachment 157863 [details] patch