Skip to content
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

Pyo3 0.23.4 broke the river build? #5321

Closed
hoodmane opened this issue Jan 13, 2025 · 5 comments · Fixed by #5325
Closed

Pyo3 0.23.4 broke the river build? #5321

hoodmane opened this issue Jan 13, 2025 · 5 comments · Fixed by #5325
Labels
bug Something isn't working
Milestone

Comments

@hoodmane
Copy link
Member

error[E0658]: use of unstable library feature `ptr_fn_addr_eq`                  
  -->                                                                           
/root/.cargo/registry/src/index.crates.io-6f17d22bba15001f/pyo3-0.23.4/src/inter
nal_tricks.rs:56:9                                                              
   |                                                                            
56 |         std::ptr::fn_addr_eq(f, g)                                         
   |         ^^^^^^^^^^^^^^^^^^^^                                               
   |                                                                            
   = note: see issue #129322 <https://github.com/rust-lang/rust/issues/129322>  
for more information                                                            
   = help: add `#!` to the crate attributes to enable                           
   = note: this compiler was built on 2024-11-30; consider upgrading it if it is
out of date

https://circleci.com/api/v1.1/project/github/hoodmane/pyodide/101450/output/106/0?file=true&allocation-id=67853a4609d7764763a89cc2-0-build%2FABCDEFGH

@hoodmane hoodmane added the bug Something isn't working label Jan 13, 2025
@hoodmane hoodmane added this to the 0.27.1 milestone Jan 13, 2025
@agriyakhetarpal
Copy link
Member

cc: the recipe maintainer, @MaxHalford, for visibility

@AdilZouitine, I remember you were working on the Rust part of the River codebase last month, at the time of our NumPy v2 update. I hope it is okay to ping you here in case you'll be able to help :)

@hoodmane
Copy link
Member Author

Per the error message the code that's failing is in pyo3. Presumably we just have to update the rust compiler though:

this compiler was built on 2024-11-30; consider upgrading it if it is out of date

@agriyakhetarpal
Copy link
Member

Ah, didn't notice that – sorry for the noise!

@hoodmane
Copy link
Member Author

The commit that introduced the problem is:
PyO3/pyo3#4816
It requires this commit to work:
rust-lang/rust#133678
So we need a nightly more recent than December 1st 2024.

@hoodmane
Copy link
Member Author

I think the best thing would be to pin pyo3 < 0.24.4 for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants