-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
singularity v2.5.0 #6
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 ( |
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipe:
|
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 ( |
I think we need |
@jakirkham if we ust host, do we need runtime dependency then. Assuming libarchive would had a run_export? |
Nothing has been built with |
This is an intermediate phase which is done in preparation for switching to Anaconda's compilers. Here we use the `compiler` syntax though it references the `toolchain`. Also this starts to ship the compiler run time with built packages on Linux in particular.
Currently failing as it thinks ref: https://circleci.com/gh/conda-forge/singularity-feedstock/14 |
On a different note, am I using the |
@jakirkham, you need to rerender with the dev version of conda-forge-pinning |
Oops. 😬 Yep, that makes sense. Thanks @isuruf. |
Thanks again @isuruf. Turns out that fixed both issues. 🎉 Worth noting then that the Anaconda Could you please review when you have a chance @bgruening? Should add that this pulls in |
Looks good to me, I just realized I need to learn a lot about the new system :) |
I don't know without investigating. The compilers provide their own system headers for glibc the kernel and the compiler runtimes though to isolate it from the build machine. At a guess (not on a computer to check) this package requires a kernel feature that CentOS6 doesn't provide. |
Re-render to leverage the `compiler` syntax with `toolchain`.
So CentOS 6 does provide the required features. Though that might not seem obvious at first. Have outlined the info regarding the NO_NEW_PRIVS prctl functions in this comment. These are confirmed present in CentOS 6.7+ (we are on CentOS 6.9) by both Red Hat and Singularity. The header it is looking for is |
This PR was created by the cf-regro-autotick-bot.
The cf-regro-autotick-bot is a service to automatically track the dependency graph and propose package version updates for conda-forge. It is very likely that the current package version for this feedstock is out of date.
Notes and instructions for merging this PR:
If you would like a local version of this bot, you might consider using rever. Rever is a tool for automating software releases and forms the backbone of the bot's conda-forge PRing capability. Rever is both conda (
conda install -c conda-forge rever
) and pip (pip install re-ver
) installable.Finally, feel free to drop us a line if there are any issues!