Conversation
Changed the coolprop version and source according to pyodide#4028 (comment)
|
Thanks @dvd101x! |
|
You'll need to merge main to get #4396 which fixes the build. |
|
Thanks! I did the merge. |
Attempting to fix error of no file or directory "source" the new source is called "CoolProp-6.6.0"
|
Hi Hood, Thanks for the help with the merge suggestion. Sadly I might need help on this one. I'm really new to this. Could you give me some pointers on my mistake? |
Not sure, looks unrelated to you since sourmash doesn't depend on coolprop. I restarted the CI, hopefully it was a flake. If not a flake and there was a change outside of this repo that caused sourmash build to persistently fail, then you could try disabling sourmash to see what happens with coolprop. |
|
Okay on rerun we get an actual failure: I guess when I get a chance I'll have to try to build it locally and see what happens. Did you build it locally and succeed or are you unable to do local builds? |
|
I can reproduce the failure locally. It seems that Coolprop does some file rearrangement on its sdist when you run |
|
Could you try removing these two line and see what happens? pyodide/packages/coolprop/meta.yaml Lines 13 to 14 in e586e7c The coolprop archive from Sourceforge and from PyPI have different structures. I guess the line above was to workaround for the Sourceforge archive so you might not need it for PyPI archive. |
|
I tried removing the lines for setup.py but it seems to fail quicker. I don't have a setup ready to reproduce locally, but there is also a newer version in sourceforge that I could try instead of the one from GitHub. |
|
Most recent failure was in scikit-image with: Which looks like another flake regarding some sort of failed http request unrelated to this PR. I don't know why we're getting so many failed requests on this PR, it's not usually very common. Anyways I reran it. |
|
The build was successful, but there are test failures. Could you check and fix the test failure? |
|
It seems it was missing some sources as mentioned in here, had to revert back to sourceforge |
Updated the coolprop version and changed source according to #4028 (comment)
Description
Checklists