Skip to content

Conversation

@grafikrobot
Copy link
Member

@grafikrobot grafikrobot commented Jul 20, 2024

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.

@grafikrobot grafikrobot marked this pull request as ready for review August 18, 2024 15:21
@grafikrobot
Copy link
Member Author

Please review and merge this PR at your earliest convenience.

@grafikrobot grafikrobot reopened this Apr 13, 2025
@pdimov
Copy link
Member

pdimov commented May 2, 2025

Too much stuff commented out in CI; while I understand why it's done, it's not for me to merge such changes.

@grafikrobot
Copy link
Member Author

Okay then.. @glenfe please merge this PR.

@pdimov
Copy link
Member

pdimov commented Jun 26, 2025

Way too many unnecessary CI changes... what's the point of renaming cxxstd to standard, for instance? This just generates diffs that have nothing to do with the point of the PR.

@grafikrobot
Copy link
Member Author

Way too many unnecessary CI changes... what's the point of renaming cxxstd to standard, for instance?

As usual.. I just wanted things to work hence copy pasted from something else.

This just generates diffs that have nothing to do with the point of the PR.

LOL I've been doing CI changes only for months. The point of the PR was done long, long, ago. ;-)

@pdimov pdimov merged commit c692e40 into boostorg:develop Jun 26, 2025
22 of 23 checks passed
@pdimov
Copy link
Member

pdimov commented Jun 26, 2025

I merged this before noticing that Appveyor needs a fix too.

@grafikrobot
Copy link
Member Author

I merged this before noticing that Appveyor needs a fix too.

I'll make another PR to try and fix Appveyor.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants