Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Upgrade to Solana
1.17.0
Problem
Upgrading Solana version in Anchor repo to
1.17.0
(coral-xyz/anchor#2645) butauction-house
program doesn't work after1.16
. We were using1.14
to run theauction-house
tests but now that we're dropping the1.14
support, that's not an option anymore.Problem 1
solana-labs/solana#31960 affected both the
auction-house
program and themetaplex-token-metadata
program.The solution to the
metaplex-token-metadata
program's error was to compile the program with1.14
and use the pre-built binary for tests, and the solution for theauction-house
program's error was to upgrade the Solana dependencies(solana-labs/solana#31960 (comment)).Problem 2
Upgrading Solana dependencies solved the previous error but it also resulted with a compile error on
metaplex-token-vault
crate.The solution was to remove the
metaplex-token-metadata
dependency and instead use themetadata
feature ofanchor-spl
to get access to the necessary functionality(mpl-token-metadata
is now exported coral-xyz/anchor#2583).Problem 3
After solving some more compile errors, the last error was in the
execute_sale
instruction which was out of memory. Apparentlyplatform-tools v1.37
(solana ^1.16.0
) uses more memory for the same instruction.The solution was to use references instead of cloning and remove the
rent
account from accounts.