3334 zonestat missing man page

Review Request #1658 — Created April 3, 2019 and submitted

citrus
illumos-gate
master
3334
ae4a2ff...
general
3334 zonestat missing man page

This is derived from the man pages at https://us-east.manta.joyent.com/jmc/public/opensolaris/ARChive/PSARC/2010/291/index.html

I added an Oracle copyright for that year.

See http://cr.illumos.org/~webrev/citrus/ig_zonestat/ for rendered versions of the man pages.

  • 0
  • 0
  • 5
  • 1
  • 6
Description From Last Updated
oetiker
  1. lgtm

  2. 
      
seeemef@mac.com
  1. 
      
  2. usr/src/man/man1m/zonestatd.1m (Diff revision 1)
     
     
  3. 
      
jbk
  1. Just a few minor nits and questions.

  2. usr/src/man/man1/zonestat.1 (Diff revision 1)
     
     

    Maybe add '...' i.e.
    .It Fl z Ar zonename Ns Oo Ar ,zonename Ns ... Oc

  3. usr/src/man/man1/zonestat.1 (Diff revision 1)
     
     

    Similar here for ...

  4. usr/src/man/man1/zonestat.1 (Diff revision 1)
     
     

    Doesn't the '.)' (no space) trip mandoc -Tlint ?

    1. No, lint is happy, but there's an extra space in the output.
      I changed this line to remove the brackets (and the problem).

  5. usr/src/man/man1/zonestat.1 (Diff revision 1)
     
     

    Maybe .Dv SIGINT ?

  6. usr/src/man/man1/zonestat.1 (Diff revision 1)
     
     

    Probably not necessary to do .Ex -std if you're already listing all the return values.

    1. I think the summary sentence is still appropriate.

  7. 
      
citrus
seeemef@mac.com
  1. LGTM

  2. 
      
domag02
  1. Ship It!
  2. 
      
citrus
Review request changed

Status: Closed (submitted)

Loading...