MNT: rerender #127
Merged
MNT: rerender #127
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
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 ( |
conda-forge-admin
requested review from
ax3l and
franzpoeschel
as code owners
|
@franzpoeschel I see this compile error on conda-forge on macOS for 0.16.1: Do we already have a patch for that? Update: found one openPMD/openPMD-api#1762 |
Typo caught by latest GCC (unused)
|
Found one: openPMD/openPMD-api#1762 |
1 task
|
Ok, PPC64le is failing with a segfault. Could be ignored. Win64 still has some install syntax error: |
ax3l
force-pushed
the
conda_forge_admin_126
branch
from
5e22390 to
12f9d38
Compare
|
@isuruf Can we see / test somehow if this PR will solve the hard-coded pthread issue? |
|
You can grep the cmake file to see if |
Windows error: ``` │ (base) %SRC_DIR%>python -c 'import openpmd_api; print(openpmd_api.__version__)' │ File "<string>", line 1 │ 'import │ ^ │ SyntaxError: unterminated string literal (detected at line 1) ```
ax3l
force-pushed
the
conda_forge_admin_126
branch
from
6c66d6f to
0d67f98
Compare
ax3l
force-pushed
the
conda_forge_admin_126
branch
from
0d67f98 to
a0462b0
Compare
|
@isuruf I see it in the arm64 files, which one would be problematic? details |
|
|
|
Oh no, I see: Is the right way to depend on the |
|
Actually |
|
@isuruf let me please know if there is something one can do in CF about these paths. In openPMD-api, we do not actively search and link |
|
Using sed to replace them would be the easiest thing. These paths are added by Something like |
|
Thank you, will do! And the same, explicit paths in there to |
They should be fixed the same way pthread is fixed too. |
ax3l
force-pushed
the
conda_forge_admin_126
branch
from
8db5ed3 to
73cfdd6
Compare
ax3l
force-pushed
the
conda_forge_admin_126
branch
from
73cfdd6 to
078449f
Compare
... after patch, they should be gone
|
Thank you so much, @isuruf! That should cover it! 🙏 Really appreciate your insight and help. |
|
@isuruf downstream in WarpX and ImpactX feedstocks I now see │ ld: library not found for -lz.tbd I see HDF5 has zlib in its host section though, why would this now not be propagated? |
This was referenced Sep 22, 2025
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi! This is the friendly automated conda-forge-webservice.
I've started rerendering the recipe as instructed in #126.
If I find any needed changes to the recipe, I'll push them to this PR shortly. Thank you for waiting!
Here's a checklist to do before merging.