Page MenuHomeFreeBSD

net-mgmt/icinga2: Use .conf suffix for /usr/local/etc/newsyslog.conf.d/

Authored by woodsb02 on Sep 9 2018, 8:40 AM.
Referenced Files
F72303256: D17091.diff
Tue, Dec 5, 8:32 AM
Unknown Object (File)
Mon, Dec 4, 12:30 AM
Unknown Object (File)
Tue, Nov 28, 11:39 PM
Unknown Object (File)
Tue, Nov 28, 11:39 PM
Unknown Object (File)
Tue, Nov 28, 11:39 PM
Unknown Object (File)
Mon, Nov 27, 1:30 PM
Unknown Object (File)
Mon, Nov 27, 11:16 AM
Unknown Object (File)
Wed, Nov 22, 6:37 AM



net-mgmt/icinga2: Use .conf suffix for /usr/local/etc/newsyslog.conf.d/

Rename files intended for installation to /usr/local/etc/newsyslog.conf.d/
to end with a '.conf' suffix.

Proposed changes to /etc/newsyslog.conf will only include files from the
/usr/local/etc/newsyslog.conf.d/ directory which end with '.conf' and do
not beginning with a '.' character.

Approved by: lme (maintainer)

Test Plan

poudriere testport -j 12amd64: ok

Diff Detail

rP FreeBSD ports repository
Lint Not Applicable
Tests Not Applicable

Event Timeline

Abandon this change - relocating existing conf files violates POLA.

D17086 has now been committed to 13-CURRENT.
Re-opening this review.
New proposal to minimise the possibility that people are caught out unaware of this:

  1. Modify port to install file to newsyslog/*.conf
  2. Add move_newsyslog_conf() to pkg-install to automatically relocate any existing copy that is missing the .conf file
  3. Add note to UPDATING and pkg-message to notify the user of this change (important if provisioning tools such as ansible, puppet, salt, etc are in use).

Thanks! Feel free to commit it.

This revision is now accepted and ready to land.Nov 10 2018, 11:29 AM
This revision was automatically updated to reflect the committed changes.