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:07
@grafikrobot
Copy link
Member Author

Please review and merge this PR at your earliest convenience.

@grafikrobot grafikrobot reopened this Apr 13, 2025
@grafikrobot grafikrobot marked this pull request as draft April 15, 2025 00:17
@grafikrobot grafikrobot reopened this May 16, 2025
@grafikrobot grafikrobot closed this Jun 6, 2025
@grafikrobot grafikrobot reopened this Jun 6, 2025
@pdimov
Copy link
Member

pdimov commented Jun 26, 2025

Looks like the installation of software-properties-common hangs because it waits for user input. Why do we need to install it anyway?

@pdimov
Copy link
Member

pdimov commented Jun 26, 2025

gcc-9 is the system compiler on ubuntu 20.04, and gcc-11 is the system compiler on ubuntu-22.04, so just switch to those instead of trying to install them on the previous releases (which doesn't work because the packages aren't present.)

@grafikrobot grafikrobot marked this pull request as ready for review June 26, 2025 19:05
@pdimov pdimov merged commit 645574e into boostorg:develop Jun 26, 2025
29 of 30 checks passed
@pdimov
Copy link
Member

pdimov commented Jun 26, 2025

I'm merging this as-is, but see boostorg/system#132. Maybe we need to make this header-only as well.

@grafikrobot
Copy link
Member Author

I'm merging this as-is, but see boostorg/system#132. Maybe we need to make this header-only as well.

Probably.

@JeffGarland
Copy link
Collaborator

I'd remove the stub -- people have had years to convert now we should make them move on.

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.

3 participants