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

Update anchor to 0.30 and solana-program to 1.18 #208

Draft
wants to merge 34 commits into
base: main
Choose a base branch
from
Draft

Conversation

wjthieme
Copy link
Member

@wjthieme wjthieme commented Aug 15, 2024

Just keeping this here in case we might need it in the future. Will leave it as a draft since we'll probably skip this version and go straight to 0.31.

  • Anchor changed something which makes the address constraint only work with either a constant or a property of a different account (but not with other expressions). I’ve changed them to use plain constraint instead.
  • Had to Box a couple more accounts to fix Access violation in stack frame. We are getting close to what we can do with Box'ing accounts 😢
  • Since we use the rent sysvar only for cpi and the account data from sysvar rent is not used in our program, I changed the type to UncheckedAccount which is only takes 8 stack space as opposed to Sysvar taking 32.
    Anchor builds idl in snake_case. Need to convert it back to camelCase to keep sdk running with minimal changes.
    Integration test results:
$anchor test
  2122 passing (38m)
  4 pending

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