Skip to content

Conversation

@sloretz
Copy link
Contributor

@sloretz sloretz commented Jan 24, 2022

I made a mistake in #421 - when boost is less than 1.67 use the Boost::python3 target instead of the Boost::pythonXY target.

This time in addition to testing this PR by building and testing cv_bridge on Focal locally, I also ran this CMake code in a stub project on almalinux 8 to make sure it works with boost 1.66

@sloretz sloretz added the bug label Jan 24, 2022
@sloretz sloretz requested a review from mjcarroll January 24, 2022 22:52
@sloretz sloretz self-assigned this Jan 24, 2022
@mjcarroll mjcarroll merged commit f54a269 into ros2 Jan 24, 2022
@sloretz sloretz deleted the sloretz__boost_1_67_python3 branch January 24, 2022 23:16
ijnek added a commit that referenced this pull request Sep 17, 2022
This reverts commit f54a269.

Signed-off-by: Kenji Brameld <[email protected]>
mergify bot pushed a commit that referenced this pull request Sep 17, 2022
This reverts commit f54a269.

Signed-off-by: Kenji Brameld <[email protected]>
(cherry picked from commit 822069c)
Ace314159 added a commit to Ace314159/vision_opencv that referenced this pull request Sep 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants