Differential D11664

syslog.d
AbandonedPublic
Actions

Authored by ngie on Jul 19 2017, 9:38 PM.

Details

Reviewers: None

Summary

Further modularize etc/newsyslog.conf.d and etc/syslog.d

etc/newsyslog.conf.d:

place cron(8), init(8), and periodic(8) entries into etc/newsyslog.conf.d/cron.conf, etc/newsyslog.conf.d/init.conf, and etc/newsyslog.conf.d/periodic.conf, respectively.
place devd and kerberos entries into etc/newsyslog.conf.d/devd.conf and etc/newsyslog.conf.d/kerberos.conf, respectively. Conditionalize dev.conf on MK_CXX and conditionalize kerberos.conf on MK_KERBEROS.

etc/syslog.d:

place cron(8) and news entries into etc/syslog.d/cron.conf and etc/syslog.d/news.conf, respectively.
place commented out devd entries into etc/syslog.d/devd.conf. Make it conditional on MK_CXX.

MFC after: never
Requested by: jhb

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 10572
Build 10978: arc lint + arc unit

Event Timeline

ngie created this revision.Jul 19 2017, 9:38 PM

Herald added a subscriber: imp. · View Herald TranscriptJul 19 2017, 9:38 PM

Harbormaster completed remote builds in B10572: Diff 30985.Jul 19 2017, 9:38 PM

ngie edited the summary of this revision. (Show Details)Jul 20 2017, 12:29 AM

Ping? I'd like to get this in before 08/01/2017, if there aren't any blocking issues that have been brought up.

The patch no longer applies.

While I see the merit in continuing this path, a bit, the only thing that it really makes sense to do this with, is devd. The changes for devd are applicable to newsyslog.conf, but don't apply to syslog.conf .