Skip to content
This repository has been archived by the owner on Mar 7, 2023. It is now read-only.

Integrate with Substrate by making yatima_core no_std (using the sp_std lib) #38

Closed
7 tasks done
johnchandlerburnham opened this issue May 26, 2021 · 2 comments · Fixed by #129
Closed
7 tasks done

Comments

@johnchandlerburnham
Copy link
Member

johnchandlerburnham commented May 26, 2021

Since the Yatima's core typechecking and evaluation via the λ-DAG machine only requires rust core, we should be able to refactor to build with only sp_std:

  • libipld (this is probably the most work, requiring forking our own sp_std version)
  • nom
  • im (im is unmaintained, we should remove this dep Is this crate still maintained? bodil/im-rs#185)
  • num-bigint
  • ropey
  • base-x (we should inline this)
  • petgraph (should move this out of yatima_core, tbh)

We will also need to switch over to using https://docs.rs/sp-std/3.0.0/sp_std/collections/index.html everywhere for Maps, Sets, etc.

@samuelburnham
Copy link
Member

multibase and nom_locate are working on their forks after disabling std as a default feature. It seems they were meant to be used in a no_std environment with cargo build --no-default-features, but this didn't work for me when using them as a dependency.

@dlight
Copy link
Contributor

dlight commented Aug 16, 2021

There's a maintained fork of im called imbl.

@samuelburnham samuelburnham linked a pull request Oct 4, 2021 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants