Skip to content

Replace use of boost/iterator/detail/enable_if.hpp #470

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 1, 2025

Conversation

pdimov
Copy link
Member

@pdimov pdimov commented Jan 28, 2025

boost/iterator/detail/enable_if.hpp has been deleted. Replace with boost/type_traits/enable_if.hpp.

@pdimov
Copy link
Member Author

pdimov commented Jan 30, 2025

Note that Boost.Python currently fails to build without this PR, so it should be applied sooner rather than later, and preferably merged to master as well.

@OgreTransporter
Copy link

: boost::iterators::enable_if<is_object_operators<L,R>, T>

Or use std::enable_if instead of boost::iterators::enable_if.

@pdimov
Copy link
Member Author

pdimov commented Feb 1, 2025

I'm merging this to unblock the Boost CI which fails because Boost.Python fails to build.

@pdimov pdimov merged commit 7b19604 into develop Feb 1, 2025
35 of 39 checks passed
@francesco-ballarin
Copy link

@pdimov can this be backported to master? The goal would be for the update to go through to the master branch at https://github.com/boostorg/boost.

Thanks,
Francesco

@pdimov
Copy link
Member Author

pdimov commented Feb 6, 2025

@stefanseefeld what's your merge to master policy? I don't want to do this without your approval.

@stefanseefeld
Copy link
Member

@pdimov thanks for asking, let me take care of it right away...done.

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.

4 participants