formal_review_process.htm 6.2 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138
  1. <html>
  2. <head>
  3. <meta http-equiv="Content-Type" content="text/html; charset=windows-1252">
  4. <meta name="GENERATOR" content="Microsoft FrontPage 4.0">
  5. <meta name="ProgId" content="FrontPage.Editor.Document">
  6. <title>Boost Formal Review Process</title>
  7. </head>
  8. <body bgcolor="#FFFFFF" text="#000000">
  9. <table border="1" bgcolor="#007F7F" cellpadding="2">
  10. <tr>
  11. <td bgcolor="#FFFFFF"><img src="../c++boost.gif" alt="c++boost.gif (8819 bytes)" width="277" height="86"></td>
  12. <td><a href="../index.htm"><font face="Arial" color="#FFFFFF"><big>Home</big></font></a></td>
  13. <td><a href="../libs/libraries.htm"><font face="Arial" color="#FFFFFF"><big>Libraries</big></font></a></td>
  14. <td><a href="../people/people.htm"><font face="Arial" color="#FFFFFF"><big>People</big></font></a></td>
  15. <td><a href="faq.htm"><font face="Arial" color="#FFFFFF"><big>FAQ</big></font></a></td>
  16. <td><a href="index.htm"><font face="Arial" color="#FFFFFF"><big>More</big></font></a></td>
  17. </tr>
  18. </table>
  19. <h1>Boost Formal Review Process</h1>
  20. <p>The Formal Review process determines if a proposed library should be accepted
  21. as a Boost library.</p>
  22. <p>The final &quot;accept&quot; or &quot;reject&quot; decision is made by the <a href="#Review Manager">
  23. Review Manager</a>, based on review comments received from boost mailing list
  24. members.</p>
  25. <p>Reviews are scheduled so that:</p>
  26. <ul>
  27. <li>Each library gets review time proportional to its size, complexity, and
  28. importance.</li>
  29. <li>Only a small number of libraries are under active review at any one time.</li>
  30. <li>Members know ahead of time when a particular library will be under active
  31. review.&nbsp;</li>
  32. </ul>
  33. <h2>Comments</h2>
  34. <p> Boost mailing list members are encouraged to submit Formal Review
  35. comments:</p>
  36. <blockquote>
  37. <ul>
  38. <li>Publicly on the mailing list.</li>
  39. <li>Privately to the Review Manager.</li>
  40. <li>Privately to the library submitter.</li>
  41. </ul>
  42. </blockquote>
  43. <p>Private comments to a library submitter may be helpful to her or him, but
  44. won't help the Review Manager reach a decision, so the other forms are
  45. preferred.</p>
  46. <h2>Preparing review comments</h2>
  47. <p>Comments may be brief or lengthy, but basically the Review Manager needs your
  48. answers to several questions.&nbsp; If you identify problems along the way,
  49. please note if they are minor, serious, or showstoppers.</p>
  50. <ul>
  51. <li>What is your evaluation of the design?<br>
  52. </li>
  53. <li>What is your evaluation of the implementation?<br>
  54. </li>
  55. <li>What is your evaluation of the documentation?<br>
  56. </li>
  57. <li>What is your evaluation of the potential usefulness of the library?<br>
  58. </li>
  59. <li>Did you try to use the library?&nbsp; With what compiler?&nbsp; Did you
  60. have any problems?<br>
  61. </li>
  62. <li>How much effort did you put into your evaluation? A glance? A quick
  63. reading? In-depth study?<br>
  64. </li>
  65. <li>Are you knowledgeable about the problem domain?<br>
  66. </li>
  67. <li>Finally, do you think the library should be accepted as a Boost
  68. library?&nbsp; Be sure to say this explicitly so that your other comments
  69. don't obscure your overall opinion.</li>
  70. </ul>
  71. <h2>Results</h2>
  72. <p>At the conclusion of the comment period, the Review Manager will post a
  73. message to the mailing list saying if the library has been accepted or
  74. rejected.&nbsp; A rationale should be provided, but its extent is up to the
  75. Review Manager.</p>
  76. <h2>Notes for <a name="Review Manager"> Review Manager</a>s</h2>
  77. <p>Before a library can be scheduled for formal review, an active boost member not
  78. connected with the library submission must volunteer to be the &quot;Review
  79. Manager&quot; for the library.</p>
  80. <p>The review manager:</p>
  81. <ul>
  82. <li>Checks the submission to make sure it really is complete enough to warrant
  83. formal review.&nbsp; See the <a href="lib_guide.htm">Boost Library
  84. Requirements and Guidelines</a>.&nbsp; If necessary, work with the submitter
  85. to verify the code compiles and runs correctly on several compilers and
  86. platforms.</li>
  87. <li>Sets the schedule, after discussion with the boost webmaster, currently
  88. Beman Dawes.</li>
  89. <li>Posts a notice of the review schedule on both the regular <b> boost mailing
  90. list</b> and the <b> boost-announce mailing list</b>.</li>
  91. </ul>
  92. <blockquote>
  93. <blockquote>
  94. <p>The notice should include a brief description of the library and what it
  95. does, to let readers know if the library is one they are interested in
  96. reviewing.</p>
  97. </blockquote>
  98. </blockquote>
  99. <ul>
  100. <li>Inspects the Boost <a href="../libs/libraries.htm">library catalogue</a> for
  101. libraries which may interact with the new submission. These potential
  102. interactions should be pointed out in the review announcement, and the
  103. author(s) of these libraries should be privately notified and urged to
  104. participate in the review.
  105. <li>Urges people to do reviews if they aren't forthcoming.</li>
  106. <li>Follows review discussions regarding the library, moderating or answering
  107. questions as needed.</li>
  108. <li>Decides if
  109. there is consensus to accept the library, and if there are any conditions
  110. attached.</li>
  111. <li>Posts a message on both the <b>boost</b> and <b>boost-announce</b> lists informing members of the review
  112. results.</li>
  113. </ul>
  114. <p>In other words, it is the Review Manager's responsibility to make sure the
  115. review process works smoothly.</p>
  116. <h2>Notes for Library Authors</h2>
  117. <p>A proposed library should remain stable during the review period; it will just
  118. confuse and irritate reviewers if there are numerous changes.&nbsp; It
  119. is, however, useful to upload fixes for serious bugs right away, particularly
  120. those which prevent reviewers from fully evaluating the library.&nbsp; Post a
  121. notice of such fixes on the mailing list.</p>
  122. <p>Library improvements suggested by reviewers should be held until after the
  123. completion of review period. Such changes can then be incorporated in the <a href="submission_process.htm#Final">final
  124. submission file</a> sent to the webmaster for posting.&nbsp; If the suggested
  125. changes might affect reviewer's judgments,&nbsp; post a notice of the pending
  126. change on the mailing list.</p>
  127. <hr>
  128. <p>Revised <!--webbot bot="Timestamp" s-type="EDITED" s-format="%d %B, %Y" startspan -->11 October, 2001<!--webbot bot="Timestamp" endspan i-checksum="38292" --></p>
  129. <p>&nbsp;</p>
  130. </body>
  131. </html>
粤ICP备19079148号