-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
- [boost] fix libiconv detection (needed by boost locale) for iOS, Android #7146
Conversation
Signed-off-by: SSE4 <[email protected]>
Signed-off-by: SSE4 <[email protected]>
I detected other pull requests that are modifying boost/all recipe:
This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there. |
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.
Having someone create a corresponding PR in upstream B2 would be nice too (https://github.com/bfgroup/b2/pulls)
opened one: bfgroup/b2#83 |
This comment has been minimized.
This comment has been minimized.
stacktrace again. :-) @SSE4 What do you think, should we simple disable stacktrace for ARM? |
I will take a look again tomorrow. AFAIK the error was suspicious and I wasn't able to reproduce, but let's check if I can do anything about it. |
okay, I cannot reproduce the problem locally, so will try to run some tests to understand on what's going on... first off,
so, it's conditional on
so, it's basically parsed version from
I'll check what does it report on CI, as well as all these CMake variables and let you know. |
okay, I see
|
okay, this is definitely a CMake issue...
checking more versions, I see that 3.20.0 doesn't work, but 3.20.1 works. TLDR: we need to upgrade CMake on macOS at least to 3.20.1. |
I can reproduce the problem locally but with a different error message (maybe CCI forces new toolchain?). It has also not worked for iOS/Android for a long time and now not for Apple M1. Maybe a cross compile issue? Cross-build from 'Macos:x86_64' to 'Macos:armv8boost/1.76.0: Applying build-requirement: b2/4.5.0
[1] clng-12.0 Component configuration:
boost/1.76.0: Package 'a731471530a71672c8c4f825a0c8e289720899b3' built |
…nough cmake for now to workaround CMAKE_SHARED_LIBRARY_RUNTIME_C_FLAG problem Signed-off-by: SSE4 <[email protected]>
Signed-off-by: SSE4 <[email protected]>
7d75ee2
to
3e05a76
Compare
@dmn-star let's check, I was told we can't upgrade CMake in CCI right away, so we have to use a workaround with build_requires("cmake/3.20.1") for M1. hopefully, CI will be happy after that. |
I also noticed.
@SSE4 But as you can see, I don't use a CMake toolchain yet. |
yeah, I had this error as well. conan-center-index/recipes/boost/all/conanfile.py Line 1549 in f833d02
for the stacktrace component, at least.but I think stacktrace_addr2line_exe shouldn't be built for iOS/Catalyst at all, as well as Boost::stacktrace_addr2line component shouldn't be provided.
|
agree. PS: Android build OK -- Build files have been written to: /Users/dmn/dev/temp/sse4/conan-center-index/recipes/boost/all/test_package/build/e68bc1eac8c9d2d506d3275c11b577f6c2c2b1f1
|
This comment has been minimized.
This comment has been minimized.
Signed-off-by: SSE4 <[email protected]>
Signed-off-by: SSE4 <[email protected]>
Signed-off-by: SSE4 <[email protected]>
Checked this morning "is being built". My old PC needs 3min. to build boost from scratch. Or does CCI have a weekend as well? :-) |
I've tried cross-build to iOS in boost/1.76.0 with this PR, and it works now. |
All green in build 10 (
|
interesting... thanks a lot for checking! |
Thanks to you for the excellent work. 👍 |
closes: #6490
closes: #3992
closes: #7114
closes: #7159
closes: #5265
Specify library name and version: boost/all
/cc @grafikrobot
This is also a good place to share with all of us why you are submitting this PR (specially if it is a new addition to ConanCenter): is it a dependency of other libraries you want to package? Are you the author of the library? Thanks!
conan-center hook activated.