-
-
Notifications
You must be signed in to change notification settings - Fork 19
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
libcxx v15.0.4 #104
libcxx v15.0.4 #104
Conversation
…nda-forge-pinning 2022.11.08.15.44.59
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 ( |
This is still going to need more work, see discussion in #98, in particular this:
|
ca8d2e8
to
1662180
Compare
1662180
to
0ad8688
Compare
@isuruf regarding your comment in #98:
This now builds, including the Beyond that, it looks libcxx is now using C11 features which aren't available in glibc 2.12 (this only appeared when activating the tests, which are removed again now)
e.g. Funnily enough, trying to build with a newer sysroot still fails during our tests with:
(I presume this is because the sysroot requirement is not propagated well enough). I can raise an discussion on the LLVM discourse about what the glibc lower-bound for libcxx is supposed to be, but I tend to think we should bite the bullet on bumping this (as otherwise, stuff on centos 6 might break at runtime). That C11 requirement is also hitting us on OSX, where some C11 symbols aren't found:
I don't think we can change this with our usual availability hacks (as it's the C library, not libcxx), but I haven't managed to google which SDK added these functions. PS. Switching on the upstream test suite crashes on linux, and runs into a known issue on osx (because we don't seem to be picking up their skips). |
OK, it seems this should work based on llvm/llvm-project@21f73d5, which says:
|
@isuruf any input on how to deal with this? This is the last missing piece for LLVM 15. |
OK, I've dealt with this by fully reverting llvm/llvm-project@21f73d5 in #108. |
It is very likely that the current package version for this feedstock is out of date.
Checklist before merging this PR:
license_file
is packagedInformation about this PR:
@conda-forge-admin,
please add bot automerge
in the title and merge the resulting PR. This command will add our bot automerge feature to your feedstock.bot-rerun
label to this PR. The bot will close this PR and schedule another one. If you do not have permissions to add this label, you can use the phrase@conda-forge-admin, please rerun bot
in a PR comment to have theconda-forge-admin
add it for you.Closes: #103
Pending Dependency Version Updates
Here is a list of all the pending dependency version updates for this repo. Please double check all dependencies before merging.
Dependency Analysis
We couldn't run dependency analysis due to an internal error in the bot. :/ Help is very welcome!
This PR was created by the regro-cf-autotick-bot. The regro-cf-autotick-bot is a service to automatically track the dependency graph, migrate packages, and propose package version updates for conda-forge. Feel free to drop us a line if there are any issues! This PR was generated by https://github.com/regro/autotick-bot/actions/runs/3422937571, please use this URL for debugging.