-
Notifications
You must be signed in to change notification settings - Fork 431
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
BUG: 'PYTHON' is undefined for multi-output builds if some outputs are generic #5424
Open
2 tasks done
Labels
type::bug
describes erroneous operation, use severity::* to classify the type
Comments
8 tasks
5 tasks
5 tasks
What's the workaround for this? |
jamesmyatt
added a commit
to jamesmyatt/pytorch_geometric-feedstock
that referenced
this issue
Feb 17, 2025
Also work around conda/conda-build#5424
Apparently, this works. |
mikemhenry
pushed a commit
to conda-forge/pytorch_geometric-feedstock
that referenced
this issue
Feb 25, 2025
…ages (#51) * Split into sub-packages to match upstream * Fix test * MNT: Re-rendered with conda-build 25.1.2, conda-smithy 3.45.4, and conda-forge-pinning 2025.02.17.08.22.26 * Refactor recipe Also work around conda/conda-build#5424 * Simplify recipe Based on https://github.com/conda-forge/seaborn-feedstock/blob/a0d4a037b784b3dd7b727acb772e8b6f1f266bc4/recipe/meta.yaml * Use noarch: generic like https://github.com/conda-forge/importlib_metadata-feedstock/blob/274403de0366c130e7ef545f13c433ad1b3cd233/recipe/meta.yaml#L41 * Try just repeating the python dependencies * Add pytorch-cpu to test dependencies * Explicit about everything * latest pytorch builds are python 3.9+ * Test against pytorch 2.5.1 This is the first version with builds for every platform on conda-forge * Remove upper pins To make it build * pgmpy not available on windows * Avoid selector * Add jamesmyatt to maintainers * Update following review Fixes #52 * Fix subpackage pin --------- Co-authored-by: conda-forge-webservices[bot] <91080706+conda-forge-webservices[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Checklist
What happened?
Consider the following recipe:
I don't see why this recipe should not be able to be built.
However, conda build fails with
Error: Failed to render jinja template in /home/conda/recipe_root/meta.yaml: 'PYTHON' is undefined
Apparently, this happens because the entire jinja template is rendered for every output but
PYTHON
is only set if the currently-processed output has Python as a host dependency.I think a better behavior would be to strip out details about other outputs including the variables they reference before rendering with patched metadata dependent on individual outputs.
Conda Info
No response
Conda Config
No response
Conda list
No response
Additional Context
No response
The text was updated successfully, but these errors were encountered: