Subject: Re: [boost] [graph] disjoint_sets
From: James E. King III (jking_at_[hidden])
Date: 2018-07-16 21:55:10
On Mon, Jul 16, 2018 at 5:17 PM Marc Glisse <marc.glisse_at_[hidden]> wrote:
> On Mon, 16 Jul 2018, James E. King III via Boost wrote:
> > Hello folks in the Boost.Graph team,
> > The CMT currently maintains Boost.DisjointSets which contains a class
> > is only used by Boost.Graph and Boost.GraphParallel. The headers are
> > published in "boost/pending" right now. Given the CMT has a considerable
> > workload already I would like to discuss moving disjoint_sets into
> > Boost.Graph, as long as it satisfies the requirements for the two
> > repositories currently consuming it. This would be for 1.69.0.
> > Any thoughts, objections, considerations on this (open to anyone)?
> You could also become a maintainer for Boost.DisjointSets without merging
> it with another package. I don't mind the merge, I am just saying the two
> need not be linked.
> The headers might be in boost/pending, but there are many external users.
> Please try not to break them gratuitously. If you move the header to a
> different subdirectory, please put in the old position a dummy header that
> includes the new one, etc.
> Marc Glisse
Providing a forward is easy enough, and certainly possible to avoid things
Nobody should be relying on a header that contains "boost/pending" in the
This repository was created with "pending" headers then abandoned.
Maintaining a separate repository for one class is incredibly expensive.
If you factor in the
CI requirements for unix, windows, code coverage, static code analysis it
becomes very expensive.
We have the same issue with Boost.Logic (boost::logic::tribool is the only
class) for example, but
there is a much higher likelihood that A) there are external users or B)
more boost::logic will appear
in the future.
These things said, will you accept a PR into Boost.Graph adding
I can set it up so that it retains any header forwarding needed.
If you decline, I will set up the complete CI environment for it and the
CMT will be forced to
continue to maintain it.