-
Notifications
You must be signed in to change notification settings - Fork 6
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
Adding support for fetching boost #10
Conversation
Signed-off-by: PritK99 <pritpjk@gmail.com>
@PritK99 you'll have to change the CI for all the platforms and remove boost installation. |
Have a look in the .github folder and look for workflows. |
CMakeLists.txt
Outdated
include(FetchContent) | ||
FetchContent_Declare( | ||
Boost | ||
GIT_REPOSITORY https://github.com/boostorg/boost.git |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we enable only some part of boost for spirit instead of cloning all of it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we fetch only https://github.com/boostorg/spirit.git and https://github.com/boostorg/regex.git instead of fetching https://github.com/boostorg/boost.git as whole ?
The code builds in ubuntu if we just fetch boost for spirit and regex.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you try to fetch spirit only and please let me know?
Signed-off-by: PritK99 <pritpjk@gmail.com>
Signed-off-by: PritK99 <pritpjk@gmail.com>
Added code to fetch boost spirit only instead boost as whole. |
Thanks, just two more things.
|
Signed-off-by: PritK99 <pritpjk@gmail.com>
Merged in #16 |
Fixes
#7
#9 (comment)