Re: [boost] [process] Formal Review starts today, 27 October

Subject: Re: [boost] [process] Formal Review starts today, 27 October
From: Paul A. Bristow (pbristow_at_[hidden])
Date: 2016-10-28 05:57:41

> -----Original Message-----
> From: Boost [mailto:boost-bounces_at_[hidden]] On Behalf Of Antony Polukhin
> Sent: 27 October 2016 07:27
> To: boost_at_[hidden] List
> Subject: [boost] [process] Formal Review starts today, 27 October
> 
> Dear Boost community,
> 
> The formal review of Klemens David Morgenstern's Process library
> begins today, 27th October and ends on 5th November.
> 
>  Process is a C++11 library to manage system processes. It can be used to:
> * create child processes
> * setup streams for child processes
> * communicate with child processes through streams (synchronously or
> asynchronously)
> * wait for processes to exit (synchronously or asynchronously)
> * terminate processes
> 
> Full documentation with examples and tutorial is available at
> http://klemens-morgenstern.github.io/process/index.html
> Stable source codes for review are available at
> https://github.com/klemens-morgenstern/boost-process/tree/boost_review
> Latest source codes available at
> https://github.com/klemens-morgenstern/boost-process
> 
> * Whether you believe the library should be accepted into Boost
Yes.

>   * Conditions for acceptance
None.

> - Your knowledge of the problem domain
Modest.
 
> You are strongly encouraged to also provide additional information:
> - What is your evaluation of the library's:

>   * Design
Many years in evolution and time for widespread use.
Wisely does not attempt a DSL, but appears to provide the tools needed to write DSLs?
(Might be useful for a 'Son of Bjam' being discussed at present?).

The passing of parameters to processes appears contentious. But the whole passing of parameters in C++ and arguments to compilers
and tools remains  pre-last-millennium - so that it is difficult to avoid the current muddled messiness.

>   * Implementation
In use.  Mature-ish.

>   * Documentation
Excellent - only a few dysfunctional links but these (and a few typos) that will no doubt be fixed when implemented in the main
Boost tree.  C++ Reference section already working helpfully using Doxygen-syntax comments in the code.  Includes examples where
helpful. Only fails to use code snippets to ensure that 'What You See is What Compiles'.  *nix-centric making excessive assumptions
about Windows programmers?  An index would be useful.

>   * Tests
Not studied but tests do exist (as do examples).  More (both tests and examples), especially for Windows, would be always be good.

>   * Usefulness
Very useful, and in real-life use.

> - Did you attempt to use the library?
No.

> - How much effort did you put into your evaluation of the review?
A brief read of documentation (and reading the development of the library over the years on the Boost lists).

Paul

---
Paul A. Bristow
Prizet Farmhouse
Kendal UK LA8 8AB
[phone]