BIND 10 #1071: introduce "null" output for logs

[BIND 10 Development]

#1071: introduce "null" output for logs
-------------------------------------+-------------------------------------
                   Reporter:         |                 Owner:  stephen
  jinmei                             |                Status:  reviewing
                       Type:  task   |             Milestone:
                   Priority:  major  |  Sprint-20110712
                  Component:         |            Resolution:
  logging                            |             Sensitive:  0
                   Keywords:         |           Sub-Project:  DNS
            Defect Severity:  N/A    |  Estimated Difficulty:  0.0
Feature Depending on Ticket:         |           Total Hours:  0
        Add Hours to Ticket:  0      |
                  Internal?:  0      |
-------------------------------------+-------------------------------------
Changes (by vorner):

 * owner:  vorner => stephen


Comment:

 Hello

 I don't know if I look correctly, but this seems to be more setting-up of
 logging options through environment variables. There seems to be no code
 regarding any null output. Do I look at a wrong branch or is it related in
 some way I don't see?

 Anyway, the code looks useful, so here are comments to it:
  * This doesn't seem to be completely true (the part about „as is“, the
 %1, etc are replaced):
 {{{
   * The rest of the line - from the first non-space character to the last
 non-
     space character - is taken exactly as-is for the text of the message.
 There
 }}}
  *  „The main program unit should include a call to isc.log.init()“ ‒
 wouldn't „must“ be better?
  * Is this a typo or word I don't know? „Checks that the logger will limit
 the output of messages less _severy_ than“?
  * In the tests, when the output is routed to stdout or stderr, wouldn't
 it be better to use a single pipeline instead of (slower and more error
 prone) store-to-file and load again? Like this?:
 {{{
 B10_LOGGER_SEVERITY=WARN ./init_logger_test 2>&1 | cut -d' ' -f3- | diff
 $tempfile -
 }}}
  * Also, this looks really strange. Why is the „1“ there?
 {{{
 ./init_logger_test 1> $destfile
 }}}

 With regards

-- 
Ticket URL: <http://bind10.isc.org/ticket/1071#comment:7>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development