-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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 "pdgraster" #26491
Add "pdgraster" #26491
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/pdgraster:
|
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/pdgraster:
|
Hi, confirming my interest to be a maintainer for this package, thanks! |
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 ( |
FYI: Windows build failed due to a dependency ( |
@conda-forge/help-python, ready for review! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excess jinja doesn't really help but it obfuscates the recipe.
Co-authored-by: Filipe <ocefpaf@gmail.com>
Personally, as a feedstock maintainer, I find having all the values that need to regularly change in one place to be helpful. But if that goes against convention, I'm fine with whatever. |
We rely on bot commands for the updates. If the package metadata is well defined, you rarely will need to change those manually. |
I have some feedstocks that don't work with bot commands for various reasons (conda-forge/quarto-feedstock#14), and I expect this might be one of them, because it neither uses GitHub Releases nor PyPI releases. |
The failure seems to be a missing log file:
The noarch package will be installable on Windows and broken there. Idealy, we should not make it installable by using os-noarch. See https://conda-forge.org/docs/maintainer/knowledge_base/#noarch-packages-with-os-specific-dependencies. Do you want to implement that or try to mint a new release upstream that doesn't have this issue? |
This is recorded as an issue in the upstream package, but myself and Rushiraj are not members of that team, we're just consumers at the moment. We expect to become more integrated with that team in October. For now, it makes sense to make this package not installable under Windows as you suggested, but I'm not sure I understand how from the link you shared, as our issue isn't os-specific dependencies, it's a bug in the code. Can you clarify? I imagined that we would leave the library as installable but broken in Windows, because that is also the state if you install it with pip. |
Correct, Python noarch packages are built on Linux but installable in all platforms and, in this case, broken on Windows. You need to add noarch_platforms:
- linux_64
- osx_64 Telling it not to build on Windows. |
Thanks for clarifying. I'll take that path. I'll push a commit to update 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/pdgraster:
|
You can add it to the recipe directory and it will be copied over to the feedstock. |
Neato! TIL, thanks :) |
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 ( |
It's expected that this didn't affect the staged-recipes build? |
Depends on #26489
Important
Not compatible with Windows yet.
cc @rushirajnenuji
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).