Re: [boost] [graph-parallel] bcp update, was: The boost component dependencies blues

Subject: Re: [boost] [graph-parallel] bcp update, was: The boost component dependencies blues
From: Jeremiah Willcock (jewillco_at_[hidden])
Date: 2009-12-17 08:45:04

On Thu, 17 Dec 2009, Daniel James wrote:

> 2009/12/17 John Maddock <john_at_[hidden]>:
>>
>> * On a related note: the parallel graph lib does not appear in the library
>> list at http://www.boost.org/doc/libs/1_41_0.
>
> Apparently, it's a part of graph, not a library in its own right. From
> http://lists.boost.org/Archives/boost/2009/08/155277.php
>
> 2009/8/21 Jeremiah Willock:
>> This code could not be directly within libs/graph/ because there would be
>> a circular dependency: graph_parallel depends on mpi, which depends on
>> graph. I had planned to put the parallel graph code into
>> libs/graph/parallel/ (like the headers are within boost/), but the CMake
>> build system for Boost is unable to handle a library that is not at the
>> top level of libs/ as far as I know. I do not believe moving the code to
>> libs/graph/parallel would be a problem for Boost.Build. I am still open
>> to that move (I would actually prefer graph/parallel/) if CMake is not
>> considered to be a problem.
>
> I missed this at the time. This is the kind of thing that the release
> managers should be told about. Do we need to make that clearer
> somewhere?
>
> I think parallel graph really should be in a subdirectory of graph.
> Cmake should understand that libraries in subdirectories are separate,
> we do it in a few other places. Is it too late to move it now?
>
> And since it's a part of graph, the main graph documentation should
> link to it. Probably on the first page of its documentation.

I don't have a problem with the move; there would probably still need to 
be two library objects (even though there are no dependencies between the 
object files for graph and graph_parallel, just the headers), but we can 
put them in the same source tree and merge the docs.  Are there any bjam 
issues with subdirectories in library directories?  The redirect issue is 
fixed, but the new directory name is graph_parallel.  I just added the 
links to the documentation.

-- Jeremiah Willcock