Page MenuHomeFreeBSD

www/owncloud: Use .conf suffix for /usr/local/etc/newsyslog.conf.d/
ClosedPublic

Authored by woodsb02 on Sep 9 2018, 7:28 AM.
Tags
None
Referenced Files
F99519013: D17090.diff
Thu, Oct 10, 10:14 AM
F99513687: D17090.id.diff
Thu, Oct 10, 9:07 AM
F99497736: D17090.diff
Thu, Oct 10, 5:54 AM
F99426524: D17090.id.diff
Wed, Oct 9, 12:12 PM
Unknown Object (File)
Sun, Oct 6, 9:44 PM
Unknown Object (File)
Sat, Oct 5, 9:17 PM
Unknown Object (File)
Sat, Oct 5, 8:27 AM
Unknown Object (File)
Fri, Oct 4, 5:41 PM
Subscribers

Details

Summary

www/owncloud: Use .conf suffix for /usr/local/etc/newsyslog.conf.d/

Rename the files installed 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. https://reviews.freebsd.org/D17086

Approved by: sunpoet (maintainer)

Test Plan

poudriere testport -j 12amd64: ok

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 20741
Build 20148: arc lint + arc unit

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).
  • Add pkg-install script to automatically move old newsyslog file to new filename
  • Add entries to UPDATING and pkg-message
This revision was not accepted when it landed; it landed in state Needs Review.Nov 24 2018, 3:44 AM
This revision was automatically updated to reflect the committed changes.