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

Anchor 0.29.0 support #121

Merged
merged 9 commits into from
Feb 19, 2024
Merged

Anchor 0.29.0 support #121

merged 9 commits into from
Feb 19, 2024

Conversation

Ikrk
Copy link
Contributor

@Ikrk Ikrk commented Feb 8, 2024

This PR introduces changes necessary to support Anchor 0.29.0.

In this version Anchor merged the PR coral-xyz/anchor#2656 which changes the way how Accounts are handled. They are not cloned anymore but rather referenced which changed the signatures of methods/functions on many levels. Therefore the snapshot struct in Trdelnik was reworked to support the new handling.

Also, the the solana-program-test crate is broken, because the entry function generated by Anchor does not have the required signature anymore. There is an issue here: coral-xyz/anchor#2711 The temporary workaround solution was to use this fixed fork of solana-program-test: dankelleher/solana@3c285b5 and to change the dependency to solana-program-anchor-fix

TODOs:

  • merge open PRs
  • rebase this branch
  • finish the snapshot_generator.rs accordingly
  • update the examples (maybe within separate PR)

anchor-spl was added as dependency to explicitly activate the idl-build feature that is required because also anchor-lang has this feature activated. This way, the developers do not have to do it manually.
@Ikrk Ikrk marked this pull request as ready for review February 19, 2024 11:18
* ✅ update fuzz_example0

* ✅ update fuzz_example1

* ✅ update fuzz_example2

* ✅ update fuzz_example3

* ✅ re-initialized escrow

* ✅ re-initialized turnstile
@lukacan lukacan merged commit a5b5afb into develop Feb 19, 2024
7 checks passed
@lukacan lukacan deleted the anchor-0.29.0 branch February 19, 2024 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants