Skip to content
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

python3-pybind11: Remove the Boost dependency #791

Open
wants to merge 1 commit into
base: kirkstone
Choose a base branch
from

Conversation

manvarpratik
Copy link
Contributor

Boost is an optional dependency and it is only used in tests which are disabled by default in this recipe.

References:
https://github.com/pybind/pybind11/blob/master/tests/CMakeLists.txt#L325

Moreover, the Boost is an enormously large and complex suite of utility libraries that cost the filesystem memory footprint of Embedded devices.

Boost is an optional dependency and it is only used in tests which are
disabled by default in this recipe.

References:
https://github.com/pybind/pybind11/blob/master/tests/CMakeLists.txt#L325

Moreover, the Boost is an enormously large and complex suite of utility
libraries that cost the filesystem memory footprint of Embedded devices.

Signed-off-by: Christian Ege <[email protected]>
Signed-off-by: Pratik Manvar <[email protected]>
@manvarpratik manvarpratik force-pushed the kirkstone-remove-boost-dependency-for-python3-pybind11 branch from 4176198 to cc7550e Compare February 16, 2024 13:30
@kraj
Copy link
Contributor

kraj commented Feb 16, 2024

it should go to master branch first, I have cherry-picked to staging branch ( master-next ) into master since it is a simple cherry-pick, in future always check if the patch is needed on master branch first.

@manvarpratik
Copy link
Contributor Author

it should go to master branch first, I have cherry-picked to staging branch ( master-next ) into master since it is a simple cherry-pick, in future always check if the patch is needed on master branch first.

Sure, I will keep this in mind. Thank you!

@kraj
Copy link
Contributor

kraj commented Feb 19, 2024

applied to master with 086eeb4

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