Add support for modular build structure. #1292
Conversation
… originally deleted!
…a source in install targets.
build.jam
Outdated
| <library>/boost/mpl//boost_mpl | ||
| <library>/boost/multiprecision//boost_multiprecision | ||
| <library>/boost/numeric_conversion//boost_numeric_conversion | ||
| <library>/boost/polygon//boost_polygon |
There was a problem hiding this comment.
Is it possible to distinguish between real dependencies (the library uses it)
and other dependencies?
Other depencie scan be:
- optional adaptations
- dependency for test only
- dependency for examples only
There was a problem hiding this comment.
Probably not that easy and this is probably generated.
I'm doubting how we should organize our library such that this is more clear.
For the test, it looks good to me, thanks for your efforts!
There was a problem hiding this comment.
We can investigate making tweaks after the initial merge. Optional deps are difficult to deal with. Especially since a goal of these changes is to make it possible for more automatic package manager use. So optional things need a human touch to put in appropriate logic. Maybe add a git issue to investigate later?
There was a problem hiding this comment.
Not sure what you mean by dependencies for test or examples. These are the dependencies as listed here https://pdimov.github.io/boostdep-report/develop/geometry.html .. Which are only for the public library headers.
There was a problem hiding this comment.
All right, it's fine for now, you made already a huge effort in all libraries. Thanks again, great work.
We will look at it ourselves later.
|
FYI.. There are more changes coming. Because of general comments from other PRs. So you get to approve again. :-) |
OK I will review it again when in "ready for review" state. Thanks again. |
|
Please review and merge this PR at your earliest convenience. |
|
Still looks great to me, I'll merge. |
This is part of the effort to make the Boost libraries "modular" for build and consumption. See https://lists.boost.org/Archives/boost/2024/01/255704.php and https://github.com/grafikrobot/boost-b2-modular/blob/b2-modular/README.adoc for more information.
This PR depends on the following other PRs being merged to both develop and master branches of the respective repos:
This PR will be changed to ready for review, i.e. not draft, when the above are merged. Do not merge this one until that time.