-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
faker-cxx: add recipe #24018
faker-cxx: add recipe #24018
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@RubenRBS
Since faker-cxx has several large source files, CCI seems to kill the compiler process. |
@toge We could add a larger container to this project. Let me check it. |
Any progress on the project? @uilianries |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@toge The faker-cxx is not prepared to build with shared + Windows. It can be dropped. I'll check with the upstream. |
Co-authored-by: Uilian Ries <uilianries@gmail.com>
Co-authored-by: Uilian Ries <uilianries@gmail.com>
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.
LGTM. Thank you for packaging this project!
Conan v1 pipeline ✔️All green in build 11 (
Conan v2 pipeline ✔️
All green in build 11 (
|
@meftunca The PR is ready now. The problem was in the upstream, as it required tons of RAM to be built using GCC. For instance, in my Linux, it stopped and took frozen after consuming 32 GB of RAM. Fortunately, the author improved the project a lot and now does not require that huge amount of memory. |
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.
<3 - great follow up @uilianries :)
Specify library name and version: faker-cxx/*