Boost logo

Boost :

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
> Apparently, it's a part of graph, not a library in its own right. From
> 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

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