Summary: | net-mgmt/net-snmp: Install newsyslog config as a sample | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | James French <james> | ||||||||||
Component: | Individual Port(s) | Assignee: | Ryan Steinmetz <zi> | ||||||||||
Status: | Closed Overcome By Events | ||||||||||||
Severity: | Affects Some People | CC: | driesm, james | ||||||||||
Priority: | --- | Keywords: | buildisok, easy, feature | ||||||||||
Version: | Latest | Flags: | bugzilla:
maintainer-feedback?
(zi) koobs: merge-quarterly? |
||||||||||
Hardware: | Any | ||||||||||||
OS: | Any | ||||||||||||
Attachments: |
|
Created attachment 216528 [details]
Poudriere build log
The attached patch only works if newsyslog.conf.d already exists. Putting together a correction now. Created attachment 216530 [details]
Patch to install newsyslog.conf.d/net-snmp.conf as a @sample
Changes:
- Installs the newsyslog file under share/snmp/newsyslog.conf.sample
- Installation has been moved to the install-config rather than post-install
Created attachment 216531 [details]
Poudriere build log
^Triage: [tags] in issue Titles are deprecated Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/173342776 *** Bug 248466 has been marked as a duplicate of this bug. *** Any chance we could get this committed under maintainer-timeout please? With bug 248466 as evidence there's a few people running into this issue. This will be included in the update to 5.9 (In reply to Ryan Steinmetz from comment #9) Perfect, thanks for the feedback! Committed as part of the 5.9 update |
Created attachment 216527 [details] Patch to install newsyslog.conf.d/net-snmp.conf as a @sample net-mgmt/net-snmp ships with a sample newsyslog.conf file that gets installed into /usr/local/etc/newsyslog.conf.d. If a user decides to make changes to this file then those changes: - Get overwritten when the package is updated - Get complained about by periodic-security because the installed file no longer matches the package checksum This patch instead installs the sample file under ${EXAMPLESDIR}/newsyslog.conf and uses @sample to install it under etc/newsyslog.conf.d.