-
-
Notifications
You must be signed in to change notification settings - Fork 14
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
Build with clang 18 #367
base: main
Are you sure you want to change the base?
Build with clang 18 #367
Conversation
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
Confirmed. Building with clang 18 reproduces the error on osx-arm64 we observed in the nightly feedstock builds |
…nda-forge-pinning 2024.11.10.22.41.52
Rebased onto main to include #370 which installs capnproto from conda-forge, which should bypass the compilation error with clang 18 |
Making progress. Previously it failed during the configuration step when trying to compile capnproto (link to old build log that installed capnproto via vcpkg):
Now that capnproto is installed from conda-forge, the configuration step completed successfully (latest build log). Unfortunately it now fails early in the build step. Looks like a libfmt-related error.
|
This looks like it's not finding the C++ standard library, which is a pretty catastrophic place to be for a C++ compiler. 🙈 I don't know why you've set up a wrapper for CXX tiledb-feedstock/recipe/cxx_wrap.sh Lines 1 to 4 in 25ae072
I mean yes, ostensibly to strip -Werror , but you could just do that in build.sh , or fix it in the source CMake files (since you're already patching things anyway). At least, this wrapper raises my hackles immediately as a potential suspect 😅
|
Example (not sure this is all the places where CMake might inject it, but you get the idea): export CFLAGS="${CFLAGS//-Werror/}"
export CXXFLAGS="${CXXFLAGS//-Werror/}" |
Checklist
0
(if the version changed)conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)Test PR to build osx-* binaries with clang 18