11190 update mandoc to 1.14.5

Review Request #1975 — Created June 6, 2019 and submitted

Information
mno-hime
illumos-gate
master
11190
1974, 1973, 1972, 1971, 1970
da42601...
Reviewers
general 
11190 update mandoc to 1.14.5
I tested updated mandoc with a clean illumos build, thus triggering -Tlint -Wwarning on all mandoc-style man pages, and, thanks to https://illumos.org/rb/r/1974/, I also linted all old man pages for errors.


Displayed several man pages from illumos and OI userland with the updated mandoc, generated webrev and verified that man page is displayed sanely as TXT and HTML.
  • 0
  • 0
  • 2
  • 0
  • 2
Description From Last Updated
mno-hime
mno-hime
rm
rm 
  1. In general, this looks good. Thanks for putting this together. I have a few minor notes on the updated documentation as it applies to us.
    1. Thank you for the review! I'll address those two minor concerns in https://www.illumos.org/issues/11232.
    2. It would be nice not to integrate the manual page with it pointing to a known bad location.
    3. You are right. I fixed that.
  2. usr/src/man/man1/mandoc.1 (Diff revision 2) 
     
     
     
    This path isn't valid on us. I'm not sure if we should ship this or if we should just change it back to being an unqualified path.
  3. usr/src/man/man1/mandoc.1 (Diff revision 2) 
     
     
     
    See earlier path comment.
  4. 
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    5. 




  

 
















 





 

  

   




domag02




   

   









  

 



 

  


   

   

    
 
domag02


    
   


  

  

  



    




  1. 
 
    
  
    
   
    White space nits.
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    2. 

















  2. 
 
    
  
  
    
  
 
    
  
    
   
    
    
     
      usr/src/tools/scripts/webrev.sh
      


       (Diff revision 2)


      
     
    
    
       
   
    
     
    

    
     
    

       
  
    
 
    

   
    
    
    



    
 
    
  
    
   
   
    
    
     
     
     
     
     
    
    Show all issues
   
  
    
 
    

    



    
    Trailing white space.
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    3. 






  3. 
 
    
  
  
    
  
 
    
  
    
   
    
    
     
      usr/src/tools/scripts/webrev.sh
      


       (Diff revision 2)


      
     
    
    
       
   
    
     
    

    
     
    

       
  
    
 
    

   
    
    
    



    
 
    
  
    
   
   
    
    
     
     
     
     
     
    
    Show all issues
   
  
    
 
    

    



    
    This line contains only blanks.
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    4. 




  4. 
 
    
  
    
   
    
  
    
 
    

 
    
  


    
 
    
 
      

  

 
    


    

 
    

    5. 




  

 
















 



 

  

   




mno-hime




   

   

  

 



 















 



 

  

   




mno-hime




   

   

  

 



 















 



 

  

   




mno-hime




   

   

  

 



 

  


   

   

    
 Review request changed

    
   


  

  

  






 

  Status:
  

  Closed (submitted)

  
 







 


 










  

 









     

    

   

  

  

   
   Loading...