-
-
Notifications
You must be signed in to change notification settings - Fork 5k
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
Add graalpy package to make Graalpy available as a Python implementation #20882
Conversation
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 recipes/graalpy:
For recipes/graalpy:
Documentation on acceptable licenses can be found here. For recipes/graalpy-meta:
For recipes/graalpy-meta:
Documentation on acceptable licenses can be found here. |
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 recipes/graalpy:
For recipes/graalpy-meta:
|
Hi, I confirm I want to be listed as a maintainer of graalpy packages. |
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 recipes/python_abi:
|
It looks like the docker build runs out of memory building graalpy. Anything we can do there? @isuruf I included the changes to python_abi in this PR, because the changes to the abi must be available to the |
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 recipes/graalpy:
For recipes/python_abi:
|
I don't understand why I get these builds in the log
There shouldn't be any variants, why am I not simply getting a single build? |
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 recipes/graalpy:
For recipes/python_abi:
|
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 recipes/python_abi:
|
@conda-forge-admin, please ping conda-forge/help-python |
Hi! This is the friendly automated conda-forge-webservice. I was asked to ping @conda-forge/help-python and so here I am doing that. |
Some explanation: There are two distributions for GraalPy: a) in a GraalVM, which means it runs the Java code on top of the Graal compiler and b) as a standalone native executable generated with GraalVM native image. The latter is created via a static analysis of the entire codebase, which takes about 11GB of RAM, too much for the Azure runners here. So I disabled the standalone distribution for now and only build the GraalVM-based distribution. The standalone should be preferred, which I've tried to express via the The python_abi maintained by @isuruf needs to be updated to be able to select GraalPy as a python implementation for other Python packages. Because there's a bit of a circular dependency here, with GraalPy providing the packages
Are there better ways? |
Co-authored-by: Isuru Fernando <isuruf@gmail.com>
Co-authored-by: Isuru Fernando <isuruf@gmail.com>
317fafc
to
38601b0
Compare
Co-authored-by: Isuru Fernando <isuruf@gmail.com>
And just as I wrote this, I saw the new commit :) Sorry for the noise, I just thought maybe I was expected to already install pip in the graalpy prefix. |
@isuruf thanks for the help with updating it. it played with the current recipe locally with various noarch packages, and it seems to work fine, that's great :) |
@conda-forge/openjdk, @conda-forge/python, we'll be adding variants of |
Hi, and happy new year :) Was there anything from the Python or Openjdk teams? (I don't see @isuruf's comment linking those, so not sure if the notification worked for those teams) |
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).