Page MenuHomeFreeBSD

newsyslog.conf.5: Add a CAVEAT
Needs ReviewPublic

Authored by ziaee on Thu, Feb 5, 4:05 AM.

Details

Reviewers
feld
michaelo
Group Reviewers
manpages
Summary

PR: 282639

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 70426
Build 67309: arc lint + arc unit

Event Timeline

ziaee requested review of this revision.Thu, Feb 5, 4:05 AM

Stupid question: Should we (then) mention execution order as well?

Should we (then) mention execution order as well?

I don't know, is there anything special about the execution order? I've never had a complicated enough newsyslog setup to run into this problem.

Stupid question:

I don't know that there are stupid questions in multi-generational, international systems engineering. We absolutely must consider corner cases, and we absolutely must make things as obvious as possible. We have no idea who will be working on this next, or under what kind of duress, or which SMEs will get hit by a bus. Therefore, all questions from competent people that seem stupid are highlighting some sort of deficiency in obviousness.

This is also why I am inclined to screaming when competency questions are raised over apparent stupid questions towards people who have earned their stripes. This is a tier 1 planetary system and things need to be made obvious.

Should we (then) mention execution order as well?

I don't know, is there anything special about the execution order? I've never had a complicated enough newsyslog setup to run into this problem.

Me either, but as far as I can see (I could be wrong) main() builds a list of conf entries and processes them in order. So I think that your change is just fine. Let's wait for the others.