Boost logo

Boost :

Subject: Re: [boost] Formal Review Request: Generic Geometry Library (GGL).
From: John Phillips (phillips_at_[hidden])
Date: 2009-10-13 20:48:04

vicente.botet wrote:
> I think I understand how the system works. And I know I have a lot of
> things to learm from this list, as for example, what is the good way
> to signal something is missing or wrong. I will try to do better next
> time :)
> I hope this will help you to understand what was my initial concern:
> Should we accept a formal review request when there is no doc, no
> test and no examples?
> Best regards, Vicente

   I think I would phrase things differently, to place the emphasis in a
better place. Should we schedule a review for a library that is missing
any of docs, tests, examples, ... . No, we should not. That is why the
prospective review manager is expected to look at the library before
scheduling the review. Not as a formal review itself, but to make sure
that the library is in good enough shape to warrant a review at all.

   However, we don't have a good way to make that check before a library
author requests a review on the developer list. In fact, in some cases
the first anyone on the list hears about a library is the request for
review. (This is not a process I suggest, nor is it usually successful,
but it does happen.)

   There have been library submitters in the past that requested review
for libraries that just weren't ready for what Boost expects. In some
cases, interested review managers have worked with the submitter to help
them understand the expectations and more thoroughly develop the library
before asking for a review date. However, there is no history of the
review wizards checking the library before allowing an author to request
a review.

   To date, the role of the wizards has been more administration and
advice than checking library details. Most of our review managers are
quite good, and so there is no need for us to try and micromanage their
work. We try to stay informed, and on the rare occasion that we need to
step in on the details we do so, but that is thankfully rare. (All
review managers deserve to be thanked for making it rare, by the way.)


Boost list run by bdawes at, gregod at, cpdaniel at, john at